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
OCPBUGS-28819: Support Permanent Session Affinity #2046
OCPBUGS-28819: Support Permanent Session Affinity #2046
Conversation
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/test 4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade |
retesting this for now, but will double check versus next run |
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
OVN drops the first packet from a given client for every new session which is undesirable and a regression compared to how SDN behaves (it can be argued that this is an implementation detail but ultimately users want same behaviour and no drops). This is a complicated fix and will take time. The OVN fix will take time and in K8s there is no affinity without a timeout unfortunately. So in OVNKube we will introduce permanent session affinity as an alternative. Hence if user sets 86000 which is the highest timeout value it would mean no timeout. This is not how the K8s definition works, this will be an ovnkube implementation detail. There is no harm in changing what a 1 day timeout means from it being a day to it being infinite affinity because either ways OVN only supports upto 18hours of timeout value max (UINT_MAX). So currently range 18-24 is not used and is configured as 18 itself in OVN. So from this change forward, if timeout is set to 86000 by the user, they will get permanent session affinity. Signed-off-by: Surya Seetharaman <suryaseetharaman.9@gmail.com> (cherry picked from commit 88a5140) (cherry picked from commit 85e3fdf)
Since we are cherry-picking packaged library rerunning go mod vendor and tidy on latest Signed-off-by: Surya Seetharaman <suryaseetharaman.9@gmail.com>
714ddb1
to
3e6cc2a
Compare
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/retest-required |
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
@tssurya The PR needs backport assessed label. /label cherry-pick-approved |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: trozet, tssurya The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is invalid:
Comment In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/retest-required |
/jira refresh |
@tssurya: This pull request references Jira Issue OCPBUGS-28819, which is valid. The bug has been moved to the POST state. 6 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Jira (jechen@redhat.com), skipping review request. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/test 4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade |
/label backport-risk-assessed |
@tssurya: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
/test e2e-metal-ipi-ovn-dualstack |
4d6dd53
into
openshift:release-4.14
@tssurya: Jira Issue OCPBUGS-28819: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-28819 has been moved to the MODIFIED state. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
[ART PR BUILD NOTIFIER] This PR has been included in build ose-ovn-kubernetes-base-container-v4.14.0-202402151739.p0.g4d6dd53.assembly.stream.el9 for distgit ovn-kubernetes-base. |
Fix included in accepted release 4.14.0-0.nightly-2024-02-16-015828 |
CLEAN Cherry-pick of #2045