Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

fix: remove unnecessary RBAC permissions #278

Merged
merged 1 commit into from
Jun 28, 2021

Conversation

vivienfricadelamadeus
Copy link
Contributor

This PR updates RBAC policies assigned to the applicationset controller so that it does not request unused privileges not available by default at namespace level

  • remove unnecessary RBAC permissions 'delete events' and 'update events'

…events' that are not available at namespace level
@CLAassistant
Copy link

CLAassistant commented Jun 28, 2021

CLA assistant check
All committers have signed the CLA.

@vivienfricadelamadeus
Copy link
Contributor Author

@jgwest I succeeded to sign CLA and updated the manifests as suggested

Copy link
Member

@jgwest jgwest left a comment

Choose a reason for hiding this comment

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

@jgwest jgwest merged commit 4310bd8 into argoproj:master Jun 28, 2021
@jgwest
Copy link
Member

jgwest commented Jul 12, 2021

HI @vivienfricadelamadeus - I'm working on the v0.2.0 release contributor list #291, please see that issue, and if you'd like to be credited by real name (rather than your GitHub ID), let me know what it is on that issue... thanks!

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.

None yet

3 participants