【FIX】fix yurt-controller-manager clusterrole missing pod delete #1338
Conversation
|
@JameKeal: GitHub didn't allow me to assign the following users: your_reviewer. Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
Kudos, SonarCloud Quality Gate passed!
|
|
You have successfully added a new Trivy configuration |
|
You have successfully added a new Trivy configuration |
|
You have successfully added a new Trivy configuration |
Codecov Report
@@ Coverage Diff @@
## master #1338 +/- ##
==========================================
+ Coverage 53.01% 53.08% +0.07%
==========================================
Files 102 102
Lines 12833 12833
==========================================
+ Hits 6803 6813 +10
+ Misses 5472 5462 -10
Partials 558 558
Flags with carried forward coverage won't be shown. Click here to find out more. see 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
/lgtm |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JameKeal, rambohe-ch The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JameKeal, rambohe-ch The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind bug
What this PR does / why we need it:
When i use daemonset auto updating, yurt-controller-manager have error, update failed.
yurt-controller-manager don`t have pods delete permission.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
@rambohe-ch
Does this PR introduce a user-facing change?
other Note