-
Notifications
You must be signed in to change notification settings - Fork 456
Bump CAPI to v1.9.6 #5530
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
Bump CAPI to v1.9.6 #5530
Conversation
/cherry-pick release-1.19 |
@mboersma: once the present PR merges, I will cherry-pick it on top of 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-sigs/prow repository. |
/cherry-pick release-1.18 |
@mboersma: once the present PR merges, I will cherry-pick it on top of 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-sigs/prow repository. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5530 +/- ##
=======================================
Coverage 52.86% 52.86%
=======================================
Files 272 272
Lines 29474 29474
=======================================
Hits 15582 15582
Misses 13080 13080
Partials 812 812 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
/test pull-cluster-api-provider-azure-e2e-workload-upgrade I was surprised this didn't run, so added this for the future: kubernetes/test-infra#34632 |
Failed to list deployments for the cluster API controllers
Expected success, but got an error:
<*errors.StatusError | 0xc000d1d040>:
deployments.apps is forbidden: User "system:serviceaccount:test-pods:azure" cannot list resource "deployments" in API group "apps" at the cluster scope
{ I think kubernetes-sigs/cluster-api#11865 caused these failures. |
/retest |
2033585
to
3f7f249
Compare
/retest |
3f7f249
to
0f580a3
Compare
Thank you for putting this together ! |
LGTM label has been added. Git tree hash: cbc85623100d9ee749df3fd60ef8d1ec733dca87
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: nawazkh 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 |
@mboersma: #5530 failed to apply on top of branch "release-1.19":
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-sigs/prow repository. |
@mboersma: #5530 failed to apply on top of branch "release-1.18":
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-sigs/prow repository. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Updates CAPI to v1.9.6.
Which issue(s) this PR fixes:
N/A, but see #5438 for prior art.
Special notes for your reviewer:
TODOs:
Release note: