-
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
[processor/probabilisticsampling] use mdatagen for metrics #33874
[processor/probabilisticsampling] use mdatagen for metrics #33874
Conversation
|
||
telemetry: | ||
metrics: | ||
count_traces_sampled: |
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 think this needs the otelcol_processor prefix. Can you confirm with the metrics we have today?
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.
Metrics before the change:
# HELP otelcol_processor_probabilistic_sampler_count_traces_sampled Count of traces that were sampled or not
# TYPE otelcol_processor_probabilistic_sampler_count_traces_sampled counter
otelcol_processor_probabilistic_sampler_count_traces_sampled{policy="missing_randomness",sampled="false",service_instance_id="f1a27d2b-5019-44b7-8dbb-b35b6f852c8c",service_name="otelcontribcol",service_version="0.104.0-dev"} 2
otelcol_processor_probabilistic_sampler_count_traces_sampled{policy="trace_id_hash",sampled="true",service_instance_id="f1a27d2b-5019-44b7-8dbb-b35b6f852c8c",service_name="otelcontribcol",service_version="0.104.0-dev"} 2
After the change:
# HELP otelcol_processor_probabilistic_sampler_count_logs_sampled Count of logs that were sampled or not
# TYPE otelcol_processor_probabilistic_sampler_count_logs_sampled counter
otelcol_processor_probabilistic_sampler_count_logs_sampled{policy="missing_randomness",sampled="false",service_instance_id="4d31ffcc-1341-4ae0-84be-5d77592319f9",service_name="otelcontribcol",service_version="0.104.0-dev"} 1
# HELP otelcol_processor_probabilistic_sampler_count_traces_sampled Count of traces that were sampled or not
# TYPE otelcol_processor_probabilistic_sampler_count_traces_sampled counter
otelcol_processor_probabilistic_sampler_count_traces_sampled{policy="trace_id_hash",sampled="true",service_instance_id="4d31ffcc-1341-4ae0-84be-5d77592319f9",service_name="otelcontribcol",service_version="0.104.0-dev"} 2
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.
Note the change in the name of the logs vs traces sampled counter is a bugfix
Fixes open-telemetry#33465 Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
b73a52f
to
58894ef
Compare
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Fixes #33465