Skip to content
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.

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

api: deprecate v1alpha1 #1044

Merged
merged 1 commit into from
Oct 17, 2024
Merged

api: deprecate v1alpha1 #1044

merged 1 commit into from
Oct 17, 2024

Conversation

ffromani
Copy link
Member

@ffromani ffromani commented Oct 15, 2024

The API v1alpha1 was not only deprecated before GA, but never advertised. It's time to deprecate. Later we will stop servingf it in 4.18, and remove in 4.19/20.

This change can and should be backported back to 4.16 or perhaps even below

Key references:

@ffromani ffromani added the cherry-pick-candidate Possible cherry-pick in the future label Oct 15, 2024
@openshift-ci openshift-ci bot requested review from shajmakh and Tal-or October 15, 2024 14:31
Copy link
Contributor

openshift-ci bot commented Oct 15, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ffromani

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 Oct 15, 2024
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 16, 2024
@ffromani ffromani force-pushed the deprecate-crd-v1alpha1 branch from 95e0904 to 2ce5c8a Compare October 17, 2024 05:20
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 17, 2024
@ffromani ffromani force-pushed the deprecate-crd-v1alpha1 branch from 2ce5c8a to 80ffff2 Compare October 17, 2024 06:40
@ffromani ffromani changed the title api: stop serving v1alpha1 api: deprecate v1alpha1 Oct 17, 2024
use kube mechanism to make it really loud and obvious
this API version is deprecated

xref: https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definition-versioning/#version-deprecation

Signed-off-by: Francesco Romani <[email protected]>
@ffromani ffromani force-pushed the deprecate-crd-v1alpha1 branch from 80ffff2 to 446414b Compare October 17, 2024 06:42
@ffromani
Copy link
Member Author

/cherry-pick release-4.17

@openshift-cherrypick-robot

@ffromani: once the present PR merges, I will cherry-pick it on top of release-4.17 in a new PR and assign it to you.

In response to this:

/cherry-pick release-4.17

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.

@ffromani ffromani merged commit d6be3e2 into main Oct 17, 2024
13 of 14 checks passed
@ffromani ffromani deleted the deprecate-crd-v1alpha1 branch October 17, 2024 09:15
@openshift-cherrypick-robot

@ffromani: #1044 failed to apply on top of branch "release-4.17":

Applying: api: deprecate v1alpha1
Using index info to reconstruct a base tree...
M	bundle/manifests/nodetopology.openshift.io_numaresourcesoperators.yaml
M	bundle/manifests/nodetopology.openshift.io_numaresourcesschedulers.yaml
M	bundle/manifests/numaresources-operator.clusterserviceversion.yaml
M	config/crd/bases/nodetopology.openshift.io_numaresourcesoperators.yaml
M	config/crd/bases/nodetopology.openshift.io_numaresourcesschedulers.yaml
Falling back to patching base and 3-way merge...
Auto-merging config/crd/bases/nodetopology.openshift.io_numaresourcesschedulers.yaml
Auto-merging config/crd/bases/nodetopology.openshift.io_numaresourcesoperators.yaml
Auto-merging bundle/manifests/numaresources-operator.clusterserviceversion.yaml
CONFLICT (content): Merge conflict in bundle/manifests/numaresources-operator.clusterserviceversion.yaml
Auto-merging bundle/manifests/nodetopology.openshift.io_numaresourcesschedulers.yaml
Auto-merging bundle/manifests/nodetopology.openshift.io_numaresourcesoperators.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 api: deprecate v1alpha1

In response to this:

/cherry-pick release-4.17

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.

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. cherry-pick-candidate Possible cherry-pick in the future
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants