Skip to content

[pkg/confighttp] Remove stabilized gate confighttp.framedSnappy#15420

Merged
codeboten merged 4 commits into
open-telemetry:mainfrom
codeboten:codeboten/rm-confighttp-gate
Jun 9, 2026
Merged

[pkg/confighttp] Remove stabilized gate confighttp.framedSnappy#15420
codeboten merged 4 commits into
open-telemetry:mainfrom
codeboten:codeboten/rm-confighttp-gate

Conversation

@codeboten

@codeboten codeboten commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@codeboten
codeboten requested a review from a team as a code owner June 9, 2026 15:08
@codeboten
codeboten requested a review from evan-bradley June 9, 2026 15:08
codeboten
codeboten commented Jun 9, 2026
View reviewed changes
Comment thread .chloggen/codeboten_rm-confighttp-gate.yaml Outdated
@codeboten
codeboten enabled auto-merge June 9, 2026 15:09
@codspeed-hq

codspeed-hq Bot commented Jun 9, 2026 •
edited
Loading

Copy link
Copy Markdown

Merging this PR will degrade performance by 53.01%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 2 regressed benchmarks
✅ 5 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ BenchmarkBatchMetricProcessor2k 62.6 ms 141.1 ms -55.66%
❌ BenchmarkMultiBatchMetricProcessor2k 71.5 ms 143.5 ms -50.2%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

Comparing codeboten:codeboten/rm-confighttp-gate (9aa211d) with main (ca97993)

Open in CodSpeed

@codecov

codecov Bot commented Jun 9, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.27%. Comparing base (e4d1967) to head (9aa211d).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main   #15420   +/-   ##
=======================================
  Coverage   91.27%   91.27%           
=======================================
  Files         709      709           
  Lines       46224    46224           
=======================================
  Hits        42192    42192           
  Misses       2817     2817           
  Partials     1215     1215

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codeboten
tidy
9aa211d 
Signed-off-by: Alex Boten <[email protected]>
@codeboten
codeboten added this pull request to the merge queue Jun 9, 2026
Merged via the queue into open-telemetry:main with commit 4115c80 Jun 9, 2026
67 of 69 checks passed
@codeboten
codeboten deleted the codeboten/rm-confighttp-gate branch June 9, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bogdandrutu bogdandrutu bogdandrutu approved these changes
@songy23 songy23 songy23 approved these changes
@evan-bradley evan-bradley Awaiting requested review from evan-bradley evan-bradley is a code owner automatically assigned from open-telemetry/collector-approvers

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@codeboten @bogdandrutu @songy23