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

Updating the mainfests with minimal rbac changes. #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Adarshkumar14
Copy link
Collaborator

Signed-off-by: Adarshkumar14 adarsh@chaosnative.com

  • Removed delete and deletecollection permission for the events.
  • Removed delete permission for sts,ds,rs and deployments(Only subscriber-cr have delete permission to delete deployment.)
  • Removed delete permission for the namespace,secrets,configmaps,services.(Only subscriber-cluster-role has permission to delete configmaps and only litmus-cluster-scope-for-litmusportal-server has permission to delete services, It is required to delete services of the predefined workflow.)
  • By merging this PR predefined workflows like bank-of-anthos and sock-shop will not work. Users have to manually install these apps.

Testing:

@Jonsy13 and I had tested it In both cluster-scope Installation and namespace -scope Installation, Control plane and Execution plane components are working as expected, predefined workflow(podato-head) and pod-delete experiment are also working* fine.
Note: Need to do the testing for other experiments also.

Dependency:

Before merging this PR, https://github.com/chaosnative/cle/pull/74 needs to be merged.

Signed-off-by: Adarshkumar14 <adarsh@chaosnative.com>
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.

2 participants