treehouses container refactor (fixes #2055) (#2062)

Co-authored-by: dogi <dogi@users.noreply.github.com>

Author: rjpadilla

modules/globals.sh

@@ -1,32 +1,29 @@
 function start_service {
-  if [ "$(systemctl is-active "$1" 2>"$LOGFILE")" = "inactive" ]
-  then
-    systemctl start "$1" >"$LOGFILE" 2>"$LOGFILE"
+  if [ "$(systemctl is-active "$1" 2>"$LOGFILE")" = "inactive" ]; then
+    systemctl start "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to start service $1"
   fi
 }
 
 function restart_service {
-  systemctl stop "$1" >"$LOGFILE" 2>"$LOGFILE"
-  systemctl start "$1" >"$LOGFILE" 2>"$LOGFILE"
+  systemctl stop "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to stop service $1"
+  systemctl start "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to start service $1"
 }
 
 function stop_service {
-  if [ "$(systemctl is-active "$1" 2>"$LOGFILE")" = "active" ]
-  then
-    systemctl stop "$1" >"$LOGFILE" 2>"$LOGFILE"
+  if [ "$(systemctl is-active "$1" 2>"$LOGFILE")" = "active" ]; then
+    systemctl stop "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to stop service $1"
   fi
 }
 
 function enable_service {
-  if [ "$(systemctl is-enabled "$1" 2>"$LOGFILE")" = "disabled" ]
-  then
-    systemctl enable "$1" >"$LOGFILE" 2>"$LOGFILE"
+  if [ "$(systemctl is-enabled "$1" 2>"$LOGFILE")" = "disabled" ]; then
+    systemctl enable "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to enable service $1"
   fi
 }
 
 function disable_service {
   if [ "$(systemctl is-enabled "$1" 2>"$LOGFILE")" = "enabled" ]; then
-    systemctl disable "$1" >"$LOGFILE" 2>"$LOGFILE"
+    systemctl disable "$1" >"$LOGFILE" 2>"$LOGFILE" || log_and_exit1 "Failed to disable service $1"
   fi
 }
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@treehouses/cli",
-  "version": "1.25.20",
+  "version": "1.25.21",
   "remote": "4000",
   "description": "Thin command-line interface for Raspberry Pi low level configuration.",
   "main": "cli.sh",