-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Conversation
[metricbeat] fix test_cluster_role_rules
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.
LGTM, I'd like @ChrsMark to also review it, as his team is the stakeholder of the Helm chart reference
- apiGroups: ["apps"] | ||
resources: | ||
- statefulsets | ||
- deployments |
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.
We also need replicasets
here. There was an issue with this: elastic/beats#15741
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.
fixed in 470fecd
This mount don't seem to be used by metricbeat as we don't use `add_docker_metadata` processor.
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.
lgtm!
[metricbeat] fix goss tests
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.
LGTM 👍
in_cluster
config fromadd_kubernetes_metadata
clusterRoleRules
with recommended values/var/lib/docker/container
mountFixes #559
README.md updated with any new values or changes${CHART}/tests/*.py
${CHART}/examples/*/test/goss.yaml