feat: add otlp logging support#454
Merged
stefanprodan merged 1 commit intostefanprodan:masterfrom Mar 17, 2026
Merged
Conversation
4fae3a4 to
2cf3cc5
Compare
Contributor
Author
|
hi @stefanprodan I'm not sure what the project's contribution rules are. If an Issue is needed where we discuss the functional requirements first, that's fine as well, else when you have the time, please have a look. |
Owner
|
@hansbogert did you tested the changes, are the metrics still working, do logs show up in Loki? |
Contributor
Author
|
Logs show up, traces still work (metrics over otlp was never configured/implemented): 
|
- Adds a loggerprovider based on otlp logger - In demo directory of oltp: - Added grafana for unified view of both traces and logs - tracing now uses oltp from the collector to the jaeger instance Signed-off-by: Hans van den Bogert <hansbogert@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Disclaimer, made with help of claude code.