Skip to content
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

Update the kuma-control-plane ClusterRole to enable create/patch of kubernetes events. #884

Merged

Conversation

ateirney
Copy link
Contributor

@ateirney ateirney commented Jul 4, 2020

Summary

Update the kuma-control-plane ClusterRole to enable create/patch of kubernetes events.

Full changelog

  • Fix the kuma-control-plane ClusterRole to enable create/patch of kubernetes events.

    The kuma-control-plane registers kubernetes events that without the appropriate permissions results in the event not being recorded, and an error log being generated.

Issues resolved

Fix #815

Documentation

@ateirney ateirney requested a review from a team July 4, 2020 13:54
@CLAassistant
Copy link

CLAassistant commented Jul 4, 2020

CLA assistant check
All committers have signed the CLA.

@ateirney ateirney force-pushed the fix/update-control-plane-cluster-role branch from 12b7666 to c0987ca Compare July 4, 2020 22:20
The kuma-control-plane registers events that without the appropriate
permissions when running on kubernetes results in the event not being
recorded and an error log being generated.

Fixes kumahq#815
@ateirney ateirney force-pushed the fix/update-control-plane-cluster-role branch from c0987ca to 5361174 Compare July 4, 2020 22:36
@nickolaev
Copy link
Contributor

Looks quite reasonable to me. @lobkovilya I'll wait for your approval before we merge it.

Copy link
Contributor

@lobkovilya lobkovilya left a comment

Choose a reason for hiding this comment

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

LGTM

@nickolaev nickolaev merged commit 66f8935 into kumahq:master Jul 6, 2020
@jakubdyszkiewicz
Copy link
Contributor

Thank you for fixing it! @andrew-teirney

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

Successfully merging this pull request may close these issues.

Kuma CP logs cannot patch resource "events" in API group ""
5 participants