Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

Conversation

@angelos-p
Copy link
Contributor

@angelos-p angelos-p commented May 13, 2022

@angelos-p angelos-p requested a review from a team as a code owner May 13, 2022 09:52
@mhmxs
Copy link
Contributor

mhmxs commented May 13, 2022

@aeroniero33 Thanks for the fix, but RBAC is managed by Kubebuilder, so you have to edit here: https://github.com/storageos/operator/blob/main/main.go#L62 and then run make manifests. Thanks

@angelos-p
Copy link
Contributor Author

Ah thank you @mhmxs will do!

// +kubebuilder:rbac:groups=security.openshift.io,resources=securitycontextconstraints,resourceNames=privileged,verbs=get;create;update;delete;use
// +kubebuilder:rbac:groups=api.storageos.com,resources=volumes;nodes,verbs=create;patch;get;watch;delete;list
// +kubebuilder:rbac:groups=api.storageos.com,resources=volumes/status;nodes/status,verbs=get;update;patch
// +kubebuilder:rbac:groups=extensions,resources=podsecuritypolicies,verbs=use
Copy link
Contributor

Choose a reason for hiding this comment

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

You don't have to duplicate as i know. One place is enough.

@angelos-p angelos-p merged commit a3169c7 into main May 16, 2022
@angelos-p angelos-p deleted the add-psp-perms-clusterroles branch May 16, 2022 09:58
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