opentelemetry-collector icon indicating copy to clipboard operation
opentelemetry-collector copied to clipboard

Stabilize module confighttp

Open codeboten opened this issue 1 year ago • 8 comments

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.

codeboten avatar Jan 24 '24 17:01 codeboten

cd config/confighttp
git grep Deprecated

returns no matches.

atoulme avatar Jan 24 '24 17:01 atoulme

Announced at SIG meeting 3/13.

atoulme avatar Mar 13 '24 16:03 atoulme

Published https://cloud-native.slack.com/archives/CJFCJHG4Q/p1717127923554559 and https://cloud-native.slack.com/archives/C01N6P7KR6W/p1717127914499889

atoulme avatar May 31 '24 03:05 atoulme

I'd like to get this one in before marking this stable: https://github.com/open-telemetry/opentelemetry-collector/pull/10445

jpkrohling avatar Jul 19 '24 08:07 jpkrohling

New config option to take into consideration (not a breaking change) https://github.com/open-telemetry/opentelemetry-collector/issues/10870

rogercoll avatar Sep 02 '24 13:09 rogercoll

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

atoulme avatar Jul 26 '25 04:07 atoulme

Discussion about headers field https://github.com/open-telemetry/opentelemetry-collector/issues/13930

codeboten avatar Oct 06 '25 15:10 codeboten

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.

axw avatar Oct 24 '25 03:10 axw