-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[exporter/googlemanagedprometheus] Add target and scope info metrics by default #24372
[exporter/googlemanagedprometheus] Add target and scope info metrics by default #24372
Conversation
20a06c6
to
c9da95b
Compare
@@ -54,6 +56,11 @@ func (c *GMPConfig) toCollectorConfig() collector.Config { | |||
cfg.ProjectID = c.ProjectID | |||
cfg.UserAgent = c.UserAgent | |||
cfg.MetricConfig.ClientConfig = c.MetricConfig.ClientConfig | |||
// add target_info and scope_info 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.
Can we add config for enabling/disabling these, similar to the PRW exporer: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/prometheusremotewriteexporter/config.go#L41 ?
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.
yup, added and enabled by default
c9da95b
to
a557564
Compare
I think you need to run |
0563082
to
9d6a749
Compare
34d00f4
to
ecd5e03
Compare
Description:
This enabled
target_info
andotel_scope_info
metrics by default in the GMP exporter, to be compatible with the prometheus spec.Link to tracking Issue: GoogleCloudPlatform/opentelemetry-operations-go#543
Testing: Downstream unit + integration tests
Documentation: Downstream + specification