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

[builder]: Fix version for providers in the default config #11123

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

bogdandrutu
Copy link
Member

No description provided.

@bogdandrutu bogdandrutu requested review from a team and codeboten September 10, 2024 21:30
@bogdandrutu bogdandrutu added the bug Something isn't working label Sep 10, 2024
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.23%. Comparing base (65bebec) to head (13e67c0).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #11123   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files         414      414           
  Lines       19810    19810           
=======================================
  Hits        18271    18271           
  Misses       1167     1167           
  Partials      372      372           

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

@bogdandrutu bogdandrutu merged commit 6fb8246 into open-telemetry:main Sep 10, 2024
49 checks passed
@bogdandrutu bogdandrutu deleted the fixdefaultcfg branch September 10, 2024 22:08
@github-actions github-actions bot added this to the next release milestone Sep 10, 2024
@cwegener
Copy link

cwegener commented Sep 10, 2024

I assume there will be a 0.109.1 for this fix shortly? Or is there a workaround for this issue?

Nevermind. I think I misread this change.

@cwegener
Copy link

I assume there will be a 0.109.1 for this fix shortly? Or is there a workaround for this issue?

Nevermind. I think I misread this change.

Ah. This change actually does fix the builder errors. I was running into caching problems during my testing. Applying this change explicitly to my builder config file does indeed fix the errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants