-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Elasticsearchexporter: dynamic index #5854
Comments
index: should support ILM or template... |
Pinging code owners: @urso @faec @blakerouse. See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@djaglowski @JaredTan95 i will be working on it. just telling you to avoid any redundant effort. |
Idea: elasticsearch/log:
endpoints: [http://localhost:9200]
logs_index: myLogIndex
logs_dynamic_index:
disabled: false #default: true
prefix: "staticPrefix-" # both static or attribute/resource based prefix (if one of prefix & suffix does not match, then fall back to logs_index"
suffix: "-{resource['k8s.namespace']}" # both static or attribute/resource based suffix (if one of prefix & suffix does not match, then fall back to logs_index" ex) staticPrefix-myLogIndex-someNamespace or resource attribute based. elasticsearch/log:
endpoints: [http://localhost:9200]
logs_index: myLogIndex
logs_dynamic_index:
disabled: false #default: true which checks resource attribute named "elasticsearchexporter.logs_index_suffix", which is added to default index as a suffix. I actually prefer second one because it prevents many other error prone operations and checks only existence of dynamic or predefined resources What do you think folks? :) @djaglowski @JaredTan95 |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been closed as inactive because it has been stale for 120 days with no activity. |
Is your feature request related to a problem? Please describe.
For future usecase:
According to the existing readme of Elasticsearchexporter, which is inactivated in the current collector version , there seems to be no dynamic index capability e.g. overwriting index name with prefix and namespace etc.
This feature would make another integration component like logstash obsolete and simplifies the overall architecture.
Describe the solution you'd like
Optional configuration: overwriting index name with predefined prefix and value of a ecs field name like elasticsearch plugin in fluentd.
Describe alternatives you've considered
None
Additional context
The text was updated successfully, but these errors were encountered: