-
Notifications
You must be signed in to change notification settings - Fork 98
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
[DCA] [orchestrator] add pv and pvc rbac #365
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, documentation
@@ -1359,6 +1359,17 @@ func buildClusterAgentClusterRole(dda *datadoghqv1alpha1.DatadogAgent, name, age | |||
datadoghqv1alpha1.GetVerb, | |||
}, | |||
}) | |||
|
|||
// PV and PVC | |||
rbacRules = append(rbacRules, rbacv1.PolicyRule{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it could be good to add unit-test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you also add RBAC in this file https://github.com/DataDog/datadog-operator/blob/main/controllers/datadogagent_controller.go#L108
like this the Operator RBAC will be automatically generated, to allow him to create the cluster-checks-runner or dca rbac
you will need to run: make generate
after updating this file
3b2f66a
to
d1797a5
Compare
What does this PR do?
add pv and pvc rbac
Motivation
collect pv and pvc for orchestrator
Additional Notes
Anything else we should know when reviewing?
Describe your test plan
Write there any instructions and details you may have to test your PR.