Skip to content

[pkg/configoptional] Remove stabilized gate configoptional.AddEnabledField - #15421

Merged
codeboten merged 3 commits into
open-telemetry:mainfrom
codeboten:codeboten/configoptional.AddEnabledField
Jun 9, 2026
Merged

codeboten merged 3 commits into
open-telemetry:mainfrom
codeboten:codeboten/configoptional.AddEnabledField

Conversation

@codeboten

Copy link
Copy Markdown
Member

No description provided.

…dField`.

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codeboten
codeboten requested a review from a team as a code owner June 9, 2026 15:59
@codeboten
codeboten requested a review from dmitryax June 9, 2026 15:59
Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Comment thread .chloggen/codeboten_configoptional.AddEnabledField.yaml Outdated
Co-authored-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codspeed

codspeed Bot commented Jun 9, 2026 •

Copy link
Copy Markdown

Merging this PR will not alter performance

⚠️ 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 improved benchmarks
❌ 2 regressed benchmarks
✅ 3 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ BenchmarkBatchMetricProcessor2k 62.6 ms 129.2 ms -51.54%
❌ BenchmarkMultiBatchMetricProcessor2k 71.5 ms 134.9 ms -47.03%
⚡ BenchmarkGRPCLogsSequential 6.3 ms 3 ms ×2.1
⚡ BenchmarkHTTPProtoLogsSequential 5.1 ms 2.5 ms ×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/configoptional.AddEnabledField (f34af64) with main (ca97993)

Open in CodSpeed

@codecov

codecov Bot commented Jun 9, 2026

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 (35124bb).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #15421   +/-   ##
=======================================
  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
codeboten added this pull request to the merge queue Jun 9, 2026
Merged via the queue into open-telemetry:main with commit 49709e3 Jun 9, 2026
67 of 69 checks passed
@codeboten
codeboten deleted the codeboten/configoptional.AddEnabledField branch June 9, 2026 17:08
raghu999 pushed a commit to raghu999/opentelemetry-collector that referenced this pull request Aug 6, 2026
…dField` (open-telemetry#15421)

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants