Skip to content

Conversation

@gman0
Copy link
Contributor

@gman0 gman0 commented Apr 25, 2025

Summary

This PR syncs kcp to be in line with k8s v1.32.3.

Things of note:

Other TODO items that will need to be worked on after the rebase:

What Type of PR Is This?

/kind feature

Related Issue(s)

Fixes #3209

Release Notes

* Rebased to Kubernetes v1.32.3
* ValidatingAdmissionPolicy has been removed from admissionregistration.k8s.io/v1alpha1 as per Kubernetes v1.32 API lifecycle management
* admissionregistration.k8s.io/v1alpha1 is no longer being served by default

gman0 added 15 commits April 25, 2025 18:29
On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
Also replaced k8s.io/apiserver/pkg/util/version import that is now removed
with its correct path k8s.io/component-base/version.

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
Ran:
    make codegen

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
garbagecollector.Run now requires an intial timeout argument.

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
The feature gate is now stable and removed.

See kubernetes/kubernetes#126645

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
k8s.io/kubernetes/pkg/controlplane/apiserver/options/options.Complete() has a different
signature that includes a ctx, and so we need to wire it to its callers.

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
coordination.k8s.io/v1alpha2 replaced v1alpha1, see k/k#127857.

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
@kcp-ci-bot kcp-ci-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. dco-signoff: yes Indicates the PR's author has signed the DCO. area/cli Issues or PRs related to CLI changes kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Apr 25, 2025
@gman0 gman0 added kind/feature Categorizes issue or PR as related to a new feature. area/kubernetes Issues or changes for Kubernetes as it relates to kcp area/dependency Issues or PRs related to dependency changes area/core Issues or PRs related to core changes and removed kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. labels Apr 25, 2025
@gman0 gman0 mentioned this pull request Apr 28, 2025
1 task
@embik embik self-requested a review April 28, 2025 08:03
gman0 added 2 commits April 28, 2025 14:34
Kubernetes v1.32 implicitly removes ValidatingAdmissionPolicy and
ValidatingAdmissionPolicyBinding from admissionregistration.k8s.io/v1alpha
in accordance with APILifecycle policy.

This commit removes the relevant CRDs, informers and APIExport builtin types,
and also removes the admissionregistration.k8s.io/v1alpha1 override from
pkg/server/options/options.go.

On-behalf-of: @SAP [email protected]
Signed-off-by: Robert Vasek <[email protected]>
@gman0 gman0 force-pushed the updates-kcp-1.32.3 branch from 3b10a1b to 90b3b75 Compare April 28, 2025 12:40
@embik
Copy link
Member

embik commented Apr 28, 2025

/retest

Copy link
Member

@embik embik left a comment

Choose a reason for hiding this comment

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

LGTM, will leave final approval up for a while if someone else wants to review as well.

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Apr 28, 2025
@kcp-ci-bot
Copy link
Contributor

LGTM label has been added.

Git tree hash: ed0b1c3eb374b6393ef305f574724a5547bd45bd

@embik
Copy link
Member

embik commented Apr 28, 2025

/retest

@kcp-ci-bot
Copy link
Contributor

@ntnn: changing LGTM is restricted to collaborators

In response to this:

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.

@gman0
Copy link
Contributor Author

gman0 commented Apr 28, 2025

/retest

2 similar comments
@gman0
Copy link
Contributor Author

gman0 commented Apr 28, 2025

/retest

@gman0
Copy link
Contributor Author

gman0 commented Apr 28, 2025

/retest

@mjudeikis
Copy link
Contributor

/approve
/lgtm

@kcp-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mjudeikis

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

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 29, 2025
@kcp-ci-bot kcp-ci-bot merged commit 12c3075 into kcp-dev:main Apr 29, 2025
13 checks passed
@embik embik added this to the v0.28.0 milestone May 30, 2025
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. area/cli Issues or PRs related to CLI changes area/core Issues or PRs related to core changes area/dependency Issues or PRs related to dependency changes area/kubernetes Issues or changes for Kubernetes as it relates to kcp dco-signoff: yes Indicates the PR's author has signed the DCO. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. 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.

feature: rebase to Kubernetes 1.32.3

5 participants