Skip to content

adds global configuration for remaining global otel flags#1785

Merged
ChrisJBurns merged 3 commits intomainfrom
adds-global-config-otel
Sep 8, 2025
Merged

adds global configuration for remaining global otel flags#1785
ChrisJBurns merged 3 commits intomainfrom
adds-global-config-otel

Conversation

@ChrisJBurns
Copy link
Copy Markdown
Collaborator

@ChrisJBurns ChrisJBurns commented Sep 8, 2025

Adds the ability to set/get/unset remaining OTel global configuration flags.

Closes: #1142

Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.73%. Comparing base (c704a07) to head (a2b1d83).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1785   +/-   ##
=======================================
  Coverage   40.73%   40.73%           
=======================================
  Files         184      184           
  Lines       21425    21425           
=======================================
  Hits         8727     8727           
  Misses      12031    12031           
  Partials      667      667           

☔ View full report in Codecov by Sentry.
📢 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.

@ChrisJBurns ChrisJBurns enabled auto-merge (squash) September 8, 2025 21:51
Signed-off-by: ChrisJBurns <29541485+ChrisJBurns@users.noreply.github.com>
@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 38.346% (-0.2%) from 38.509%
when pulling a2b1d83 on adds-global-config-otel
into c704a07 on main.

Copy link
Copy Markdown
Contributor

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

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

I didn't review the generated files.

@ChrisJBurns ChrisJBurns merged commit 40aa283 into main Sep 8, 2025
18 checks passed
@ChrisJBurns ChrisJBurns deleted the adds-global-config-otel branch September 8, 2025 22:02
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.

Add more OpenTelemetry params to config file

3 participants