From: Frantisek Sumsal <frantisek@sumsal.cz>
Date: Mon, 1 Jul 2019 07:27:59 +0000 (+0200)
Subject: test: replace `tail -f` with journal cursor which should be...
X-Git-Tag: v243-rc2~63^2~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d0b2178f3e79f302702bd7140766eee03643f734;p=thirdparty%2Fsystemd.git

test: replace `tail -f` with journal cursor which should be...

more reliable
---

diff --git a/test/TEST-36-NUMAPOLICY/testsuite.sh b/test/TEST-36-NUMAPOLICY/testsuite.sh
index e15087b137a..306a96b5170 100755
--- a/test/TEST-36-NUMAPOLICY/testsuite.sh
+++ b/test/TEST-36-NUMAPOLICY/testsuite.sh
@@ -29,6 +29,9 @@ testUnitNUMAConf="$testUnitFile.d/numa.conf"
 journalSleep=5
 sleepAfterStart=1
 
+# Journal cursor for easier navigation
+journalCursorFile="jounalCursorFile"
+
 startStrace() {
     coproc strace -qq -p 1 -o $straceLog -e set_mempolicy -s 1024 $1
 }
@@ -38,18 +41,16 @@ stopStrace() {
 }
 
 startJournalctl() {
-    coproc journalctl -u init.scope -f > $journalLog
+    # Save journal's cursor for later navigation
+    journalctl --no-pager --cursor-file="$journalCursorFile" -n0 -ocat
 }
 
 stopJournalctl() {
-    # Wait a few seconds until the messages get properly queued...
-    sleep $journalSleep
-    # ...and then force journald to write them to the backing storage
-    # Also, using journalctl --sync should be better than using SIGRTMIN+1, as
+    # Using journalctl --sync should be better than using SIGRTMIN+1, as
     # the --sync wait until the synchronization is complete
     echo "Force journald to write all queued messages"
     journalctl --sync
-    kill -s TERM $COPROC_PID
+    journalctl -u init.scope --cursor-file="$journalCursorFile" > "$journalLog"
 }
 
 checkNUMA() {