Skip to content
This repository was archived by the owner on Nov 5, 2021. It is now read-only.

External Probe: Allow dynamic labels for metrics that are not aggregated in cloudprober.#486

Merged
manugarg merged 1 commit intomasterfrom
63729A23134379909BFCA3AC3C378B48
Oct 8, 2020
Merged

External Probe: Allow dynamic labels for metrics that are not aggregated in cloudprober.#486
manugarg merged 1 commit intomasterfrom
63729A23134379909BFCA3AC3C378B48

Conversation

@manugarg
Copy link
Contributor

@manugarg manugarg commented Oct 8, 2020

If we are not aggregating metrics in cloudprober, we can export one EventMetrics per payload metric line, and with that we can support dynamic labels in the metric lines, for example, something like the following will work:

db_stats{name=db01,region=us-west-2,status=running,ptype=external,probe=external_test} 1.0
db_stats{name=db02,region=us-east-1,status=running,ptype=external,probe=external_test} 1.0

Implements: #466
PiperOrigin-RevId: 336164768

…ted in cloudprober.

If we are not aggregating metrics in cloudprober, we can export one EventMetrics per payload metric line, and with that we can support dynamic labels in the metric lines, for example, something like the following will work:

db_stats{name=db01,region=us-west-2,status=running,ptype=external,probe=external_test} 1.0
db_stats{name=db02,region=us-east-1,status=running,ptype=external,probe=external_test} 1.0

Implements: #466
PiperOrigin-RevId: 336164768
@manugarg manugarg merged commit f63a118 into master Oct 8, 2020
@manugarg manugarg deleted the 63729A23134379909BFCA3AC3C378B48 branch October 8, 2020 22:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant