-
Notifications
You must be signed in to change notification settings - Fork 425
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.7.0-rc.1 #4646
Bump CAPI to v1.7.0-rc.1 #4646
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This comment was marked as resolved.
This comment was marked as resolved.
4266120
to
9822684
Compare
9822684
to
de57311
Compare
a9414bd
to
b6d5c87
Compare
b6d5c87
to
c8b5601
Compare
/retitle [WIP] Bump CAPI to v1.7.0-beta.1 |
c8b5601
to
92df662
Compare
This comment was marked as resolved.
This comment was marked as resolved.
4ada6b7
to
d787ca4
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4646 +/- ##
=======================================
Coverage 62.47% 62.47%
=======================================
Files 195 195
Lines 15624 15624
=======================================
Hits 9761 9761
Misses 5178 5178
Partials 685 685 ☔ View full report in Codecov by Sentry. |
The e2e tests are failing when trying to do Edit: I'd forgotten to add a 1.7 stanza to the test metadata file apparently. 🤦🏻 |
6363bb3
to
8502354
Compare
/retitle Bump CAPI to v1.7.0-beta.1 |
/test pull-cluster-api-provider-azure-e2e-optional |
/test pull-cluster-api-provider-azure-conformance |
8502354
to
5238319
Compare
Rebased after #4529 merged. |
/retest |
5238319
to
0636bec
Compare
/retitle Bump CAPI to v1.7.0-rc.0 |
0636bec
to
660f858
Compare
@mboersma: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
/retest The AKS test specs pass for me when I run e2e locally. ❓ |
660f858
to
069f821
Compare
/retitle Bump CAPI to v1.7.0-rc.1 |
/close See #4740 |
@mboersma: Closed this PR. 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. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Updates CAPI to v1.7.0-rc.1 and all that entails. This PR is for testing and isn't intended to be merged yet. We will refresh it when CAPI v1.7.0 is released.
Which issue(s) this PR fixes:
Fixes #4647
Special notes for your reviewer:
See the migration guide for more information.
TODOs:
Release note: