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

gateway-api: update types to read v1beta1 #41092

Merged
merged 8 commits into from
Sep 30, 2022

Conversation

howardjohn
Copy link
Member

@howardjohn howardjohn commented Sep 21, 2022

The upstream API is dropping support for the v1alpha2 types for
HTTPRoute, Gateway, and GatewayClass in the next release. So, unlike
Istio where we always read the alpha versions in Istiod, we must
switch to reading the beta types or users on v0.6.0 (next release of
API) will be broken.

This means that the minimum version of the API is v0.5.0; most users are
upgrading quickly since v0.4.0 was still an alpha API.

@istio-testing
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Sep 21, 2022
@howardjohn
Copy link
Member Author

/test all

@istio-testing istio-testing added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Sep 21, 2022
@howardjohn howardjohn marked this pull request as ready for review September 21, 2022 19:02
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Sep 21, 2022
@howardjohn
Copy link
Member Author

/retest

@howardjohn howardjohn force-pushed the gateway-api/use-beta-types branch from 1e7a061 to 5f84c40 Compare September 22, 2022 19:29
@istio-testing istio-testing added the needs-rebase Indicates a PR needs to be rebased before being merged label Sep 26, 2022
@howardjohn howardjohn force-pushed the gateway-api/use-beta-types branch from 5f84c40 to 1e0b248 Compare September 27, 2022 17:29
The upstream API is dropping support for the v1alpha2 types for
HTTPRoute, Gateway, and GatewayClass in the next release. So, unlike
Istio where we always read the alpha versions in Istiod, we must
switch to reading the beta types or users on v0.6.0 (next release of
API) will be broken.

This means that the minimum version of the API is v0.5.0; most users are
upgrading quickly since v0.4.0 was still an alpha API.
@howardjohn howardjohn force-pushed the gateway-api/use-beta-types branch from 1e0b248 to c5345e8 Compare September 30, 2022 16:38
@istio-testing istio-testing removed the needs-rebase Indicates a PR needs to be rebased before being merged label Sep 30, 2022
@howardjohn howardjohn force-pushed the gateway-api/use-beta-types branch from c5345e8 to ef34bc9 Compare September 30, 2022 16:39
@howardjohn howardjohn force-pushed the gateway-api/use-beta-types branch from ef34bc9 to 454b6be Compare September 30, 2022 17:19
@costinm
Copy link
Contributor

costinm commented Sep 30, 2022

I think API moving to beta is a perfect moment to start recommending users, starting with 1.16, to use HttpRoute and k8s Gateway in preference to VS/old Istio Gateway.

And maybe start updating istio.io examples.

@howardjohn
Copy link
Member Author

howardjohn commented Sep 30, 2022 via email

@istio-testing istio-testing merged commit ba4684c into istio:master Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

5 participants