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

fix csi rbac in kustomize #107

Merged
merged 1 commit into from
Aug 8, 2024
Merged

fix csi rbac in kustomize #107

merged 1 commit into from
Aug 8, 2024

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Aug 8, 2024

Moving the csi RBAC to a new folder can be built and applied differently and we dont need to have the same namePrefix and namespacePrefix for these files as this is not required to be deployed in the same namespace as the operator.

fixes: #87
fixes: #106

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 8, 2024

Note:- This still needs some changes to work for both downstream and upstream. putting this one hold for now

@Madhu-1 Madhu-1 force-pushed the fix-csi-rbac branch 2 times, most recently from 40ed3f3 to b4788f9 Compare August 8, 2024 08:13
@Madhu-1 Madhu-1 force-pushed the fix-csi-rbac branch 6 times, most recently from a77b19c to 2293d74 Compare August 8, 2024 13:24
@Madhu-1 Madhu-1 force-pushed the fix-csi-rbac branch 2 times, most recently from 4ee14ad to 9072e84 Compare August 8, 2024 13:35
Copy link
Collaborator Author

@Madhu-1 Madhu-1 left a comment

Choose a reason for hiding this comment

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

Am not fixing the variable to set the prefix in deployment and code changes in this PR.

@Madhu-1 Madhu-1 requested a review from nb-ohad August 8, 2024 13:38
@Madhu-1 Madhu-1 force-pushed the fix-csi-rbac branch 3 times, most recently from 16f183a to 06ce496 Compare August 8, 2024 13:56
Moving the csi RBAC to a new folder
which can be build and applied differently
and we dont need to have same namePrefix
and namespacePrefix for these files as this
is not required to be deployed in the same
namespace as the operator.

Signed-off-by: Madhu Rajanna <[email protected]>
@Madhu-1 Madhu-1 requested a review from nb-ohad August 8, 2024 13:58
@nb-ohad nb-ohad merged commit 25fe09b into ceph:main Aug 8, 2024
8 checks 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
2 participants