-
Notifications
You must be signed in to change notification settings - Fork 632
Description
What would you like to be added:
The conformance test results list agentgateway as it's own gateway api controller. As far as I can tell agentgateway is not a gateway api controller but rather a dataplane proxy. KGateway is a gateway api controller that supports both agentgateway and envoy as the dataplane proxy but is listed as kgateway for the envoy conformance and as agentgateway for the agentgateway conformance. I could understand the organization name being agentgateway if for example agentgateway support in KGateway is primarily managed by agentgateway, however at least the project name should imo be KGateway (Envoy) and KGateway (AgentGateway).
This was discussed here but does not seem to have been properly resolved (imo)
#4209 (comment)
The way the project would prefer to position it as is that it is Agentgateway with mode Kgateway rather than the inverse. This is also how it's described on the implementation list (Agentgateway with Kgateway).
I can understand why AgentGateway might want to position it this way, however AgentGateway simply is not a gateway api controller. This positioning is confusing to users.
The KGateway docs are pretty imo pretty clear so I don't know where this position comes from:
What is kgateway?
Kgateway is a control plane that implements the Kubernetes Gateway API for both microservices and AI workloads. The control plane translates your Kubernetes Gateway API resources into the configuration that the underlying data plane proxy can understand. The proxy layer is handled by kgateway’s implementation of Envoy for microservices or agentgateway for AI workloads.
Why this is needed:
In my opinion conformance tests should reference the controller implementation as the implementation name, not the dataplane proxy being used. This so users can make a fair comparison between different implementations. This is especially import now that ingress-nginx deprecation has been properly announced and many users will be exploring their migration options.