From 22ca9e496d21e2a2b88cb149b4e8123375697518 Mon Sep 17 00:00:00 2001 From: Justin Date: Mon, 12 Feb 2024 17:36:32 -0500 Subject: [PATCH] fix: increase event timeout (#118) Issue #, if available: Still seeing failures: https://github.com/runfinch/finch/actions/runs/7820846068/job/21371714848?pr=795#step:10:6233 *Description of changes:* - Double event timeout *Testing done:* - The same tests currently runs twice, with different arguments. Sometimes, one of these times succeeds in just under 15 seconds, so increasing the timeout to over 15s may make the test more reliable - https://github.com/runfinch/finch/actions/runs/7820846068/job/21371714848?pr=795#step:10:6184 - [x] I've reviewed the guidance in CONTRIBUTING.md #### License Acceptance By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. Signed-off-by: Justin Alvarez --- tests/logs.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/logs.go b/tests/logs.go index 62d72b5..e874b3e 100644 --- a/tests/logs.go +++ b/tests/logs.go @@ -101,7 +101,7 @@ func Logs(o *option.Option) { command.Run(o, "exec", testContainerName, "sh", "-c", fmt.Sprintf("echo %s >> /proc/1/fd/1", newLog)) // allow propagation time gomega.Eventually(strings.TrimSpace(string(session.Out.Contents()))). - WithTimeout(15 * time.Second). + WithTimeout(30 * time.Second). WithPolling(1 * time.Second). Should(gomega.Equal(newLog)) })