Skip to content
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

influxdb receiver - allow configurable field separator for joining measurement and field names #34693

Open
clheikes opened this issue Aug 14, 2024 · 3 comments

Comments

@clheikes
Copy link

clheikes commented Aug 14, 2024

Component(s)

receiver/influxdb

Is your feature request related to a problem? Please describe.

When converting to opentelemetry metrics, The influxdb receiver is joining the measurement name and each field with a "_".

Several of the telegraf output plugins are joining measurement and fields with a "." making it difficult to migrate to the open telemetry collector without changing metric names in all of the downstream dashboards and monitors impacted.

Describe the solution you'd like

Allow the field separator to be configurable when ingesting metrics.

receivers:
  influxdb:
    endpoint: 0.0.0.0:8080
    measurementFieldSeparator: "."

Describe alternatives you've considered

We looked at using transforms but our field names have underscores in them as well so there is no simple way to convert these back to a period as a separator.

Additional context

Relavant code

metricName := fmt.Sprintf("%s_%s", measurement, k)

@clheikes clheikes added enhancement New feature or request needs triage New item requiring triage labels Aug 14, 2024
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

Copy link
Contributor

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 @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@atoulme
Copy link
Contributor

atoulme commented Nov 2, 2024

@jacobmarble please take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants