-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added a version tag to the collector's own telemetry metrics #3960
Conversation
@emaderer please give me some details about why this is needed? Also in OpenCensus there is a concept of constant labels that you can use :) |
Here is the API you need to use https://github.com/census-ecosystem/opencensus-go-exporter-prometheus/blob/master/prometheus.go#L50 |
Added description, thanks 👍 |
addCollectorVersionPtr = flags.Bool( | ||
"add-collector-version", | ||
true, | ||
"Flag to control the addition of 'service.version' to the collector metrics.") | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll wait until https://github.com/open-telemetry/opentelemetry-collector/milestone/21 is done.
Added a version tag to the collector's own telemetry metrics because today when looking at those metrics there is no way to know the collector's version, this can also help to know what collector versions are running out there.