From cc2ec1c8b982333afe1fb11c3e12a534f9303744 Mon Sep 17 00:00:00 2001
From: Zhichang Yu <yuzhichang@gmail.com>
Date: Sat, 23 Dec 2023 14:21:09 +0800
Subject: [PATCH] Fix timezone of build container

---
 .github/workflows/nightly.yml |  4 ++-
 .github/workflows/tests.yml   | 55 +++++++++++++++++++++++++++++------
 2 files changed, 49 insertions(+), 10 deletions(-)

diff --git a/.github/workflows/nightly.yml b/.github/workflows/nightly.yml
index 4cf07a01f1..e17ff181de 100644
--- a/.github/workflows/nightly.yml
+++ b/.github/workflows/nightly.yml
@@ -18,7 +18,9 @@ jobs:
       run: git branch --show-current && git rev-parse HEAD
 
     - name: Start builder container
-      run: sudo docker rm -f infinity_build && sudo docker run -d --name infinity_build --network=host -v $PWD:/infinity infiniflow/infinity_build:0.1
+      run: |
+        TZ=$(readlink -f /etc/localtime | awk -F '/zoneinfo/' '{print $2}')
+        sudo docker rm -f infinity_build && sudo docker run -d --name infinity_build --network=host -e TZ=$TZ -v $PWD:/infinity infiniflow/infinity_build:0.1
 
     - name: Build release version
       run: sudo docker exec infinity_build bash -c "cd /infinity && rm -fr cmake-build-release && mkdir -p cmake-build-release && cmake -G Ninja -DCMAKE_BUILD_TYPE=Release -S /infinity -B /infinity/cmake-build-release && cmake --build /infinity/cmake-build-release"
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 9440827630..3465f2dbac 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -18,7 +18,9 @@ jobs:
       uses: actions/checkout@v3
 
     - name: Start builder container
-      run: sudo docker rm -f infinity_build && sudo docker run -d --name infinity_build --network=host -v $PWD:/infinity infiniflow/infinity_build:0.1
+      run: |
+        TZ=$(readlink -f /etc/localtime | awk -F '/zoneinfo/' '{print $2}')
+        sudo docker rm -f infinity_build && sudo docker run -d --name infinity_build --network=host -e TZ=$TZ -v $PWD:/infinity infiniflow/infinity_build:0.1
     
     - name: Build debug version
       run: sudo docker exec infinity_build bash -c "cd /infinity && rm -fr cmake-build-debug && mkdir -p cmake-build-debug && cmake -G Ninja -DCMAKE_BUILD_TYPE=Debug -S /infinity -B /infinity/cmake-build-debug && cmake --build /infinity/cmake-build-debug"
@@ -30,14 +32,31 @@ jobs:
       run: |
         # Run a command in the background
         sudo docker exec infinity_build bash -c "cd /infinity/ && rm -fr /tmp/infinity && cmake-build-debug/src/infinity > debug.log 2>&1" &
-        # Save its process ID to a file for later use
-        echo $! > background_process_pid.txt
 
     - name: pysdk & sqllogictest debug version
       run: sudo docker exec infinity_build bash -c "cd /infinity/ && python3 tools/sqllogictest.py"
 
-    - name: Collect infinity debug output and stop it
-      run: cat debug.log && sudo kill $(cat background_process_pid.txt) && rm -f background_process_pid.txt
+    - name: Stop infinity debug
+      run: |
+        pid=$(pidof cmake-build-debug/src/infinity)
+        if [ -z "$pid" ]; then
+          echo "inifnity is already dead"
+          exit 1  # Set the step to fail
+        fi
+        sudo kill $pid 2>/dev/null
+        sleep 10
+        sudo kill -0 $pid 2>/dev/null
+        if [ $? -eq 0 ]; then
+          echo "Process with PID $pid is alive."
+          exit 1  # Set the step to fail
+        else
+          echo "Process with PID $pid is not alive."
+        fi
+      if: always() # always run this step even if previous steps failed
+
+    - name: Collect infinity debug output
+      # GitHub Actions interprets output lines starting with "Error" as error messages, and it automatically sets the step status to failed when such lines are detected. 
+      run: cat debug.log 2>/dev/null || true
       if: always() # always run this step even if previous steps failed
 
     - name: Build release version
@@ -45,12 +64,30 @@ jobs:
 
     - name: Start infinity release version
       run: |
-        sudo docker exec infinity_build bash -c "cd /infinity/ &&rm -fr /tmp/infinity && cmake-build-release/src/infinity > release.log 2>&1" &
-        echo $! > background_process_pid.txt
+        sudo docker exec infinity_build bash -c "cd /infinity/ && rm -fr /tmp/infinity && cmake-build-release/src/infinity > release.log 2>&1" &
 
     - name: pysdk & sqllogictest release version
       run: sudo docker exec infinity_build bash -c "cd /infinity/ && python3 tools/sqllogictest.py"
 
-    - name: Collect infinity release output and stop it
-      run: cat release.log && sudo kill $(cat background_process_pid.txt) && rm -f background_process_pid.txt
+    - name: Stop infinity release
+      run: |
+        pid=$(pidof cmake-build-release/src/infinity)
+        if [ -z "$pid" ]; then
+          echo "inifnity is already dead"
+          exit 1  # Set the step to fail
+        fi
+        sudo kill $pid 2>/dev/null
+        sleep 10
+        sudo kill -0 $pid 2>/dev/null
+        if [ $? -eq 0 ]; then
+          echo "Process with PID $pid is alive."
+          exit 1  # Set the step to fail
+        else
+          echo "Process with PID $pid is not alive."
+        fi
+      if: always() # always run this step even if previous steps failed
+
+    - name: Collect infinity release output
+      # GitHub Actions interprets output lines starting with "Error" as error messages, and it automatically sets the step status to failed when such lines are detected. 
+      run: cat release.log 2>/dev/null || true
       if: always() # always run this step even if previous steps failed