From 21d048299ecdca7ec28dbc449c52fc2d46f985b1 Mon Sep 17 00:00:00 2001
From: leej3 <“johnleenimh@gmail.com>
Date: Thu, 9 May 2024 14:24:44 +0000
Subject: [PATCH] kill whole process group when retrying

---
 .github/workflows/unit-tests.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml
index fe7c8e55b26d..b71e89046b9f 100644
--- a/.github/workflows/unit-tests.yml
+++ b/.github/workflows/unit-tests.yml
@@ -127,9 +127,10 @@ jobs:
           shell: bash
           command: SKIP_DISTRIB_TESTS=${{ matrix.skip-distrib-tests }} bash tests/run_cpu_tests.sh
           on_retry_command: |
+            echo sending kill signal until process group no longer exists...
             tests_pid=$(cat /tmp/ignite_testing.pid)
-            kill -0 $tests_pid && kill -INT $tests_pid && sleep 5
-            while kill -0 $tests_pid > /dev/null 2>&1; do kill -9 $tests_pid; sleep 3; done
+            kill -0 -- -$tests_pid && kill -INT -- -$tests_pid && sleep 5
+            while kill -0 -- -$tests_pid > /dev/null 2>&1; do kill -9 -- -$tests_pid; sleep 3; done
           new_command_on_retry: USE_LAST_FAILED=1  SKIP_DISTRIB_TESTS=${{ matrix.skip-distrib-tests }} bash tests/run_cpu_tests.sh
 
       - name: Upload coverage to Codecov