Skip to content

[patch] move ClusterRole and ClusterBinding from MVI OLM to Ansible - #2407

Open
dev-patidar17 wants to merge 4 commits into
masterfrom
3430
Open

[patch] move ClusterRole and ClusterBinding from MVI OLM to Ansible#2407
dev-patidar17 wants to merge 4 commits into
masterfrom
3430

Conversation

@dev-patidar17

@dev-patidar17 dev-patidar17 commented Aug 12, 2026

Copy link
Copy Markdown

add Clusterrole for MVI

Description

Move clusterrole and cluster_binding from mvi to ansible.

https://jsw.ibm.com/browse/MASMVI-3430

Test Results

Ranplaybooks/mas_add_visualinspection.yml for visualinspection_clusterrole

Screenshot 2026-08-13 at 2 43 15 PM

Clusterrole and Clusterbinding are successfully created on fips cluster after successfully ran playbook.
Screenshot 2026-08-13 at 2 56 25 PM
Screenshot 2026-08-13 at 2 56 32 PM

On dev910a instance

Screenshot 2026-08-19 at 7 19 34 PM Screenshot 2026-08-19 at 7 20 44 PM

removed clusterPermissions from csv to avoid OLM owns the ClusterRole and ClusterRoleBinding


⚠️ Notes for Reviewers

  • Ensure you have understood the PR guidelines in the Playbook before proceeding with a review.
  • Ensure all sections in the PR template are appropriately completed.

Dev Patidar added 2 commits August 11, 2026 12:32
add Clusterrole for MVI
@dev-patidar17
dev-patidar17 marked this pull request as ready for review August 13, 2026 14:40
@dev-patidar17
dev-patidar17 requested a review from a team as a code owner August 13, 2026 14:40
joseph122333
joseph122333 previously approved these changes Aug 19, 2026

@joseph122333 joseph122333 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me

This reverts commit 6e06e4f.
@dev-patidar17 dev-patidar17 changed the title [patch] add Clusterrole for MVI [patch] move ClusterRole and ClusterBinding from MVI OLM to Ansible Aug 19, 2026
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