-
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
Automatically enrich Kubernetes module events #7470
Merged
Merged
Changes from 20 commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
0a08eab
Allow metadata generation for any k8s resource
392bfbb
Add kubernetes module metadata enricher
776cf83
Enrich `node` events with Kubernetes metadata
b280707
Enrich `state_node` events with Kubernetes metadata:
138440d
Enrich `state_pod` events with Kubernetes metadata
8a427f1
Enrich `pod` events with Kubernetes metadata
a486f1b
Add flag to disable metadata enriching
39f423f
Enrich `state_deployment` with Kubernetes metadata
0314211
Enrich `state_replicaset` with Kubernetes metadata
25d0bbb
Enrich `container` metricsets with Kubernetes metadata
7de196b
make hound happy
90c7ee8
Disable metadata retrieval in tests
1a43ff2
Add tests
7c79197
Add docs
4870722
Update metricbeat role to get access to watch needed resources
280884b
Enrich `state_statefulset` with Kubernetes metadata
d28aae2
Implement Closer interface to stop watcher on shutdown
eb72c5a
Move nilEnricher to its own implementation
a2707a9
Update config.yml
7d3d613
Add a note to meta generator config settings
c2b0454
fix typo
18d509f
Move warnings to errors
b6d8f76
Move `uid` back to `pod.uid`
99b9f84
make update
68929d0
Merge branch 'master' into kubernetes-module-metadata
307734b
Add locking to start/stop
d582e8b
Merge branch 'master' into kubernetes-module-metadata
cb88d1e
Add missing license headers
5dcb137
Merge branch 'master' into kubernetes-module-metadata
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
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
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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
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.
As
uid
has be already around before, we should not change anything here so this is only a note from my side.I often use
id
instead ofuuid
oruid
for consistency. And as long as there is only oneid
I expect it to be unique.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.
UID
is a well-known field in Kubernetes, changing it toid
could confuse some users IMOThere 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.
ok