From c32de12ceedfe47bc394c895f0ad9b7261978a5c Mon Sep 17 00:00:00 2001 From: ionutboangiu Date: Wed, 29 May 2024 19:20:19 +0300 Subject: [PATCH] Reduce client connection attempts to test engine status (tests) 200 -> 16 Previously, considering the fibonacci back off strategy, if engine failed to start it would have tried to attempt connections for 2.862799e+33 years. Now, the amount of attempts is 16, equivalent to around 2 and a half seconds. Should be more than enough, considering that usually 6-7 attempts are enough. --- engine/libtest.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/engine/libtest.go b/engine/libtest.go index 0d5126ca6..c210040bf 100644 --- a/engine/libtest.go +++ b/engine/libtest.go @@ -346,7 +346,7 @@ func StartEngine(cfgPath string, waitEngine int) (*exec.Cmd, error) { } fib := utils.FibDuration(time.Millisecond, 0) var connected bool - for i := 0; i < 200; i++ { + for i := 0; i < 16; i++ { time.Sleep(fib()) if _, err := jsonrpc.Dial(utils.TCP, cfg.ListenCfg().RPCJSONListen); err != nil { utils.Logger.Warning(fmt.Sprintf("Error <%s> when opening test connection to: <%s>", @@ -374,7 +374,7 @@ func StartEngineWithContext(ctx context.Context, cfgPath string, waitEngine int) return } fib := utils.FibDuration(time.Millisecond, 0) - for i := 0; i < 200; i++ { + for i := 0; i < 16; i++ { time.Sleep(fib()) if _, err = jsonrpc.Dial(utils.TCP, cfg.ListenCfg().RPCJSONListen); err != nil { continue