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

docs(KFLUXVNGD-38): update project permissions #209

Conversation

yftacherzog
Copy link
Member

Maintainers should not have delete permissions on Project* resources as they should not be allowed to delete applications or components.

Maintainers should not have delete permissions on Project* resources
as they should not be allowed to delete applications or components.

Signed-off-by: Yftach Herzog <[email protected]>
Copy link
Member

@ifireball ifireball left a comment

Choose a reason for hiding this comment

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

lgtm, but lets make sure we hold this until the technical change lands

@yftacherzog
Copy link
Member Author

@hugares this PR is documenting the change you merged in redhat-appstudio/infra-deployments#4660. Can you take a look?

Copy link
Contributor

@hugares hugares left a comment

Choose a reason for hiding this comment

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

/lgtm

@hugares hugares merged commit 9eb9c42 into konflux-ci:main Oct 9, 2024
1 check passed
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.

3 participants