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

[1.14] gateway-api: update types to read v1beta1 if present (#41238) #41259

Merged
merged 6 commits into from
Oct 6, 2022

Conversation

howardjohn
Copy link
Member

@howardjohn howardjohn commented Oct 4, 2022

  • gateway-api: update types to read v1beta1 if present

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.

(cherry picked from commit fc33f94)

  • fixes

(cherry picked from commit dbf424b)

Please provide a description of this PR:

)

* gateway-api: update types to read v1beta1 if present

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.

(cherry picked from commit fc33f94)

* fixes

(cherry picked from commit dbf424b)
@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 Oct 4, 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 Oct 4, 2022
@howardjohn
Copy link
Member Author

/test all

@howardjohn
Copy link
Member Author

/test all

1 similar comment
@howardjohn
Copy link
Member Author

/test all

@howardjohn
Copy link
Member Author

/retest

@howardjohn
Copy link
Member Author

/test all

@howardjohn
Copy link
Member Author

/test all

@howardjohn
Copy link
Member Author

/tests all

@howardjohn
Copy link
Member Author

/test all

go.mod Show resolved Hide resolved
@howardjohn
Copy link
Member Author

/retest

@howardjohn howardjohn marked this pull request as ready for review October 5, 2022 23:45
@howardjohn howardjohn requested a review from a team October 5, 2022 23:45
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Oct 5, 2022
@istio-testing istio-testing merged commit 84a45e7 into istio:release-1.14 Oct 6, 2022
howardjohn added a commit to howardjohn/istio that referenced this pull request Oct 6, 2022
) (istio#41259)

* [1.14] gateway-api: update types to read v1beta1 if present (istio#41238)

* gateway-api: update types to read v1beta1 if present

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.

(cherry picked from commit fc33f94)

* fixes

(cherry picked from commit dbf424b)

* fixes

* fixes

* fix conformance

* lint

* remove entirely

(cherry picked from commit 84a45e7)
istio-testing pushed a commit that referenced this pull request Oct 6, 2022
…41259) (#41289)

* [1.14] gateway-api: update types to read v1beta1 if present (#41238)

* gateway-api: update types to read v1beta1 if present

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.

(cherry picked from commit fc33f94)

* fixes

(cherry picked from commit dbf424b)

* fixes

* fixes

* fix conformance

* lint

* remove entirely

(cherry picked from commit 84a45e7)
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.

3 participants