Adjust sticky session logic in dynamic mode to match static mode

[ElvinEfendi]

In #2451 we started to conditionally set Path parameter on the generated cookie. But the PR did not adjust

ingress-nginx/rootfs/etc/nginx/lua/balancer/sticky.lua

Line 44 in ced6c5b

path = "/",

accordingly. So people using sticky sessions in dynamic mode will see unexpected behaviour.

We should make sure they are consistent.

[mbugeia]

Hi,
Can you explain what "unexpected behaviour" should we expect by mixing dynamic mode and sticky session ? Is it usable in production ?

[ElvinEfendi]

Hi @mbugeia, please refer to #1930 and #2451.

Is it usable in production ?

yes, I've been using it in production with dynamic mode enabled.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[fejta-bot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

[fejta-bot]

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

[k8s-ci-robot]

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.