Ensure that server logs are archived.

facebookincubator · Jan 11, 2024 · e9f6b9b · e9f6b9b
1 parent 57d8781
commit e9f6b9b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/experimental.yml b/.github/workflows/experimental.yml
@@ -117,7 +117,10 @@ jobs:
 
       - name: "Run Aggregate Fuzzer"
         run: |
-          /opt/start-prestojava.sh 2>&1 >> /tmp/server.log &
+          /opt/start-prestojava.sh > /tmp/server.log 2>&1 &
+          sleep 60
+          echo 'CREATE SCHEMA hive.tpch;' > /tmp/hive_create.sql
+          /opt/presto-cli --server 127.0.0.1:8080 --file /tmp/hive_create.sql
           mkdir -p /tmp/aggregate_fuzzer_repro/
           rm -rfv /tmp/aggregate_fuzzer_repro/*
           chmod -R 777 /tmp/aggregate_fuzzer_repro
@@ -139,6 +142,7 @@ jobs:
           name: aggregate-fuzzer-failure-artifacts
           path: |
             /tmp/aggregate_fuzzer_repro
+            /tmp/server.log
 
   linux-spark-fuzzer-run:
     runs-on: ubuntu-latest

diff --git a/scripts/start-prestojava.sh b/scripts/start-prestojava.sh
@@ -15,8 +15,5 @@
 
 set -e
 
-nohup $PRESTO_HOME/bin/launcher --pid-file=/tmp/pidfile  run >  /tmp/server.log
-#wait a few seconds for presto to start
-sleep 60
-echo 'CREATE SCHEMA hive.tpch;' > /tmp/hive_create.sql
-/opt/presto-cli --server 127.0.0.1:8080 --file /tmp/hive_create.sql
+$PRESTO_HOME/bin/launcher --pid-file=/tmp/pidfile  run
+