Skip to content

feat: upgrade [email protected] #2032

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Oct 10, 2025
Merged

feat: upgrade [email protected] #2032

merged 9 commits into from
Oct 10, 2025

Conversation

@zerospiel
Copy link
Collaborator

@zerospiel zerospiel commented Sep 23, 2025 •
edited
Loading

What this PR does / why we need it:
Updates the following provider templates:

Updates the following cluster templates:

  • docker-hosted-cp
  • vsphere-standalone-cp
  • vsphere-hosted-cp

Updates the following go modules:

WARN

The following dependencies have been fetched from the main branches:

  • github.com/telekom/cluster-api-ipam-provider-infoblox

The sigs.k8s.io/cluster-api-ipam-provider-in-cluster has been updated to the v1.1.0-rc.1 version.

Updates the minimal required Go version to 1.25.1.

Adds a dedicated module for the deprecated v1alpha1 API to ensure the old dependencies.

Fixes lint (hence utils package rearrangement) issues.

Fixes the corresponding failed integration tests due to the changes in the cluster-api and fluxcd (several types migrated between modules) API.

Tool bumps:

Which issue(s) this PR fixes (optional, Fixes #123):
Closes #1850

@zerospiel
Copy link
Collaborator Author

zerospiel commented Sep 23, 2025

Waiting for the CAPI Operator kubernetes-sigs/cluster-api-operator#880 to be merged and the new tag in order to unblock the PR

@zerospiel zerospiel added the keep A PR or an issue should be kept for either further investigation or discussion label Sep 23, 2025
@zerospiel zerospiel force-pushed the capi_bump branch 3 times, most recently from bb9af3a to 79f7cd8 Compare September 30, 2025 16:10
@zerospiel zerospiel added test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran and removed keep A PR or an issue should be kept for either further investigation or discussion labels Oct 1, 2025
@zerospiel zerospiel removed the test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran label Oct 1, 2025
@zerospiel zerospiel marked this pull request as ready for review October 1, 2025 09:27
@zerospiel zerospiel force-pushed the capi_bump branch 3 times, most recently from d3997f0 to 9d167eb Compare October 1, 2025 09:34
@zerospiel zerospiel marked this pull request as draft October 1, 2025 10:57
@zerospiel
Copy link
Collaborator Author

zerospiel commented Oct 1, 2025 •
edited
Loading

Blockers:
- missing capi-operator chart with the code supporting v1beta2

(Potentially) Optional:

  • ipam-infoblox
  • ipam-in-cluster
  • CAPA/CAPG/CAPZ/CAPO (downstream; upstream is ready)
  • k0smotron

@zerospiel
Copy link
Collaborator Author

zerospiel commented Oct 6, 2025

Now blocked because of this change in the flag set by the cluster-api-operator, which k0smotron does not support, causing its deployments to crash

@zerospiel zerospiel force-pushed the capi_bump branch 2 times, most recently from 23dd7ed to 65a689b Compare October 8, 2025 16:13
@zerospiel zerospiel added the test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran label Oct 8, 2025
@zerospiel
chore(bumps): provider templates
14999a0 
Bumps kcm's flux chart dependency to 2.16.4.

Bumps the following providers to support capi v1.11.x:
- in-cluster-ipam to 1.1.0-rc.1
- capi to 1.11.1
- docker to 1.11.1
- vsphere to 1.14.0
- projectsveltos to 1.1.1
@zerospiel
chore(bump): extra charts bumps
15fad8e 
Bumps capi-operator chart to v0.24.0.

Bumps the following modules:
- github.com/vmware-tanzu/[email protected]
- github.com/onsi/ginkgo/[email protected]
- github.com/fluxcd/source-controller/[email protected]
- github.com/fluxcd/helm-controller/[email protected]
- sigs.k8s.io/[email protected]
@zerospiel
chore(ctpl): upgrade ct vsphere and docker
452673e
@zerospiel
chore(bumps): more deps and charts
3c77914 
Bumps the following charts:
- cert-manager to 1.19.0

Bumps the following providers:
- k0smotron to 1.8.0
- cluster-api to 1.11.2
- CAPD to 1.11.2

Bumps the following go modules:
- github.com/cert-manager/[email protected]
- github.com/fluxcd/pkg/apis/[email protected]
- github.com/fluxcd/pkg/[email protected]
- github.com/fluxcd/source-controller/[email protected]
- golang.org/x/[email protected]
- golang.org/x/[email protected]
- sigs.k8s.io/[email protected]
- sigs.k8s.io/[email protected]
@zerospiel
test(e2e): amend according to new conditions
3d59525
@zerospiel zerospiel removed the test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran label Oct 9, 2025
@zerospiel zerospiel marked this pull request as ready for review October 9, 2025 11:55
@zerospiel zerospiel added the test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran label Oct 9, 2025
@zerospiel zerospiel removed the test e2e Runs the entire provider E2E test suite, controller E2E tests are always ran label Oct 9, 2025
@zerospiel zerospiel changed the title feat: upgrade [email protected] feat: upgrade [email protected] Oct 9, 2025
@zerospiel
Copy link
Collaborator Author

zerospiel commented Oct 9, 2025

All bumps are done, the updated versions have been checked on several cloud providers and on the CAPD.

The e2e tests for the remote template should be amended further since the machine's readiness now includes the state of a node, on which tests do not install cni plugin, and the node isn't ready. Such an amendment is out of the scope for the already huge PR

kylewuolle
kylewuolle requested changes Oct 9, 2025
View reviewed changes
Copy link
Contributor

@kylewuolle kylewuolle left a comment •
edited
Loading

Choose a reason for hiding this comment

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

looks like clusterctl needs to be updated to v1.11.1 as well in the Makefile

kylewuolle
kylewuolle previously requested changes Oct 9, 2025
View reviewed changes
@zerospiel
chore(bumps): more bumps
cf02b2f 
Bumps:
- github.com/fluxcd/helm-controller/[email protected]
- github.com/fluxcd/pkg/[email protected]

Removes stale image tag from projectsveltos provider template.
@zerospiel zerospiel requested a review from kylewuolle October 10, 2025 09:24
eromanova
eromanova reviewed Oct 10, 2025
View reviewed changes
@zerospiel zerospiel merged commit 943e59f into k0rdent:main Oct 10, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this to Done in k0rdent Oct 10, 2025
@zerospiel zerospiel deleted the capi_bump branch October 10, 2025 12:39
This was referenced Oct 10, 2025
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eromanova eromanova eromanova approved these changes

@Kshatrix Kshatrix Awaiting requested review from Kshatrix Kshatrix is a code owner

@a13x5 a13x5 Awaiting requested review from a13x5 a13x5 is a code owner

@wahabmk wahabmk Awaiting requested review from wahabmk wahabmk is a code owner

@BROngineer BROngineer Awaiting requested review from BROngineer BROngineer is a code owner

@kylewuolle kylewuolle Awaiting requested review from kylewuolle kylewuolle is a code owner

Assignees

No one assigned

Labels

None yet

Projects

k0rdent
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migrate to clusterapi v1beta2

3 participants

@zerospiel @kylewuolle @eromanova