From 31b8ef0e82f28c24d54b009e8f662d2fd0e8654f Mon Sep 17 00:00:00 2001 From: jymaire Date: Mon, 6 Jul 2026 09:40:49 +0200 Subject: [PATCH] test(scripts): raise ExecuteFlow timeout to 5m to fix cold-start flakes The @ExecuteFlow await defaults to 60s. The all_*.yaml sanity-check flows run script tasks that compile/build inside Docker; on a cold CI runner, image pull plus compilation can exceed 60s, leaving the execution RUNNING at the deadline and failing parameter resolution (seen on plugin-script-go). Bump the timeout to PT5M on the heavier runner tests (go, dotnet, deno, bun, php) for headroom while still failing fast on a genuine hang. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../src/test/java/io/kestra/plugins/scripts/bun/RunnerTest.java | 2 +- .../test/java/io/kestra/plugins/scripts/deno/RunnerTest.java | 2 +- .../test/java/io/kestra/plugin/scripts/dotnet/RunnerTest.java | 2 +- .../src/test/java/io/kestra/plugins/scripts/go/RunnerTest.java | 2 +- .../src/test/java/io/kestra/plugin/scripts/php/RunnerTest.java | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/plugin-script-bun/src/test/java/io/kestra/plugins/scripts/bun/RunnerTest.java b/plugin-script-bun/src/test/java/io/kestra/plugins/scripts/bun/RunnerTest.java index efff81da..58e8f10b 100644 --- a/plugin-script-bun/src/test/java/io/kestra/plugins/scripts/bun/RunnerTest.java +++ b/plugin-script-bun/src/test/java/io/kestra/plugins/scripts/bun/RunnerTest.java @@ -14,7 +14,7 @@ @KestraTest(startRunner = true) class RunnerTest { @Test - @ExecuteFlow("sanity-checks/all_bun.yaml") + @ExecuteFlow(value = "sanity-checks/all_bun.yaml", timeout = "PT5M") void all_bun(Execution execution) { assertThat(execution.getTaskRunList(), hasSize(2)); assertThat(execution.getState().getCurrent(), is(State.Type.SUCCESS)); diff --git a/plugin-script-deno/src/test/java/io/kestra/plugins/scripts/deno/RunnerTest.java b/plugin-script-deno/src/test/java/io/kestra/plugins/scripts/deno/RunnerTest.java index ec513d6a..0883eb4d 100644 --- a/plugin-script-deno/src/test/java/io/kestra/plugins/scripts/deno/RunnerTest.java +++ b/plugin-script-deno/src/test/java/io/kestra/plugins/scripts/deno/RunnerTest.java @@ -14,7 +14,7 @@ @KestraTest(startRunner = true) class RunnerTest { @Test - @ExecuteFlow("sanity-checks/all_deno.yaml") + @ExecuteFlow(value = "sanity-checks/all_deno.yaml", timeout = "PT5M") void all_deno(Execution execution) { assertThat(execution.getTaskRunList(), hasSize(2)); assertThat(execution.getState().getCurrent(), is(State.Type.SUCCESS)); diff --git a/plugin-script-dotnet/src/test/java/io/kestra/plugin/scripts/dotnet/RunnerTest.java b/plugin-script-dotnet/src/test/java/io/kestra/plugin/scripts/dotnet/RunnerTest.java index 34750f8e..4ef9cbe4 100644 --- a/plugin-script-dotnet/src/test/java/io/kestra/plugin/scripts/dotnet/RunnerTest.java +++ b/plugin-script-dotnet/src/test/java/io/kestra/plugin/scripts/dotnet/RunnerTest.java @@ -14,7 +14,7 @@ @KestraTest(startRunner = true) class RunnerTest { @Test - @ExecuteFlow("sanity-checks/all_dotnet.yaml") + @ExecuteFlow(value = "sanity-checks/all_dotnet.yaml", timeout = "PT5M") void all_dotnet(Execution execution) { assertThat(execution.getTaskRunList(), hasSize(2)); assertThat(execution.getState().getCurrent(), is(State.Type.SUCCESS)); diff --git a/plugin-script-go/src/test/java/io/kestra/plugins/scripts/go/RunnerTest.java b/plugin-script-go/src/test/java/io/kestra/plugins/scripts/go/RunnerTest.java index c85b705f..fe4b6ac3 100644 --- a/plugin-script-go/src/test/java/io/kestra/plugins/scripts/go/RunnerTest.java +++ b/plugin-script-go/src/test/java/io/kestra/plugins/scripts/go/RunnerTest.java @@ -14,7 +14,7 @@ @KestraTest(startRunner = true) class RunnerTest { @Test - @ExecuteFlow("sanity-checks/all_go.yaml") + @ExecuteFlow(value = "sanity-checks/all_go.yaml", timeout = "PT5M") void all_go(Execution execution) { assertThat(execution.getTaskRunList(), hasSize(2)); assertThat(execution.getState().getCurrent(), is(State.Type.SUCCESS)); diff --git a/plugin-script-php/src/test/java/io/kestra/plugin/scripts/php/RunnerTest.java b/plugin-script-php/src/test/java/io/kestra/plugin/scripts/php/RunnerTest.java index f13cb555..47f41360 100644 --- a/plugin-script-php/src/test/java/io/kestra/plugin/scripts/php/RunnerTest.java +++ b/plugin-script-php/src/test/java/io/kestra/plugin/scripts/php/RunnerTest.java @@ -14,7 +14,7 @@ @KestraTest(startRunner = true) class RunnerTest { @Test - @ExecuteFlow("sanity-checks/all_php.yaml") + @ExecuteFlow(value = "sanity-checks/all_php.yaml", timeout = "PT5M") void all_php(Execution execution) { assertThat(execution.getTaskRunList(), hasSize(2)); assertThat(execution.getState().getCurrent(), is(State.Type.SUCCESS));