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

Remove additional deprecated CRD apis #6239

Merged

Conversation

antoninbas
Copy link
Contributor

This is a follow-up to #6162

We remove the following APIs in preparation for Antrea v2:
v1alpha3.ClusterGroup, v1alpha3.Group, v1alpha2.ExternalIPPool.

All 3 of them are Alpha APIs which have been deprecated since Antrea v1.13.

@antoninbas antoninbas changed the title Remove additional deprecated crd apis Remove additional deprecated CRD apis Apr 18, 2024
@antoninbas antoninbas force-pushed the remove-additional-deprecated-crd-apis branch from afbd55b to e2ffd23 Compare April 18, 2024 17:07
@antoninbas antoninbas added kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API. action/release-note Indicates a PR that should be included in release notes. labels Apr 18, 2024
This is a follow-up to antrea-io#6162

We remove the following APIs in preparation for Antrea v2:
v1alpha3.ClusterGroup, v1alpha3.Group, v1alpha2.ExternalIPPool.

All 3 of them are Alpha APIs which have been deprecated since Antrea
v1.13.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
It should have been v1beta1

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
@antoninbas antoninbas force-pushed the remove-additional-deprecated-crd-apis branch from e2ffd23 to 09b26b2 Compare April 18, 2024 17:22
While the API was not officially marked as deprecated in the
documentation (docs/api.md), we did mention its deprecation in the
CHANGELOG for the 1.13 release. It makes sense to delete this API at the
same time as the other deprecated Alpha APIs, for Antrea 2.0.

Signed-off-by: Antonin Bas <antonin.bas@broadcom.com>
Copy link
Contributor

@luolanzone luolanzone left a comment

Choose a reason for hiding this comment

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

LGTM, @edwardbadboy could you also take a look at this and make sure there is no impact to your side if we remove all deprecated APIs? Thanks.

@antoninbas
Copy link
Contributor Author

/test-all

@antoninbas antoninbas merged commit 7876c3f into antrea-io:main Apr 22, 2024
51 of 56 checks passed
@antoninbas antoninbas deleted the remove-additional-deprecated-crd-apis branch April 22, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/release-note Indicates a PR that should be included in release notes. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants