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

[connector/spanmetrics] CHANGELOG is fixed to reflect the correct fea… #35852

Merged
merged 1 commit into from
Nov 10, 2024

Conversation

tbeemster
Copy link
Contributor

Description

The wrong feature gate was exposed in the CHANGELOG, this has been corrected to the correct feature gate 'connector.spanmetrics.legacyMetricNames'

Link to tracking issue

Fixes #35766

Documentation

CHANGELOG.md is updated to have the correct feature gate name.

@tbeemster tbeemster requested a review from a team as a code owner October 17, 2024 11:37
Copy link

linux-foundation-easycla bot commented Oct 17, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Member

@songy23 songy23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR looks good, please sign the CLA @tbeemster

@songy23 songy23 added documentation Improvements or additions to documentation Skip Changelog PRs that do not require a CHANGELOG.md entry labels Oct 23, 2024
@tbeemster
Copy link
Contributor Author

I've just signed the CLA and rebased

@songy23
Copy link
Member

songy23 commented Oct 25, 2024

Looks like your commit has 2 authors @timbeemster @tbeemster and both need to sign CLA

or you can create a new PR with commit only from @tbeemster

…ture gate

The wrong feature gate was exposed in the CHANGELOG, this has been corrected to the correct feature gate 'connector.spanmetrics.legacyMetricNames'
@tbeemster
Copy link
Contributor Author

Hi @songy23 , sorry for the late response.
I've accidentally committed with 2 authors. I've removed the one that did not have the CLA signed and rebased on main again.
Let me know if this is sufficient to be merged.

Best regards,

Tim

@MovieStoreGuy MovieStoreGuy merged commit b0e89cd into open-telemetry:main Nov 10, 2024
158 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 10, 2024
pull bot pushed a commit to abaguas/opentelemetry-collector-contrib that referenced this pull request 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
Labels
documentation Improvements or additions to documentation Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release notes of 0.109.0 has incorrect feature gate for spanmetrics
4 participants