On Tue, Dec 20, 2022 at 4:31 PM Dumitru Ceara <dce...@redhat.com> wrote:
> The OVN feature used to implement session affinity timeout doesn't exist > on this branch. ovn-kubernetes code already moved to implementing the > feature by using the newer OVN option so, in order to keep testing as > many features as possible in CI, we now disable all session affinity > tests (but keep executing the rest). > > Signed-off-by: Dumitru Ceara <dce...@redhat.com> > --- > NOTE: This commit should only be applied to branches <= branch-22.09. > --- > .ci/ovn-kubernetes/custom.patch | 31 +++++++++++++++++++++++++++++++ > 1 file changed, 31 insertions(+) > > diff --git a/.ci/ovn-kubernetes/custom.patch > b/.ci/ovn-kubernetes/custom.patch > index e69de29bb2..ea5dd75408 100644 > --- a/.ci/ovn-kubernetes/custom.patch > +++ b/.ci/ovn-kubernetes/custom.patch > @@ -0,0 +1,31 @@ > +From 903eef2dd6f9fec818a580760f4757d8137b9974 Mon Sep 17 00:00:00 2001 > +From: Dumitru Ceara <dce...@redhat.com> > +Date: Mon, 19 Dec 2022 12:18:55 +0100 > +Subject: [PATCH] DOWNSTREAM: Disable session affinity tests. > + > +Commit https://github.com/ovn-org/ovn-kubernetes/commit/898d2f8f10c4 > +enabled affinity timeout tests but the underlying OVN feature is > +not supported in this branch. Disable affinity tests. > + > +Signed-off-by: Dumitru Ceara <dce...@redhat.com> > +--- > + test/scripts/e2e-kind.sh | 3 +++ > + 1 file changed, 3 insertions(+) > + > +diff --git a/test/scripts/e2e-kind.sh b/test/scripts/e2e-kind.sh > +index 69959fa1b..c3b2a5c3e 100755 > +--- a/test/scripts/e2e-kind.sh > ++++ b/test/scripts/e2e-kind.sh > +@@ -26,6 +26,9 @@ kube-proxy > + should set TCP CLOSE_WAIT timeout > + \[Feature:ProxyTerminatingEndpoints\] > + > ++# Disable session affinity tests completely. > ++session affinity > ++ > + # NOT IMPLEMENTED; SEE DISCUSSION IN > https://github.com/ovn-org/ovn-kubernetes/pull/1225 > + named port.+\[Feature:NetworkPolicy\] > + > +-- > +2.31.1 > + > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Looks good to me, thanks. Acked-by: Ales Musil <amu...@redhat.com> -- Ales Musil Senior Software Engineer - OVN Core Red Hat EMEA <https://www.redhat.com> amu...@redhat.com IM: amusil <https://red.ht/sig> _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
