Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue warning when configuration incompatible with sticky session option is provided #10678

Open
2 tasks done
jnoordsij opened this issue May 2, 2024 · 1 comment
Open
2 tasks done
Labels
area/documentation contributor/wanted Participation from an external contributor is highly requested kind/enhancement a new or improved feature. priority/P3 maybe

Comments

@jnoordsij
Copy link
Contributor

Welcome!

  • Yes, I've searched similar issues on GitHub and didn't find any.
  • Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?

Traefik allows enabling sticky sessions for services, which set a cookie to identify the server handling the request. However, in some configurations, this makes little sense and might even be confusing to end-users.

The concrete confusing example that I found is using sticky sessions combined with the nativeLB option on a Kubernetes IngressRoute (or the new nativeLBByDefault option). In this case, the user might expect the cookie to ensure traffic is routed to the same pod, while in practice this does not happen as all traffic is routed through the "Kubernetes-level" service and no pod choice can be made. It would be nice to warn users that this combination of settings may not yield the results they expect.

I suspect there might be other cases where a warning could be relevant (e.g. cases where there's only a single backing service by configuration, making the sticky sessions flag obsolete), so the scope of this issue could be expanded to add additional validation.

@nmengin
Copy link
Contributor

nmengin commented May 2, 2024

Hy @jnoordsij,

Thank you for reaching out.

The idea sounds great, such documentation would help a lot of people.

Do not hesitate to open a PR: Traefik is an Open Source project and any help is appreciated (and, concerning the documentation, it's more than appreciated 😅 ).

@nmengin nmengin added kind/enhancement a new or improved feature. area/documentation priority/P3 maybe contributor/wanted Participation from an external contributor is highly requested and removed status/0-needs-triage labels May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation contributor/wanted Participation from an external contributor is highly requested kind/enhancement a new or improved feature. priority/P3 maybe
Projects
None yet
Development

No branches or pull requests

3 participants