-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[elastic-agent] initial instrumentation #29031
Merged
stuartnelson3
merged 38 commits into
elastic:master
from
stuartnelson3:instrument-elastic-agent
Feb 1, 2022
Merged
Changes from 4 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
53cbb1b
add some initial instrumentation
stuartnelson3 9f960a2
update tests
stuartnelson3 97f5658
update NOTICE.txt
stuartnelson3 234ab4e
make update
stuartnelson3 8bf85e9
fix order capturing the error/ending the span
stuartnelson3 1303efc
close the default tracer
stuartnelson3 be720ae
use higher-level span name
stuartnelson3 e81239a
add agent instrumentation config
stuartnelson3 c3fc48e
pass config values to tracer
stuartnelson3 0e4f1f7
add explicit tls config
stuartnelson3 69c327b
add span for emitter
stuartnelson3 696d0ea
instrument dispatcher
stuartnelson3 8207973
instrument Update fn
stuartnelson3 55bd680
instrument acker
stuartnelson3 ba1b101
update tests
stuartnelson3 10d67d7
mage check
stuartnelson3 b9b5e5b
remove libbeat tls config
stuartnelson3 4dc7c89
tracing for enroll command
stuartnelson3 49a08da
client is already wrapped
stuartnelson3 3956d45
wrap base round tripper with apm
stuartnelson3 c6f4aa2
remove some comments
stuartnelson3 a9620bc
nest apm config under agent.monitoring.apm
stuartnelson3 eb315d9
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 b2560ae
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 4cf02d0
re-add mistakenly deleted log value
stuartnelson3 83c1979
remove TODO
stuartnelson3 e6ead42
Apply suggestions from code review
stuartnelson3 e2b75fc
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 4b7fd3d
remove old instrumentation config
stuartnelson3 c89a52d
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 6f05e1b
comments
stuartnelson3 5c2fddb
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 7657748
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 010bb60
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 8d9603a
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 d303e3d
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 92a87e2
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 c0b4b9e
Merge branch 'master' into instrument-elastic-agent
stuartnelson3 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
Large diffs are not rendered by default.
Oops, something went wrong.
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
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.
Interesting I didn't know about go.elastic.co, when the restructuring of the repository is complete we should take a look a it. @kvch @cmacknz @ruflin
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.
TBH, I am not a huge fan of vanity imports. It looks great, but it is just vanity... :)