-
Notifications
You must be signed in to change notification settings - Fork 199
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
Use Azure SDK exporter #1653
Merged
Merged
Use Azure SDK exporter #1653
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
de936a1
Temporarily add to this repo
trask fc61795
Bump version to not conflict
trask a3eacc9
Add azure-monitor-opentelemetry-exporter dependency
trask 7daedb2
Remove tests
trask 8fac403
Remove create default telemetry configuration
trask dcef630
Replace Channel with Rest Client
trask 800fb8c
Remove internal.channel package
trask f61986a
More
trask 07c7f8b
Update lockfiles
trask 1f2fdc5
Core compiling
trask cfaffdb
Core tests passing
trask 6944be2
Remove Exporter
trask ae6b4e7
Unit tests passing (still some ignored tho)
trask 125873e
Remove more
trask 6a671b7
Fix some FIXMEs
trask 08150df
Fix more FIXMEs
trask bc77fb9
Fix more FIXMEs
trask 69175d3
Bump version to 3.1.0
trask e53cdac
Some fixes
trask ce51afe
Not using external exporter for now
trask 82e337b
Need to use BatchSpanProcessor now
trask a537f46
Need to upstream: fix double slash in track url
trask 529bddb
Need to upstream: remove statusDescription property
trask 0753eea
DO NOT MERGE, TOO VERBOSE
trask 4c11910
Need to upstream: fix formatted duration
trask 45c1cc0
Optimization
trask df13221
More
trask 5d49601
Update gradle dependencies from compile to impl
trask e47fec6
Bring back old exporter
trask 40f1ed1
Update old exporter
trask b98b02d
More
trask c6d31c5
Temporary? Remove NDJSON from fake ingestion
trask 4ad2bdf
Use external exporter artifact after all
trask b2469c5
Fix wildfly
trask 7d4aaa6
Fix dependency version and lock files
trask 472f7d8
Fix metric telemetry and more
trask daffd16
Fix LGTM issue
trask 56d9d15
Rename TelemetryConfiguration to TelemetryClient
trask 62be412
Set time
trask fbc882c
Remove system.out.println
trask 64ef4e5
Fix metrics
trask b475843
Fix sporadic failing test
trask 63483c6
Fix tests
trask 9109eb5
Sporadic failing tests
trask 8b5e3cf
Fix test
trask 90fc600
Ignore failing test
trask f1e5018
Ignore failing test
trask 0f2f217
Merge remote-tracking branch 'origin/master' into trask/azure-sdk-exp…
trask b7fc56c
Bump submodule to fix build issue (#1658)
trask 0126540
Merge remote-tracking branch 'origin/master' into trask/azure-sdk-exp…
trask 4e98424
Fix ndjson
trask 35d68e2
Revert "Temporary? Remove NDJSON from fake ingestion"
trask 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 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 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 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 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 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 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 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.
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.
should we track config.customDimensions as a feature in statsbeat?
I wasn't aware of this config before. if nobody uses it, we can consider removing it?
does telemetry processor enable users to add custom dimensions?