diff mbox series

[ovs-dev,12/15] tests: fixed "SCTP Load balancer health checks"a

Message ID 20230918164714.3144984-13-xsimonar@redhat.com
State Superseded
Headers show
Series Fixed another set of flaky Unit Tests | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes fail github build: failed

Commit Message

Xavier Simonart Sept. 18, 2023, 4:47 p.m. UTC 
There is no guarantee that ovn-northd and not its 'backup' gets
the lock. Hence we need to check both logs.

Signed-off-by: Xavier Simonart <xsimonar@redhat.com>
---
 tests/ovn.at | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/tests/ovn.at b/tests/ovn.at
index d4e0016e5..02c763593 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -25418,7 +25418,9 @@  check_row_count Service_Monitor 0
 # Let's also be sure the warning message about SCTP load balancers is
 # is in the ovn-northd log
 
-AT_CHECK([test 1 = `grep -c "SCTP load balancers do not currently support health checks" northd/NORTHD_TYPE.log`])
+cnt1=`grep -c "SCTP load balancers do not currently support health checks" northd/NORTHD_TYPE.log`
+cnt2=`grep -c "SCTP load balancers do not currently support health checks" northd-backup/NORTHD_TYPE.log`
+AT_CHECK([test 1 -le "$(($cnt1 + $cnt2))"])
 
 AT_CLEANUP
 ])