Skip to content

Update to opentelemetry-configuration v1.1.0#8451

Open
jack-berg wants to merge 5 commits into
open-telemetry:mainfrom
jack-berg:declarative-config-1.1
Open

Update to opentelemetry-configuration v1.1.0#8451
jack-berg wants to merge 5 commits into
open-telemetry:mainfrom
jack-berg:declarative-config-1.1

Conversation

@jack-berg
Copy link
Copy Markdown
Member

@jack-berg jack-berg commented Jun 4, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.07%. Comparing base (53fc695) to head (eaf7878).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #8451      +/-   ##
============================================
+ Coverage     91.00%   91.07%   +0.06%     
- Complexity     7817     7826       +9     
============================================
  Files           893      893              
  Lines         23721    23733      +12     
  Branches       2364     2368       +4     
============================================
+ Hits          21588    21615      +27     
+ Misses         1410     1399      -11     
+ Partials        723      719       -4     

☔ 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.

@jack-berg jack-berg changed the title Update to opentelemetry-configuration latest ref Update to opentelemetry-configuration v1.1.0 Jun 5, 2026
@jack-berg jack-berg marked this pull request as ready for review June 5, 2026 17:16
@jack-berg jack-berg requested a review from a team as a code owner June 5, 2026 17:16
/** Reflectively set the max export batch size for the {@link SdkMeterProviderBuilder}. */
public static SdkMeterProviderBuilder setMaxExportBatchSize(
SdkMeterProviderBuilder sdkMeterProviderBuilder, int maxExportBatchSize) {
public static PeriodicMetricReaderBuilder setMaxExportBatchSize(
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This was wrong. Had to fix to add support for the new declarative config property for configuring batch size.


@Override
public String getName() {
return "event_to_span_event_bridge/development";
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Now that event_to_span_event_bridge is formally supported in declarative config, we can directly reference instance of using SPI.

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.

1 participant