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

Prometheus exporter config -> type_suffix needs clarification #4265

Open
psx95 opened this issue Oct 17, 2024 · 0 comments
Open

Prometheus exporter config -> type_suffix needs clarification #4265

psx95 opened this issue Oct 17, 2024 · 0 comments
Labels
sig-issue A specific SIG should look into this before discussing at the spec spec:metrics Related to the specification/metrics directory

Comments

@psx95
Copy link

psx95 commented Oct 17, 2024

What are you trying to achieve?

The current prometheus exporter spec suggests that there can be a configuration - without_type_suffix which if enabled, produces metrics without type_suffix.

IIUC, this only applies to _total that gets added in case counters are used. Unlike units, the possible values of type_suffix are not apparent when reading the metric metadata portion of the spec.

This could cause some confusion where type_suffix could be confused with _bucket or _sum that gets added to the metrics when using histograms or summaries.

What did you expect to see?

If the only possible type_suffix, currently is a _total. The config option could be renamed to without_counter_suffix OR the spec mentions that without_type_suffix only affects Counters.

I'm not sure if this is a legitimate ask, but I feel making this more explicit could be useful.

@psx95 psx95 added the spec:metrics Related to the specification/metrics directory label Oct 17, 2024
@danielgblanco danielgblanco added the sig-issue A specific SIG should look into this before discussing at the spec label Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig-issue A specific SIG should look into this before discussing at the spec spec:metrics Related to the specification/metrics directory
Projects
None yet
Development

No branches or pull requests

2 participants