envoyproxy / gateway

Manages Envoy Proxy as a Standalone or Kubernetes-based Application Gateway
https://gateway.envoyproxy.io
Apache License 2.0
1.55k stars 335 forks source link

Session Persistence #3315

Closed arkodg closed 1 month ago

arkodg commented 5 months ago

Description:

Describe the desired behavior, what scenario it enables and how it would be used.

Add support for the upstream Session Persistence feature outlined in this GEP https://gateway-api.sigs.k8s.io/geps/gep-1619/

[optional Relevant Links:]

Any extra documentation required to understand the issue.

arkodg commented 5 months ago

blocked on https://github.com/envoyproxy/gateway/issues/3265

arkodg commented 5 months ago

related https://github.com/envoyproxy/gateway/issues/2624#issuecomment-2047564216

sanposhiho commented 2 months ago

/assign

arkodg commented 2 months ago

hey @sanposhiho thanks for picking this one up !

zhaohuabing commented 2 months ago

dont add support for BackendLBPolicy, it was specifically designed to support implementations without Route level support for session persistence, Envoy can support it, so imo lets reduce the number of ways to configure sessionPersistence https://github.com/envoyproxy/gateway/issues/2624#issuecomment-2047564216

@arkodg @sanposhiho Sorry for chiming in late, so the final decision is supporting session sticky in both EG BTP and Gateway BackendLBPolicy?

zhaohuabing commented 2 months ago

dont add support for BackendLBPolicy, it was specifically designed to support implementations without Route level support for session persistence, Envoy can support it, so imo lets reduce the number of ways to configure sessionPersistence #2624 (comment)

@arkodg @sanposhiho Sorry for chiming in late, so the final decision is supporting session sticky in both EG BTP and Gateway BackendLBPolicy?

NVM, I figured it out: strong affinity(Stateful session HTTP filter) vs weak affinity(Hash LB) :-)