Merge pull request #18 from efrecon/feature/one-removal

Log cleanup

Author: efrecon

lib/common.sh

@@ -206,7 +206,7 @@ sublog() {
   verbose "$1 now present on disk"
 
   # Then reroute its content through our logging printf style
-  tail -n +0 -f "$1" | while IFS= read -r line; do
+  tail -n +0 -f "$1" 2>/dev/null | while IFS= read -r line; do
     if [ -n "$line" ]; then
       printf '[%s] [%s] %s\n' \
         "${2:-}@${KRUNVM_RUNNER_BIN:-$(basename "$0")}" \

orchestrator.sh

@@ -93,7 +93,7 @@ cleanup() {
 
   for pid in $ORCHESTRATOR_PIDS; do
     verbose "Killing runner loop $pid"
-    kill "$pid"
+    kill "$pid" 2>/dev/null || true
   done
   verbose "Waiting for runners to die"
   # shellcheck disable=SC2086 # We want to wait for all pids

runner/runner.sh

@@ -236,7 +236,11 @@ runner_unregister() {
   trap - INT TERM EXIT
 
   # Remember or request (again) a runner registration token
-  verbose "Caught termination signal, unregistering runner"
+  if [ "${1:-0}" = "1" ]; then
+    verbose "Caught termination signal/request, unregistering runner"
+  else
+    verbose "Caught regular exit signal, unregistering runner"
+  fi
   if [ -n "${RUNNER_PAT:-}" ]; then
     if [ -n "${RUNNER_TOKENFILE:-}" ] && [ -f "$RUNNER_TOKENFILE" ]; then
       verbose "Reading runner token from $RUNNER_TOKENFILE"