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).

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

Successfully merging this pull request may close these issues.

4 participants