🐛 Fix eks awsmanagedcontrolplane finalizer permission - #5668
Conversation
Signed-off-by: serngawy <serngawy@gmail.com>
damdo
left a comment
There was a problem hiding this comment.
/lgtm
/assign @richardcase @nrb
|
LGTM label has been added. DetailsGit tree hash: 6c8decb9c04b7d334ecff51aed50791196af268f |
|
/approve |
|
/cherry-pick release-2.9 |
|
@richardcase: once the present PR merges, I will cherry-pick it on top of DetailsIn 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-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: richardcase The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@richardcase: new pull request created: #5669 DetailsIn 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-sigs/prow repository. |
The EKS is provisioned successfully. But the cluster kubeconfig is missing in the cluster namespace.
find a error condition in AWSManagedControlPlane
the error message in capa-manager-contrller
after add awsmanagedcontrolplanes/finalizer update permission in the clusterrole capa-manager-role, it works.
What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Checklist:
Release note: