🌱 Bump CAPI to v1.7.1
What this PR does / why we need it:
Updates CAPI to v1.7.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 #190 Closes #213
/kind cleanup
/area dependency
The linter failure should be fixed by #181.
/hold
It looks like e2e tests are still pulling CAPI v1.6.1 components. I'll update this PR (and make a separate one to bump main to 1.6.3, which is where CAPI is in go.mod.)
/remove-label needs-rebase
@mboersma: The label(s) /remove-label needs-rebase cannot be applied. These labels are supported: api-review, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, team/katacoda, refactor. Is this label configured under labels -> additional_labels or labels -> restricted_labels in plugin.yaml?
In response to this:
/remove-label needs-rebase
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.
/retitle Bump CAPI to v1.7.0-rc.0
/retitle 🌱 Bump CAPI to v1.7.0-rc.0
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mboersma
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [mboersma]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
/retitle 🌱 Bump CAPI to v1.7.0-rc.1
/test ls
@jackfrancis: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:
-
/test pull-cluster-api-addon-provider-helm-build-main -
/test pull-cluster-api-addon-provider-helm-e2e -
/test pull-cluster-api-addon-provider-helm-test-main -
/test pull-cluster-api-addon-provider-helm-test-mink8s-main -
/test pull-cluster-api-addon-provider-helm-verify-main
The following commands are available to trigger optional jobs:
-
/test pull-cluster-api-addon-provider-helm-apidiff-main -
/test pull-cluster-api-addon-provider-helm-apiversion-upgrade -
/test pull-cluster-api-addon-provider-helm-capi-e2e -
/test pull-cluster-api-addon-provider-helm-e2e-workload-upgrade
Use /test all to run the following jobs that were automatically triggered:
-
pull-cluster-api-addon-provider-helm-apidiff-main -
pull-cluster-api-addon-provider-helm-apiversion-upgrade -
pull-cluster-api-addon-provider-helm-build-main -
pull-cluster-api-addon-provider-helm-capi-e2e -
pull-cluster-api-addon-provider-helm-e2e -
pull-cluster-api-addon-provider-helm-test-main -
pull-cluster-api-addon-provider-helm-test-mink8s-main -
pull-cluster-api-addon-provider-helm-verify-main
In response to this:
/test ls
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.
/test pull-cluster-api-addon-provider-helm-e2e-workload-upgrade
/retitle 🌱 Bump CAPI to v1.7.0 /hold cancel
/hold
Maybe we should merge #213 first.
/retest
/hold cancel
We can merge #213 first, or just merge this and have it close #213.
/retest
/test pull-cluster-api-addon-provider-helm-e2e-workload-upgrade
@mboersma Think you need to rebase here.
Think you need to rebase here.
Done!
Seems like CAAPH doesn't automatically get the needs-rebase tag applied or removed? I thought this was default prow behavior.
@Jont828 this is ready for review.