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

OpenTelemetry Readme #134

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

mikehaller
Copy link
Contributor

Some readme documentation for the OpenTelemetry setup in PR #119 - e.g. how to build, how to install the otel components and the Jaeger UI.

Copy link

codecov bot commented Feb 6, 2025

Codecov Report

Attention: Patch coverage is 20.83333% with 38 lines in your changes missing coverage. Please review.

Project coverage is 59.44%. Comparing base (ec91336) to head (50b965e).

Files with missing lines Patch % Lines
databroker/src/lib.rs 0.00% 23 Missing ⚠️
databroker/src/open_telemetry.rs 0.00% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #134      +/-   ##
==========================================
- Coverage   60.14%   59.44%   -0.71%     
==========================================
  Files          33       34       +1     
  Lines       16198    15112    -1086     
==========================================
- Hits         9743     8983     -760     
+ Misses       6455     6129     -326     

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

@mikehaller
Copy link
Contributor Author

The build fails due to markdown-check trying to validate a http://localhost links in the readme.

Fix in kuksa-actions, please have a look at eclipse-kuksa/kuksa-actions#18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants