This is an automated email from the ASF dual-hosted git repository.

snuyanzin pushed a commit to branch release-1.20
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/release-1.20 by this push:
     new 657c6222aac [FLINK-40451][ci] E2E test timeout keeps CI running until 
the timeout ends, even after the test passes
657c6222aac is described below

commit 657c6222aace702a229211238d01fb57fda5eb25
Author: Sergey Nuyanzin <[email protected]>
AuthorDate: Thu Sep 3 19:13:55 2026 +0200

    [FLINK-40451][ci] E2E test timeout keeps CI running until the timeout ends, 
even after the test passes
---
 flink-end-to-end-tests/test-scripts/common.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/flink-end-to-end-tests/test-scripts/common.sh 
b/flink-end-to-end-tests/test-scripts/common.sh
index 9a69c468e00..f65973e822b 100755
--- a/flink-end-to-end-tests/test-scripts/common.sh
+++ b/flink-end-to-end-tests/test-scripts/common.sh
@@ -974,7 +974,11 @@ internal_run_with_timeout() {
 
   (
       command_pid=$BASHPID
-      (sleep "${timeout_in_seconds}" # set a timeout for this command
+      (
+      # Kill the sleep on teardown so it doesn't keep the CI stdout pipe open 
until timeout.
+      trap 'kill $sleep_pid 2>/dev/null; exit 0' TERM
+      sleep "${timeout_in_seconds}" & sleep_pid=$! # set a timeout for this 
command
+      wait $sleep_pid
       echo "${command_label:-"The command '${command}'"} (pid: $command_pid) 
did not finish after $timeout_in_seconds seconds."
       eval "${on_failure}"
       kill "$command_pid") & watchdog_pid=$!

Reply via email to