-
Notifications
You must be signed in to change notification settings - Fork 162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does it work with ES datastreams? #17
Comments
datastreams need the @applike-ss I read your PR opsgenie#165 and I think it's a good solution. I suggest to send the PR 🙂 . |
a workaround to people willing to use datastreams right now:
|
I realize the question here was for ElasticSearch, but I stumbled upon this issue when trying to make DataStreams work with OpenSearch. Unfortunately it doesn't look like OpenSearch 2.4 has the However, in OpenSearch you can just natively set the DataStream
And here's my config:
logLevel: debug
logFormat: pretty
receivers:
- name: "opensearch"
opensearch:
hosts:
- https://opensearch-cluster.metrics:9200
index: kube-events
username: eventlog
password: XXXXXXXXXXXXXXX
useEventID: false
deDot: true
tls:
...
route:
routes:
- match:
- receiver: "opensearch"
I didn't see any references to the I prefer this solution a lot more than adding an extra field to the Kubernetes Event Exporter source, so I'm curious if ElasticSearch has something similar. It seems a bit silly to totally lock DataStreams to the |
There is no
Maybe I missed something inside the mapping documentation, but as far I know it's not possible to rename a field without a ingestion pipeline. The copy_from is inside a ingestion pipeline, I think this is the equivalent, but I'm not sure: https://opensearch.org/docs/latest/data-prepper/data-prepper-reference/#copy_values
Well, it is hard-coded inside elastic, so it makes sense to hard code here in the patch too. |
For the opstree kubernetes-event-exporter i had to hack together a solution to support elasticsearch datastreams, see: opsgenie#165
I know that this is far from ideal, but i needed something to worked quickly at that point and couldn't spend more time on making it proper.
Has a fix like this (or proper 😉 ) been applied to this fork?
The text was updated successfully, but these errors were encountered: