Skip to content

Update Otel Deps & Setup Action#1557

Open
aphralG wants to merge 17 commits intomainfrom
update-otel-dep
Open

Update Otel Deps & Setup Action#1557
aphralG wants to merge 17 commits intomainfrom
update-otel-dep

Conversation

@aphralG
Copy link
Copy Markdown
Contributor

@aphralG aphralG commented Mar 12, 2026

Proposed changes

Update Dependencies & Actions
Includes Change to naming of Otel exporters which were changed from otlp to otlp_grpc & otlphttp to otlp_http
The above is a breaking change to anyone using the bring your own OTel Config option in agent.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • I have run make install-tools and have attached any dependency changes to this pull request
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • If applicable, I have updated any relevant documentation (README.md)
  • If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

@aphralG aphralG self-assigned this Mar 12, 2026
@aphralG aphralG requested a review from a team as a code owner March 12, 2026 15:43
@github-actions github-actions bot added chore Pull requests for routine tasks dependencies labels Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 13, 2026

Dependency Review

The following issues were found:

  • ❌ 1 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 44 package(s) with unknown licenses.
  • ⚠️ 2 packages with OpenSSF Scorecard issues.

View full job summary

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.05%. Comparing base (90bbc5e) to head (b7e9ad3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/collector/settings.go 33.33% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1557   +/-   ##
=======================================
  Coverage   85.05%   85.05%           
=======================================
  Files         105      105           
  Lines       13691    13694    +3     
=======================================
+ Hits        11645    11648    +3     
+ Misses       1531     1530    -1     
- Partials      515      516    +1     
Files with missing lines Coverage Δ
internal/collector/nginxplusreceiver/factory.go 84.00% <100.00%> (ø)
internal/collector/nginxreceiver/factory.go 83.33% <100.00%> (ø)
...al/scraper/accesslog/operator/input/file/config.go 72.72% <ø> (ø)
internal/config/config.go 88.37% <100.00%> (ø)
internal/collector/settings.go 75.34% <33.33%> (-1.81%) ⬇️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90bbc5e...b7e9ad3. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Pull requests for routine tasks dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants