-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
Cherry pick of #59686 #67224: Add cloud-provider policies to be applied via addon mgr #66872
Cherry pick of #59686 #67224: Add cloud-provider policies to be applied via addon mgr #66872
Conversation
This PR is not for the master branch but does not have the |
/hold Need test before applying this auto cherrypick. |
/hold |
|
No conflict observed between policies created by kube-apiserver when bootstrap and those created with addon when create-lb-controller. |
/hold cancel |
/priority important-soon |
/assign @bowei |
Why does this need to be cherrypicked? What is broken right now? |
/retest |
4b0036a
to
568e82c
Compare
c67da9f
to
07c9ae8
Compare
07c9ae8
to
eef2aa2
Compare
Manual test passed in e2e cluster:
Expected role/clusterroles observed. |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bowei, grayluck 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 |
/assign @mbohlool |
/priority critical-urgent |
@grayluck Please be aware that the release note on this PR will be ignored and the release tool will use two release notes on the original PRs. If you need to change the text of release notes, please do so on those PRs. |
That's fine. The release notes on the original PRs explains themselves. Thanks Mehdy! |
/retest |
Cherry pick of #59686 and #67224 on release-1.9.
#59686: Add cloud-provider policies to be applied via addon mgr
#67224 Add namespace for (cluster)role(binding) cloud-provider.