-
Notifications
You must be signed in to change notification settings - Fork 8
Opentelemetry Integration #80
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
Merged
egekorkan
merged 39 commits into
eclipse-thingweb:main
from
sailalithkanumuri8:sai-opentelemetry-integration
Sep 1, 2025
Merged
Changes from 37 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
da7b4a7
Initial tracing code
sailalithkanumuri8 f530d61
Fix tracing and Docker integration for advanced coffee machine with J…
sailalithkanumuri8 6093f1e
fix: ran npm run format
sailalithkanumuri8 8034db9
Updated README
sailalithkanumuri8 4053ee7
Modified tracing to include input values
sailalithkanumuri8 340ac8c
Refactored tracing.ts code
sailalithkanumuri8 b211687
Updated tracing code
sailalithkanumuri8 6b9d238
Fixed merge conflict errors
sailalithkanumuri8 24c09d8
fix: ran npm format
sailalithkanumuri8 6e57e3d
Added spans for properties and events
sailalithkanumuri8 72c077f
fix: ran npm run format
sailalithkanumuri8 4e7b3e6
Merge remote-tracking branch 'upstream/main' into sai-opentelemetry-i…
sailalithkanumuri8 0db157e
fix: ran npm run format again
sailalithkanumuri8 97280dc
Enhanced OpenTelemetry tracing
sailalithkanumuri8 40bd75d
refactor: remove low-level complexity and simplify OpenTelemetry trac…
sailalithkanumuri8 2e4284e
Added test file
sailalithkanumuri8 d4ecd0e
fix: ran npm run format
sailalithkanumuri8 0ec0d83
Fix Jaeger tracing configuration spans still not showing up
sailalithkanumuri8 a5e7996
Added more spans
sailalithkanumuri8 d2f4717
Merge remote-tracking branch 'origin/sai-opentelemetry-integration' i…
sailalithkanumuri8 740e6cd
fix: ran npm run format
sailalithkanumuri8 d404f88
update lock file
egekorkan 40e84ae
Merge branch 'main' of github.com:eclipse-thingweb/test-things into s…
egekorkan f879825
Decreased redundancy to help developers code things faster
sailalithkanumuri8 680f320
fix: fixed jeager endpoint in docker-compose-things
sailalithkanumuri8 3cb0171
Removed unnecessary method
sailalithkanumuri8 f12f179
fix: hopefully fixed lint errors
sailalithkanumuri8 99f5db5
Fixed dependancy mismatch
sailalithkanumuri8 e69c3e7
Fixed hardcoded jeager endpoint
sailalithkanumuri8 71d984f
Updated paths to make tracing in util
sailalithkanumuri8 780c899
Fixed counter thing
sailalithkanumuri8 7053292
Updated loc file
sailalithkanumuri8 ff10f89
Reduced redundancy in main.ts
sailalithkanumuri8 d43998d
Updated README
sailalithkanumuri8 4d94a97
Ran npm run format
sailalithkanumuri8 9a7e000
Updated schema fetching in util.ts
sailalithkanumuri8 a43fb54
Made span name handling easier for developer
sailalithkanumuri8 13161c8
Updated span names and added more info on readme
sailalithkanumuri8 68cb5d4
Updated README
sailalithkanumuri8 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"operation" shouldn't be needed anymore