Skip to content

Conversation

@stephenfin
Copy link
Contributor

Achieved by bumping the library itself:

pushd cluster-api/providers/openstack
go get -u sigs.k8s.io/cluster-api-provider-openstack@latest
go mod tidy
go mod vendor
popd

Followed by the assets:

pushd <path-to-upstream-capo-repo>
git checkout v0.13.0
make release-manifests
popd
cp <path-to-upstream-capo-repo>/out/infrastructure-components.yaml \
  data/data/cluster-api/openstack-infrastructure-components.yaml

This has the side effect of bumping golang to 1.24 for this module.

@stephenfin stephenfin marked this pull request as draft November 17, 2025 12:36
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 17, 2025
@stephenfin
Copy link
Contributor Author

Currently WIP as I'm seeing dependency issues.

@stephenfin stephenfin marked this pull request as ready for review November 18, 2025 01:35
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 18, 2025
@stephenfin
Copy link
Contributor Author

stephenfin commented Nov 18, 2025

/retitle OSASINFRA-3965: Update cluster-api-provider-openstack to v0.13.0

@openshift-ci openshift-ci bot changed the title Update cluster-api-provider-openstack to v0.13.0 OSASINFRA-3965: Update cluster-api-provider-openstack to v0.13.0 Nov 18, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 18, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Nov 18, 2025

@stephenfin: This pull request references OSASINFRA-3965 which is a valid jira issue.

In response to this:

Achieved by bumping the library itself:

pushd cluster-api/providers/openstack
go get -u sigs.k8s.io/cluster-api-provider-openstack@latest
go mod tidy
go mod vendor
popd

Followed by the assets:

pushd <path-to-upstream-capo-repo>
git checkout v0.13.0
make release-manifests
popd
cp <path-to-upstream-capo-repo>/out/infrastructure-components.yaml \
 data/data/cluster-api/openstack-infrastructure-components.yaml

This has the side effect of bumping golang to 1.24 for this module.

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 openshift-eng/jira-lifecycle-plugin repository.

@stephenfin
Copy link
Contributor Author

/assign @mandre
/assign @patrickdillon

I believe this is good to go, and I was able to stand up a cluster successfully using a release image built with this PR.

This is split out from CAPO starting with CAPO v0.12.0. Start deploying it manually
in preparation for a CAPO bump.

Signed-off-by: Stephen Finucane <[email protected]>
Signed-off-by: Stephen Finucane <[email protected]>
Achieved by bumping the library itself:

  pushd cluster-api/providers/openstack
  go get -u sigs.k8s.io/cluster-api-provider-openstack@latest
  go mod tidy
  go mod vendor
  popd

Followed by the assets:

  pushd <path-to-upstream-capo-repo>
  git checkout v0.13.0
  make release-manifests
  popd
  cp <path-to-upstream-capo-repo>/out/infrastructure-components.yaml \
    data/data/cluster-api/openstack-infrastructure-components.yaml

This has the side effect of bumping golang to 1.24.

Signed-off-by: Stephen Finucane <[email protected]>
@stephenfin
Copy link
Contributor Author

Rebased onto main now that #10110 is merged.

@stephenfin
Copy link
Contributor Author

/test openstack-manifests

Slow mirrors?

@stephenfin
Copy link
Contributor Author

/retest

1 similar comment
@stephenfin
Copy link
Contributor Author

/retest

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 26, 2025

@stephenfin: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-azure-ovn-shared-vpc 112e876 link false /test e2e-azure-ovn-shared-vpc
ci/prow/e2e-azurestack 112e876 link false /test e2e-azurestack
ci/prow/e2e-aws-ovn-heterogeneous 112e876 link false /test e2e-aws-ovn-heterogeneous

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@stephenfin
Copy link
Contributor Author

I think this is good to go.

The last runs of e2e-openstack-proxy and e2e-openstack-nfv-intel were green. e2e-openstack-ovn is still red, but that's running on the managed cloud we use rather than the self-managed cloud, and we've been seeing a lot of issues with the former currently.

/assign @mandre

@stephenfin
Copy link
Contributor Author

/verified by multiple CI jobs

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Nov 27, 2025
@openshift-ci-robot
Copy link
Contributor

@stephenfin: This PR has been marked as verified by multiple CI jobs.

In response to this:

/verified by multiple CI jobs

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 openshift-eng/jira-lifecycle-plugin repository.

@stephenfin
Copy link
Contributor Author

/override ci/prow/e2e-openstack-ovn

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 27, 2025

@stephenfin: Overrode contexts on behalf of stephenfin: ci/prow/e2e-openstack-ovn

In response to this:

/override ci/prow/e2e-openstack-ovn

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.

Copy link
Member

@mandre mandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2025
@mandre
Copy link
Member

mandre commented Nov 28, 2025

I doubt I'm able to approve this patch, but let's try anyway.
/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mandre

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 28, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 7d0584e into openshift:main Nov 28, 2025
48 of 51 checks passed
@openshift-merge-bot openshift-merge-bot bot deleted the bump-capo-v0-13 branch November 28, 2025 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants