From 90df1e935d008b368ae6626b11d9698529bebccb Mon Sep 17 00:00:00 2001
From: "sewon.oh" <sewon.oh@samsung.com>
Date: Tue, 6 Nov 2018 18:06:53 +0900
Subject: [PATCH] [TEST] Change time out logic

To remove overhead, change the logic

Signed-off-by: sewon.oh <sewon.oh@samsung.com>
---
 packaging/nnstreamer.spec | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/packaging/nnstreamer.spec b/packaging/nnstreamer.spec
index 0935f9c..5fe1143 100644
--- a/packaging/nnstreamer.spec
+++ b/packaging/nnstreamer.spec
@@ -97,25 +97,25 @@ make %{?_smp_mflags}
 ./unittest_sink --gst-plugin-path=.
 popd
 pushd tests
-# The 'testAll.sh' script requires 6~7min to run armv7l binary files in the current CI server.
+# The ssat requires 6~7min to run armv7l binary files in the current CI server.
 # The timeout value is 10min as a heuristic value from our experience.
 timeout=600
-interval=60
 ssat &
 pid=$!
-while ((timeout > 0)); do
-    sleep $interval
-    if [[ ! $(ps | grep "$pid") ]]; then
-        break;
-    fi
-    (( timeout -= interval ))
-done
-if [[ $(ps | grep "$pid") ]]; then
-    kill $pid
-    echo "GBS is stopped because of './testAll.sh' timeout(10min)"
+# CAUTION: Note that you have to keep the coding style of the existing statement 
+# in case that you have to update the below statement in the future. 
+# a. Do not run repetitive statement(ex. while) to avoid too log messages in the log file. 
+# b. Declare appropriate heuristic timeout value
+# c. Do not declare too long sleep time to keep the reasonable waiting time after submitting PR
+(sleep $timeout 
+kill $pid 
+if [[ "$?" -eq 0 ]]; then
+    echo "[DEBUG] GBS is stopped because of 'ssat' timeout(10min)"
     exit 124 # 124 is ubuntu status code of timeout
-fi
+fi) &
+pid2=$!
 wait $pid
+kill $pid2
 popd
 
 %install
-- 
2.7.4