istio.io icon indicating copy to clipboard operation
istio.io copied to clipboard

Review docs with regard to Gateway API moving to Beta

Open craigbox opened this issue 1 year ago • 7 comments

Ambient mode requires you to configure your routes with the Gateway API. This is in Beta in 1.22, and I also understand the GAMMA work has all moved to the stable channel.

I've removed

Configuring internal mesh traffic using the Gateway API is an experimental feature currently under development.

from /docs/tasks/traffic-management/ingress/gateway-api/index.md, but the fact this path has "ingress" in it, means we haven't updated the docs to reflect that Gateway API is now Beta for mesh traffic too.

The gateway-api-gamma-support.md boilerplate also says

Configure Istio to read the alpha Gateway API resources by setting the PILOT_ENABLE_ALPHA_GATEWAY_API environment variable to true when installing Istio:

Could someone please go through and check which of these instructions are still necessary for 1.22?

We can then go on and look at how to revamp the docs per #14229, but probably not before the launch/

craigbox avatar May 08 '24 00:05 craigbox

PILOT_ENABLE_ALPHA_GATEWAY_API is about APIs that are not v1 in gw. Tcproute and tlsroute mostly.

this is 100% orthogonal from mesh support. mesh support has no API changes at all. The GW 1.1 release (coming in a few days) just declared it stable, but it has no API changes (and never did since day 0).

Istio 1.22 should claim mesh with gateway as beta.

That doesn't change that some APIs are alpha upstream and require an explicit opt in

howardjohn avatar May 08 '24 03:05 howardjohn

How do we feel about TCPRoute and TLSRoute for mesh support given it requires enablement via PILOT_ENABLE_ALPHA_GATEWAY_API? Should we still claim beta for it? see https://preliminary.istio.io/latest/docs/ambient/usage/waypoint/#attachl7policies

linsun avatar May 08 '24 13:05 linsun

No. We should probably say those 2 are alpha and maybe say "alpha resources are due to the resources themselves being alpha"

howardjohn avatar May 08 '24 13:05 howardjohn

Sounds good, thinking the same too! I'll submit a quick PR to get it updated.

linsun avatar May 08 '24 13:05 linsun

if TCPRoute and TLSRoute are going to be in Alpha for a while because Istio is the only implementation, should we make it so hard for people to use them? 🤔

craigbox avatar May 10 '24 10:05 craigbox

Problem is alpha -> stable promotion basically totally breaks in Gateway API's versioning model. Its not like Istio's model currently

howardjohn avatar May 10 '24 14:05 howardjohn

Oh but we put that everywhere. We should really not do that. A page like https://preliminary.istio.io/latest/docs/tasks/traffic-management/request-routing/ has no business telling users to turn on alpha!! it doesn't need it.

What we should do is on the few pages that do, say "This feature of GW API is alpha. We recommend you don't use it yet. If you do want to, here is how..."

howardjohn avatar May 10 '24 18:05 howardjohn