opentelemetry-collector
opentelemetry-collector copied to clipboard
Stabilize module confighttp
Before stabilizing a module, an approver or maintainer must make sure that the following criteria have been met for at least two successive minor version releases:
- [x] No open issues or PRs in the module that would require breaking changes
- [x] No TODOs in the module code that would require breaking changes
- [x] No deprecated symbols in the module
- [x] No symbols marked as experimental in the module
- [x] The module follows the Coding guidelines
Please also make sure to publicly announce our intent to stabilize the module on:
- [x] The #otel-collector CNCF Slack Channel
- [x] The #opentelemetry CNCF Slack channel
- [x] A Collector SIG meeting (if unable to attend, just add to the agenda)
To help other people verify the above criteria, please link to the announcement and other links used to complete the above in a comment on this issue.
Once all criteria are met, close this issue by moving this module to the stable module set.
cd config/confighttp
git grep Deprecated
returns no matches.
Announced at SIG meeting 3/13.
Published https://cloud-native.slack.com/archives/CJFCJHG4Q/p1717127923554559 and https://cloud-native.slack.com/archives/C01N6P7KR6W/p1717127914499889
I'd like to get this one in before marking this stable: https://github.com/open-telemetry/opentelemetry-collector/pull/10445
New config option to take into consideration (not a breaking change) https://github.com/open-telemetry/opentelemetry-collector/issues/10870
New config option to take into consideration (not a breaking change) #10870
this work has stalled and was deemed to be best addressed after v1 https://github.com/open-telemetry/opentelemetry-collector/pull/10877#pullrequestreview-2488230593
Discussion about headers field https://github.com/open-telemetry/opentelemetry-collector/issues/13930
Should we add #8752 to this list? See also https://github.com/open-telemetry/opentelemetry-collector/pull/14058#issuecomment-3430135619 -- if we update confighttp to use confignet.AddrConfig then I think that takes care of #8752, and gives us a single place to add general network config like enabling SO_REUSEPORT.