rollouts-plugin-trafficrouter-gatewayapi icon indicating copy to clipboard operation
rollouts-plugin-trafficrouter-gatewayapi copied to clipboard

Linkerd marked as supported, but does not support GRPCRoute

Open FredrikAugust opened this issue 4 months ago • 0 comments

Hey, so I've been using this plugin for a little while with linkerd, and been following the development quite closely here, on linkerd2, and gateway api itself. From what I understand, which might be incorrect and in which case please do arrest me, linkerd partially supports this plugin. I.e. it supports HTTPRoute, but not GRPCRoute.

Partially due to my fault, this plugin recently upgraded to use the stable release of the Gateway API which only serves v1 of the GRPCRoute CRD. This works fine with e.g. Traefik which delivers this version, but linkerd does not serve this version — for which I opened an issue. They reference this line in the release notes as the motivation for holding back:

If you are already using the experimental version GRPCRoute, we recommend holding off on upgrading to the standard channel version of GRPCRoute until the controllers you're using have been updated to support GRPCRoute v1. Until then, it is safe to upgrade to the experimental channel version of GRPCRoute in v1.1 that includes both v1alpha2 and v1 API versions.

And say that GCP for example does not ship the v1.

Another snippet from the discussion by @olix0r:

I think the situation is the following:

  • The stable release channel does not include alpha versions, only beta versions.
  • There are no beta versions of GRPCRoute.
  • The release notes for v1.1 explicitly mention that users should only upgrade to the experimental channel of v1.1 to maintain compatibility with controllers that use GRPCRoute v1alpha2 (like Linkerd).
  • Linkerd is currently unable to upgrade to the stable release channel of v1.1, since cloud providers like Google Cloud aren't actually shipping it yet.

This also makes me wonder if perhaps this plugin should have upgraded to the experimental channel instead and continued to reference v1alpha2 to maintain better compatibility?

FredrikAugust avatar Sep 27 '24 12:09 FredrikAugust