-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[chore] Fix an incorrect automatic replace made by a bot in otel-config.yml
#9746
Conversation
This PR fixes an incorrect automatic replace made in this PR https://github.com/open-telemetry/opentelemetry-collector/pull/9680/files#diff-c7c8156618a7f8126b25ca1bdfde3e172a0d2cb75c533d63a71617ae2a5c54ae apparently by a bot. I've taken the previous value which seems right.
Lol. Thanks for fixing this! Happened in 998f64d @open-telemetry/collector-maintainers. do you have any idea how it's possible? :) |
Looks like |
Submitted #9748 |
Haha, it happens! Thanks for the quick review |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9746 +/- ##
=======================================
Coverage 90.96% 90.96%
=======================================
Files 353 353
Lines 18630 18630
=======================================
Hits 16947 16947
Misses 1356 1356
Partials 327 327 ☔ View full report in Codecov by Sentry. |
…fig.yml` (open-telemetry#9746) This PR fixes an incorrect automatic replace made in the `otel-config.yml` file in this [pull request](https://github.com/open-telemetry/opentelemetry-collector/pull/9680/files#diff-c7c8156618a7f8126b25ca1bdfde3e172a0d2cb75c533d63a71617ae2a5c54ae) by a bot. I've taken the previous value which seems right. Co-authored-by: Alex Boten <[email protected]>
This PR fixes an incorrect automatic replace made in the
otel-config.yml
file in this pull request by a bot. I've taken the previous value which seems right.