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

Release notes of 0.109.0 has incorrect feature gate for spanmetrics #35766

Closed
tbeemster opened this issue Oct 14, 2024 · 3 comments · Fixed by #35852
Closed

Release notes of 0.109.0 has incorrect feature gate for spanmetrics #35766

tbeemster opened this issue Oct 14, 2024 · 3 comments · Fixed by #35852

Comments

@tbeemster
Copy link
Contributor

tbeemster commented Oct 14, 2024

Component(s)

connector/spanmetrics

Describe the issue you're reporting

The v0.109.0 release notes on page https://github.com/open-telemetry/opentelemetry-collector-contrib/releases talks about:

The feature flag connector.spanmetrics.legacyLatencyMetricNames

Steps to reproduce

  1. Dockerfile with Otelcollector built together with spanmetrics. Final line of Dockerfile:
    CMD ["/otelcol-custom", "--config=/etc/otel-collector-config.yml", "--feature-gates=connector.spanmetrics.legacyLatencyMetricNames" ]

  2. Build docker image

  3. Run docker image

Expected result
Running instance of the otelcollector

Actual result
Error: invalid argument "connector.spanmetrics.legacyLatencyMetricNames" for "--feature-gates" flag: no such feature gate "connector.spanmetrics.legacyLatencyMetricNames". valid gates: [component.UseLocalHostAsDefaultHost connector.spanmetrics.legacyMetricNames processor.tailsamplingprocessor.metricstatcountspanssampled service.noopTracerProvider telemetry.disableHighCardinalityMetrics telemetry.useOtelWithSDKConfigurationForInternalTelemetry]

Proposed change
Since the software is already released with the feature gate connector.spanmetrics.legacyMetricNames, I propose to change the Release notes of v0.109.0 to mention the correct feature gate.

@tbeemster tbeemster added the needs triage New item requiring triage label Oct 14, 2024
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@atoulme
Copy link
Contributor

atoulme commented Oct 14, 2024

I have gone ahead and edited the release notes, thank you for your report. Please consider opening a PR to update CHANGELOG.md.

@tbeemster
Copy link
Contributor Author

Hi @atoulme, thank you for changing the release notes.
I've just opened a PR to update the CHANGELOG.md file

MovieStoreGuy pushed a commit that referenced this issue Nov 10, 2024
#35852)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
The wrong feature gate was exposed in the CHANGELOG, this has been
corrected to the correct feature gate
'connector.spanmetrics.legacyMetricNames'

<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes #35766 

<!--Describe the documentation added.-->
#### Documentation
CHANGELOG.md is updated to have the correct feature gate name.
<!--Please delete paragraphs that you did not use before submitting.-->
pull bot pushed a commit to abaguas/opentelemetry-collector-contrib that referenced this issue Nov 11, 2024
open-telemetry#35852)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
The wrong feature gate was exposed in the CHANGELOG, this has been
corrected to the correct feature gate
'connector.spanmetrics.legacyMetricNames'

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#35766 

<!--Describe the documentation added.-->
#### Documentation
CHANGELOG.md is updated to have the correct feature gate name.
<!--Please delete paragraphs that you did not use before submitting.-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants