ztunnel icon indicating copy to clipboard operation
ztunnel copied to clipboard

Support full Service API

Open howardjohn opened this issue 3 years ago • 5 comments

If we are going to be a drop in we need to support the full featuteset of Service. This includes the locality aware stuff, affinity, etc.

Depends on https://github.com/istio/ztunnel/issues/152. This should have a proper design before any implementation

howardjohn avatar Nov 28 '22 03:11 howardjohn

cc

hzxuzhonghu avatar Nov 30 '22 09:11 hzxuzhonghu

internalTrafficPolicy: done https://github.com/istio/ztunnel/pull/868 clusterIP, ports: done sessionAffinity: https://github.com/istio/ztunnel/issues/450, TODO externalName: nothing needed externalIPs: I think nothing needed?

So we are in pretty good shape. sessionAffinity is rarely used so I don't mind putting that off for a while

howardjohn avatar Apr 04 '24 21:04 howardjohn

I want to implement serviceAffinity if possible.

My design is like what the k8s does as https://kubernetes.io/docs/reference/networking/virtual-ips/#session-affinity.

  • Add 2 mode:

    • None
    • ClientIP
  • Add seesion timeout.

  • Only work without waypoint enabled.

daixiang0 avatar Apr 11 '24 07:04 daixiang0

Do you have a strong use case for this? I worry the complexity of this outweighs the benefits, at least for the next 6 months or so while we are working on stabilizing core

howardjohn avatar Apr 11 '24 16:04 howardjohn

AFAIK the only remaining is https://github.com/istio/ztunnel/issues/450

howardjohn avatar Aug 23 '24 15:08 howardjohn