Skip to content

feat(exporters/prometheus): add explicit option to set naming scheme#6927

Draft
juliusmh wants to merge 1 commit intoopen-telemetry:mainfrom
juliusmh:remove_global_name_validation
Draft

feat(exporters/prometheus): add explicit option to set naming scheme#6927
juliusmh wants to merge 1 commit intoopen-telemetry:mainfrom
juliusmh:remove_global_name_validation

Conversation

@juliusmh
Copy link
Copy Markdown

No description provided.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Jun 24, 2025

CLA Not Signed

@dmathieu
Copy link
Copy Markdown
Member

cc @dashpole

I'm not sure this can work. NameValidationScheme is a value that the prometheus/model package uses to for its naming scheme globally. We can't configure a global config locally unless the library allows it.
See https://pkg.go.dev/github.com/prometheus/common/model#pkg-variables

@dmathieu dmathieu added the blocked:CLA Waiting on CLA to be signed before progress can be made label Jun 24, 2025
@dashpole
Copy link
Copy Markdown
Contributor

We are probably going to switch to "translation modes" to match the Prometheus server's OTLP endpoint configuration: open-telemetry/opentelemetry-specification#4533

@juliusmh juliusmh force-pushed the remove_global_name_validation branch from e5a689f to 32a3e65 Compare June 30, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked:CLA Waiting on CLA to be signed before progress can be made

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants