cluster-api-addon-provider-helm icon indicating copy to clipboard operation
cluster-api-addon-provider-helm copied to clipboard

🌱 Bump CAPI to v1.7.1

Open mboersma opened this issue 2 years ago • 20 comments

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

mboersma avatar Mar 29 '24 17:03 mboersma

/area dependency

mboersma avatar Mar 29 '24 17:03 mboersma

The linter failure should be fixed by #181.

mboersma avatar Mar 29 '24 17:03 mboersma

/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.)

mboersma avatar Mar 29 '24 17:03 mboersma

/remove-label needs-rebase

mboersma avatar Apr 02 '24 12:04 mboersma

@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.

k8s-ci-robot avatar Apr 02 '24 12:04 k8s-ci-robot

/retitle Bump CAPI to v1.7.0-rc.0

mboersma avatar Apr 02 '24 19:04 mboersma

/retitle 🌱 Bump CAPI to v1.7.0-rc.0

mboersma avatar Apr 02 '24 19:04 mboersma

[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

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Apr 15 '24 13:04 k8s-ci-robot

/retitle 🌱 Bump CAPI to v1.7.0-rc.1

mboersma avatar Apr 15 '24 13:04 mboersma

/test ls

jackfrancis avatar Apr 15 '24 15:04 jackfrancis

@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.

k8s-ci-robot avatar Apr 15 '24 15:04 k8s-ci-robot

/test pull-cluster-api-addon-provider-helm-e2e-workload-upgrade

jackfrancis avatar Apr 15 '24 15:04 jackfrancis

/retitle 🌱 Bump CAPI to v1.7.0 /hold cancel

mboersma avatar Apr 16 '24 19:04 mboersma

/hold

Maybe we should merge #213 first.

mboersma avatar Apr 16 '24 19:04 mboersma

/retest

mboersma avatar Apr 17 '24 15:04 mboersma

/hold cancel

We can merge #213 first, or just merge this and have it close #213.

mboersma avatar Apr 22 '24 16:04 mboersma

/retest

mboersma avatar Apr 23 '24 12:04 mboersma

/test pull-cluster-api-addon-provider-helm-e2e-workload-upgrade

mboersma avatar Apr 24 '24 22:04 mboersma

@mboersma Think you need to rebase here.

Jont828 avatar Apr 29 '24 21:04 Jont828

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.

mboersma avatar Apr 29 '24 22:04 mboersma

@Jont828 this is ready for review.

mboersma avatar May 03 '24 17:05 mboersma