-
Notifications
You must be signed in to change notification settings - Fork 685
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
Enable beats stack monitoring configuration #5878
Merged
Merged
Changes from 20 commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
c6b7287
Add stack monitoring for Beats
naemono 165decd
When calling GetAssociations, also add the monitoring objects.
naemono c31c29d
Only access info after chedcking error
naemono de1cddc
Add debugging
naemono e14849d
Removing debugging logging statements
naemono 23de080
Handling external, and internal monitoring refs the same.
naemono ec13b90
Fix unit test.
naemono a21aa92
Fix some issues with errorf formatting
naemono 6101d4c
Merge branch 'main' into 5563-enable-beats-monitoring
naemono 8991412
Add missing '/v2' to association package.
naemono 02b0489
Optimize ssl/ca bits when creating monitoring configuration
naemono e517733
Add copyright to beat/common/monitoring.go
naemono 42829d9
remove duplicative checks in beat e2e test
naemono 94ebcf3
remove unused monitoringAssociationConnectionInfo
naemono 77a205e
re-running generation, since wording changed in crds
naemono 0ea5dfd
Avoid extra var in range in pkg/controller/beat/common/pod_test.go
naemono 9831c38
Use new func in existing checks package to return beats monitoring st…
naemono b155d9d
Enabling webhook validation for beats stack monitoring.
naemono 25702b7
Add unit tests for beat validation.
naemono 2779620
Minor change in pod_test.go.
naemono 2994d5f
Removing unneeded BeatsStep function.
naemono 2c19f81
Moving to utilizing sidecars for beat monitoring instead of internal …
naemono 4ae5d27
revert config/recipes changes
naemono 06a3396
Fix early return in beats validation so that invalid monitoring refs …
naemono a175029
Ensure that beats http.port setting finds it's way all the way to the…
naemono ccf1323
Add missing metricbeat template yaml file.
naemono b9a0c6a
Adding missing header to new file.
naemono f21ff9c
running make generate crds
naemono 7dc4bc9
Updating api docs
naemono d14893a
Update CRD documentation
naemono d901228
Correcting spelling
naemono 3ae14e4
Add UUID to stack monitoring *beat configuration.
naemono 268944b
Ensure metricbeat sidecar for beat has cluster_uuid.
naemono d5bb9b8
Add beats stackmon tests
naemono b57902e
Add newlines to yaml files
naemono c86c85e
Update beats stackmon validation to use same common validation
naemono 025a30f
Update public docs.
naemono 97125e5
Update api docs
naemono 9bf98fe
Fix naming of static error.
naemono 710575d
fixing ghost changes causing ci failures
naemono 6840e41
Add changed yaml files vi automation thatss breaking ci
naemono 734130d
adjust all-crds in opeartor chart
naemono 1da13e0
Merge branch 'main' into 5563-enable-beats-monitoring
naemono 0b5581f
Apply https://github.com/elastic/cloud-on-k8s/pull/5955 to try and ge…
naemono aea3d88
Add missing context to beat buildPodTemplate test.
naemono c710cd5
Revert change to deployer/runner/gke.go as it was merged.
naemono 3a85f16
Merge branch 'main' into 5563-enable-beats-monitoring
naemono 45fc97a
Move Monitoring objects to commonv1.
naemono 23ae156
Add Keyed field to struct to pass linter.
naemono c646180
Merge branch 'main' into 5563-enable-beats-monitoring
naemono 2975684
Fix moved struct from esv1 -> commonv1 in stackmon tests
naemono 2182d40
Update api docs
naemono ca26311
Update test for new ssl verification mode for beats
naemono 0e37cbb
wip
naemono fe83cee
Use unix sockets for beat metrics gather when stack monitoring is ena…
naemono d4ebba2
Merge branch 'main' into 5563-enable-beats-monitoring
naemono 44a3d1e
Regenerate api docs
naemono 8864c1c
Docs and CRD wording updates
naemono 42535c0
Attempt remove specific path in filebeat.
naemono 53e7dd1
Always uppercase stack monitoring in docs.
naemono a14eb20
running make docs, and go vet
naemono e3f965d
Do not log when ES cluster UUID isn't yet available, just emit event.
naemono ed04231
add shared volume for filebeat logs consumption
naemono 0f99ccb
correct spelling in beat/*/stackmon.go
naemono f3bf6af
Ensure Beat doesn't contain '-e' startup option when log stack monito…
naemono bebdf6e
Fix beat configuration test for stackmon being enabled.
naemono 52cd829
Ensure "-e" argument to *beat is removed if provided in podtemplate c…
naemono 0cd89e2
Add documentation for the '-e' filebeat option being automatically re…
naemono 0658e37
uppercase Filebeat
naemono 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
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.
It is worth calling out that Beats monitoring will only be visible in Kibana if the corresponding Elasticsearch cluster is also monitored. Otherwise the UI breaks in interesting ways. Also probably worth reporting as a Kibana bug maybe?
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.
I've updated these docs. Let me know if the wording is good.