Skip to content

[component] Remove TelemetrySettings.MetricsLevel#12189

Merged
bogdandrutu merged 7 commits into
open-telemetry:mainfrom
mx-psi:mx-psi/remove-metricslevel
Feb 4, 2025
Merged

[component] Remove TelemetrySettings.MetricsLevel#12189
bogdandrutu merged 7 commits into
open-telemetry:mainfrom
mx-psi:mx-psi/remove-metricslevel

Conversation

@mx-psi
Copy link
Copy Markdown
Member

@mx-psi mx-psi commented Jan 27, 2025

Description

Fixes #11061.

Depends on releasing the deprecation.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.27%. Comparing base (ef5f34f) to head (c2fed7e).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12189      +/-   ##
==========================================
- Coverage   91.27%   91.27%   -0.01%     
==========================================
  Files         466      466              
  Lines       25586    25583       -3     
==========================================
- Hits        23354    23351       -3     
  Misses       1816     1816              
  Partials      416      416              

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

@mx-psi

This comment was marked as resolved.

@mx-psi mx-psi marked this pull request as ready for review February 4, 2025 10:23
@mx-psi mx-psi requested review from a team, dmathieu and evan-bradley as code owners February 4, 2025 10:23
@mx-psi mx-psi added the release:blocker The issue must be resolved before cutting the next release label Feb 4, 2025
@mx-psi mx-psi enabled auto-merge February 4, 2025 10:24
@mx-psi mx-psi added this pull request to the merge queue Feb 4, 2025
@bogdandrutu bogdandrutu removed this pull request from the merge queue due to a manual request Feb 4, 2025
@bogdandrutu bogdandrutu added this pull request to the merge queue Feb 4, 2025
Merged via the queue into open-telemetry:main with commit 97cae29 Feb 4, 2025
florianl added a commit to open-telemetry/opentelemetry-ebpf-profiler that referenced this pull request Jun 3, 2025
…o v0.127.0

MetricsLevel got removed with open-telemetry/opentelemetry-collector#12189. Therefore update the test dependency.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:blocker The issue must be resolved before cutting the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove MetricsLevel from TelemetrySettings

2 participants