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

[receiver/apache]: change feature gates for server name and port resource attributes to beta #16483

Merged
merged 1 commit into from
Nov 29, 2022

Conversation

aboguszewski-sumo
Copy link
Member

Description:
The feature gates in apachereceiver were supposed to be changed to Beta in v0.66.0, but because this version has been released earlier than expected, the change was not included in this version.

Does this need a changelog entry? Is this a breaking change in that case?

Link to tracking Issue: #14791

Testing:
Unit and integration tests. Integration tests have this feature disabled.

Documentation:
Readme has been updated.

@aboguszewski-sumo aboguszewski-sumo requested a review from a team November 25, 2022 14:04
@aboguszewski-sumo aboguszewski-sumo changed the title chore: change apachereceiver's feature gates to beta [chore]: change apachereceiver's feature gates to beta Nov 25, 2022
Copy link
Member

@TylerHelmuth TylerHelmuth left a comment

Choose a reason for hiding this comment

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

LGTM

@TylerHelmuth
Copy link
Member

@aboguszewski-sumo I think a changelog entry would be useful for this situations to tell users of the component that the featuregate switched.

@aboguszewski-sumo aboguszewski-sumo force-pushed the apache-beta branch 2 times, most recently from 3b2427c to f412376 Compare November 29, 2022 07:37
@aboguszewski-sumo aboguszewski-sumo changed the title [chore]: change apachereceiver's feature gates to beta [receiver/apache]: change feature gates for server name and port resource attributes to beta Nov 29, 2022
@aboguszewski-sumo
Copy link
Member Author

@MovieStoreGuy I can't see your comment, but I saw it via email - yes, the issue number had a typo (14971 instead of 14791), thanks for catching that.

@djaglowski djaglowski merged commit b56c4f1 into open-telemetry:main Nov 29, 2022
shalper2 pushed a commit to shalper2/opentelemetry-collector-contrib that referenced this pull request Dec 6, 2022
…urce attributes to beta (open-telemetry#16483)

chore: change apachereceiver's feature gates to beta
@plantfansam plantfansam mentioned this pull request Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants