Skip to content

Bump cluster-api to v1.11.1 and support v1beta2 api - #328

Merged
k8s-ci-robot merged 2 commits into
kubernetes-sigs:mainfrom
nunnatsa:bump_api-cluster-1.11.1
Sep 25, 2025
Merged

k8s-ci-robot merged 2 commits into
kubernetes-sigs:mainfrom
nunnatsa:bump_api-cluster-1.11.1

Conversation

@nunnatsa

@nunnatsa nunnatsa commented Sep 18, 2025 •

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

continue the work in #325, by @changjjjjjjj

  • Bump cluster api version to v1.11.1
  • Support cluster api v1beta2, especially for conditions (it changed from clusterv1.conditions to metav1.condition)
  • module updates for CVEs
  • Use golangci-lint v2 and fix all lint errors

Fixes #321

manual cherry pick of #330, by @sharath-srikanth-chellappa

This patch introduces TLS configuration enforcement for the webhook server in main.go.
Specifically, it ensures that the server uses TLS 1.2 or higher.

Fixes #329

Release notes:

Bump cluster-api to v1.11.1 and support v1beta2 api

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Sep 18, 2025
@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Sep 18, 2025
@nunnatsa

Copy link
Copy Markdown
Contributor Author

/ok-to-test

@k8s-ci-robot k8s-ci-robot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Sep 18, 2025
@coveralls

coveralls commented Sep 18, 2025 •

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 18008396631

Details

  • 109 of 213 (51.17%) changed or added relevant lines in 16 files are covered.
  • 506 unchanged lines in 12 files lost coverage.
  • Overall coverage decreased (-18.9%) to 40.037%

Changes Missing Coverage Covered Lines Changed/Added Lines %
controllers/kubevirtmachine_controller.go 81 82 98.78%
api/v1alpha1/kubevirtcluster_types.go 0 2 0.0%
api/v1alpha1/kubevirtmachine_types.go 0 2 0.0%
pkg/workloadcluster/workloadcluster.go 0 2 0.0%
pkg/kubevirt/utils.go 7 10 70.0%
pkg/ssh/ssh_command_executor.go 0 5 0.0%
api/v1alpha1/zz_generated.deepcopy.go 0 6 0.0%
pkg/ssh/utils.go 0 7 0.0%
pkg/testing/common.go 0 8 0.0%
main.go 0 10 0.0%
Files with Coverage Reduction New Missed Lines %
pkg/ssh/utils.go 1 51.47%
api/v1alpha1/kubevirtclustertemplate_types.go 3 0.0%
api/v1alpha1/kubevirtmachinetemplate_types.go 3 0.0%
api/v1alpha1/kubevirtcluster_types.go 7 0.0%
api/v1alpha1/kubevirtmachine_types.go 7 0.0%
pkg/context/cluster_context.go 10 0.0%
pkg/infracluster/mock/infracluster_generated.go 20 0.0%
pkg/workloadcluster/mock/workloadcluster_generated.go 30 0.0%
pkg/context/machine_context.go 32 0.0%
api/v1alpha1/zz_generated.deepcopy.go 99 0.0%
Totals Coverage Status
Change from base Build 16778497835: -18.9%
Covered Lines: 1091
Relevant Lines: 2725

💛 - Coveralls

@nunnatsa
nunnatsa force-pushed the bump_api-cluster-1.11.1 branch from 90e431d to f63e48a Compare September 18, 2025 11:00
@k8s-ci-robot k8s-ci-robot added the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Sep 25, 2025
@nunnatsa
nunnatsa force-pushed the bump_api-cluster-1.11.1 branch from 8c739cc to 4dce6f7 Compare September 25, 2025 04:51
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Sep 25, 2025
@nunnatsa
nunnatsa force-pushed the bump_api-cluster-1.11.1 branch 2 times, most recently from 82b22a4 to b216c45 Compare September 25, 2025 07:22
changjjjjjjj and others added 2 commits September 25, 2025 16:04
This is a manual cherry pick of sharath-srikanth-chellappa PR 330

Signed-off-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>
@qinqon

qinqon commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 25, 2025
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nunnatsa, qinqon

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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
k8s-ci-robot merged commit d46e41d into kubernetes-sigs:main Sep 25, 2025
15 of 16 checks passed
@nunnatsa
nunnatsa deleted the bump_api-cluster-1.11.1 branch September 25, 2025 13:58
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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enforce TLS v1.2 for the Webhook Server CAPI v1.11.0-beta.0 has been released and is ready for testing

5 participants