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

[processor/groupbytrace] Fix groupbytrace metrics contain duplicate prefix #32698

Merged

Conversation

Frapschen
Copy link
Contributor

The actual metrics generated by groupbytrace have duplicate prefix processor_groupbytrace

error groupbytrace metrics:

# HELP otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases Releases that are suspected to have been incomplete
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases counter
otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue Number of events currently in the queue
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory Number of traces currently in the in-memory storage
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_spans_released Spans released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_spans_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_spans_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 20
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted Traces evicted from the internal buffer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_released Traces released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 10

@Frapschen Frapschen requested review from jpkrohling and a team as code owners April 26, 2024 05:00
@github-actions github-actions bot added the processor/groupbytrace Group By Trace processor label Apr 26, 2024
@Frapschen Frapschen force-pushed the fix_groupbytrace_metrics_bug branch from b0db183 to 010b228 Compare April 26, 2024 05:04
@jpkrohling jpkrohling merged commit 14d7d4d into open-telemetry:main May 7, 2024
170 checks passed
@github-actions github-actions bot added this to the next release milestone May 7, 2024
jpkrohling added a commit that referenced this pull request May 7, 2024
This is caused by
#32698
, which apparently didn't fail with this.

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
…refix (open-telemetry#32698)

The actual metrics generated by groupbytrace have duplicate prefix
`processor_groupbytrace`

error groupbytrace  metrics:
```
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases Releases that are suspected to have been incomplete
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases counter
otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue Number of events currently in the queue
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory Number of traces currently in the in-memory storage
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_spans_released Spans released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_spans_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_spans_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 20
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted Traces evicted from the internal buffer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_released Traces released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 10
```
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
This is caused by
open-telemetry#32698
, which apparently didn't fail with this.

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
@Frapschen Frapschen deleted the fix_groupbytrace_metrics_bug branch May 10, 2024 06:38
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
…refix (open-telemetry#32698)

The actual metrics generated by groupbytrace have duplicate prefix
`processor_groupbytrace`

error groupbytrace  metrics:
```
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases Releases that are suspected to have been incomplete
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases counter
otelcol_processor_groupbytrace_processor_groupbytrace_incomplete_releases{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue Number of events currently in the queue
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_events_in_queue{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory Number of traces currently in the in-memory storage
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory gauge
otelcol_processor_groupbytrace_processor_groupbytrace_num_traces_in_memory{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_spans_released Spans released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_spans_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_spans_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 20
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted Traces evicted from the internal buffer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_evicted{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 0
# HELP otelcol_processor_groupbytrace_processor_groupbytrace_traces_released Traces released to the next consumer
# TYPE otelcol_processor_groupbytrace_processor_groupbytrace_traces_released counter
otelcol_processor_groupbytrace_processor_groupbytrace_traces_released{service_instance_id="ac4299e2-8c43-47df-bdd6-90028fb39cd1",service_name="otelcontribcol",service_version="0.99.0-dev"} 10
```
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
This is caused by
open-telemetry#32698
, which apparently didn't fail with this.

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
processor/groupbytrace Group By Trace processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants