ci: fix flaky test "update memory vs CheckBeforeUpdate"

kolyshkin · kolyshkin · commit f88a7654601a · 2023-08-02T23:00:46.000-07:00
This test fails in CI sometimes with the following error:

&gt; `testcontainer test_update stopped' failed

Give OOM killer some time to do its job.

Signed-off-by: Kir Kolyshkin &lt;kolyshkin@gmail.com&gt;
diff --git a/tests/integration/update.bats b/tests/integration/update.bats
@@ -854,5 +854,5 @@ EOF
 	# The container will be OOM killed, and runc might either succeed
 	# or fail depending on the timing, so we don't check its exit code.
 	runc update test_update --memory 1024
-	testcontainer test_update stopped
+	wait_for_container 10 1 test_update stopped
 }

Original file line number	Diff line number	Diff line change
`@@ -854,5 +854,5 @@ EOF`
`854`	`854`	`# The container will be OOM killed, and runc might either succeed`
`855`	`855`	`# or fail depending on the timing, so we don't check its exit code.`
`856`	`856`	`runc update test_update --memory 1024`
`857`		`- testcontainer test_update stopped`
	`857`	`+ wait_for_container 10 1 test_update stopped`
`858`	`858`	`}`