Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[metricbeat] Enable events access to cluster role #254

Merged
merged 2 commits into from
Sep 19, 2019
Merged

[metricbeat] Enable events access to cluster role #254

merged 2 commits into from
Sep 19, 2019

Conversation

Azuka
Copy link
Contributor

@Azuka Azuka commented Aug 10, 2019

Metricbeat apparently needs access to events. I had to manually make this change to the clusterrole when installing the chart due to the following error:

Performing a resource sync err kubernetes api: Failure 403 events is forbidden: User "system:serviceaccount:monitoring:metricbeat-metricbeat" cannot list resource "events" in API group "" at the cluster scope for *v1.EventList

I haven't tried filebeat yet so I'm not sure this change is also required for it.

  • Chart version not bumped (the versions are all bumped and released at the same time)
  • README.md updated with any new values or changes
  • Updated template tests in ${CHART}/tests/*.py
  • Updated integration tests in ${CHART}/examples/*/test/goss.yaml

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@Azuka
Copy link
Contributor Author

Azuka commented Aug 29, 2019

@Crazybus, @Conky5 do you mind taking a look at this?

@chendo
Copy link

chendo commented Sep 18, 2019

On chart version v7.3.0, I used applied the clusterrole defined at https://github.com/elastic/beats/blob/master/deploy/kubernetes/metricbeat-kubernetes.yaml#L312-L340 and it solved my issues, which primarily manifested as no CPU or memory data being visible in the Infrastructure tab on Kibana.

However, I'm still unable to Group By: Namespace

Copy link
Contributor

@Crazybus Crazybus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Crazybus
Copy link
Contributor

jenkins test this please

@Crazybus
Copy link
Contributor

jenkins test this please

@Crazybus Crazybus merged commit 0f19184 into elastic:master Sep 19, 2019
@Crazybus
Copy link
Contributor

Thanks for fixing this one up! And sorry for the slow review (I was on vacation).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants