-
Notifications
You must be signed in to change notification settings - Fork 491
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
promote HTTPRoute backend protocol from experimental to extended #3108
promote HTTPRoute backend protocol from experimental to extended #3108
Conversation
Is there any website documentation we need to update for this as well? Aside from that, lgtm, I'll approve. /approve |
Sorry yeah - I need to drop this notice on this page https://gateway-api.sigs.k8s.io/guides/backend-protocol/ |
done |
Oh I didn't realize you don't have branches for the docs - should I rollback the docs change? |
Yeah unfortunately that's probably the best for now, but I really don't want to miss changing docs in the future. Hoping someone can help us figure out a better approach to docs versioning to make this less painful in the future. |
Pulled the docs changes into a separate PR - #3120 |
d72fddb
to
2e48abc
Compare
I pulled out the docs changes into a separate PR. This change is ready to be merged. |
weird verify took 2h /retest |
Thanks @dprotaso! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso, robscott, youngnick 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 |
What type of PR is this?
/kind gep
/area conformance
What this PR does / why we need it:
This promotes backend protocol selection as part of Gateway v1.2 release
Which issue(s) this PR fixes:
Part of #1911
Does this PR introduce a user-facing change?: