Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temporarily remove expiration from OTEL exporter #1046

Closed
wants to merge 1 commit into from

Conversation

mariomac
Copy link
Contributor

@mariomac mariomac commented Jul 22, 2024

Willing to remove old metrics forced us to work with our own fork of the OTEL SDK by means of a replace directive in go.mod, which created transitive incompatibilities with other components.

We will temporarily remove the invocations to this feature until the Removal of metrics is provided in the main OTEL SDK.

Status tracking:
open-telemetry/opentelemetry-specification#4135

@mariomac mariomac marked this pull request as ready for review July 22, 2024 14:08
@mariomac mariomac requested review from grcevski and marctc as code owners July 22, 2024 14:08
@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.42%. Comparing base (99e4b40) to head (933a17c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1046      +/-   ##
==========================================
- Coverage   81.46%   81.42%   -0.05%     
==========================================
  Files         139      139              
  Lines       11108    11105       -3     
==========================================
- Hits         9049     9042       -7     
- Misses       1544     1550       +6     
+ Partials      515      513       -2     
Flag Coverage Δ
integration-test 56.52% <0.00%> (+0.01%) ⬆️
k8s-integration-test 59.07% <0.00%> (+0.05%) ⬆️
oats-test 37.41% <0.00%> (+0.01%) ⬆️
unittests 50.85% <100.00%> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@marctc marctc added the do-not-merge WIP or something that musn't be merged label Jul 22, 2024
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

LGTM!

@mariomac
Copy link
Contributor Author

Already done in another PR

@mariomac mariomac closed this Jul 30, 2024
@mariomac mariomac deleted the rm-expiration branch July 30, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge WIP or something that musn't be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants