From 4542016cbe985d2af60c25f6a5b24df50bb50aba Mon Sep 17 00:00:00 2001 From: Dennis Chen Date: Fri, 12 Jan 2018 11:52:35 +0000 Subject: [PATCH] Fix timeout issue of multi-services cration on AArch64 Now we only adjust the timeout value for `arm` while not `arm64`, actually the avarage duration for this test is about 25s to crate multiple services on arm64, else the integration test will terminate with below error: > --- FAIL: TestCreateServiceMultipleTimes (24.11s) > daemon.go:285: [ddc3c7c1476c2] waiting for daemon to start > daemon.go:317: [ddc3c7c1476c2] daemon started > poll.go:121: timeout hit after 10s: task count at 4 waiting for 0 > daemon.go:275: [ddc3c7c1476c2] exiting daemon > clean.go:108: Removing image sha256:e6a8d12d58602a19277ee5632b7ff9fa56a4ea52ba00eedf1d3f6f5a495fe761 > clean.go:108: Removing image sha256:876244cc2ecb8fe1b0b2e817e3b78709a2a735edb093bc6849f99aa6c18f3a01 This PR adjusts the timeout value for both `arm64` and `arm` to mitigate this issue on those 2 platforms. Signed-off-by: Dennis Chen --- integration/service/create_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/integration/service/create_test.go b/integration/service/create_test.go index a4373721f5..7e78b94992 100644 --- a/integration/service/create_test.go +++ b/integration/service/create_test.go @@ -43,7 +43,8 @@ func TestCreateServiceMultipleTimes(t *testing.T) { require.NoError(t, err) pollSettings := func(config *poll.Settings) { - if runtime.GOARCH == "arm" { + // It takes about ~25s to finish the multi services creation in this case per the pratical observation on arm64/arm platform + if runtime.GOARCH == "arm64" || runtime.GOARCH == "arm" { config.Timeout = 30 * time.Second config.Delay = 100 * time.Millisecond }