Merge pull request #269 from github/snh/stop-services-2

Stop cron and timerd during restore #2

Author: rubiojr

bin/ghe-restore

@@ -200,6 +200,28 @@ fi
 ghe-backup-store-version  ||
 echo "Warning: storing backup-utils version remotely failed."
 
+# Stop cron and timerd, as scheduled jobs may disrupt the restore process.
+if [ "$GHE_VERSION_MAJOR" -ge 2 ]; then
+  echo "Stopping cron and github-timerd ..."
+  if $cluster; then
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "ghe-cluster-each -- sudo service cron stop"; then
+      ghe_verbose "* Warning: Failed to stop cron on one or more nodes"
+    fi
+
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "ghe-cluster-each -- sudo service github-timerd stop"; then
+      ghe_verbose "* Warning: Failed to stop github-timerd on one or more nodes"
+    fi
+  else
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "sudo service cron stop"; then
+      ghe_verbose "* Warning: Failed to stop cron"
+    fi
+
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "sudo service github-timerd stop"; then
+      ghe_verbose "* Warning: Failed to stop github-timerd"
+    fi
+  fi
+fi
+
 # Restore settings and license if restoring to an unconfigured appliance or when
 # specified manually.
 if $restore_settings; then
@@ -323,6 +345,20 @@ elif $instance_configured; then
   fi
 fi
 
+# Start cron. Timerd will start automatically as part of the config run.
+if [ "$GHE_VERSION_MAJOR" -ge 2 ]; then
+  echo "Starting cron ..."
+  if $cluster; then
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "ghe-cluster-each -- sudo service cron start"; then
+      echo "* Warning: Failed to start cron on one or more nodes"
+    fi
+  else
+    if ! ghe-ssh "$GHE_HOSTNAME" -- "sudo service cron start"; then
+      echo "* Warning: Failed to start cron"
+    fi
+  fi
+fi
+
 # Update the remote status to "complete". This has to happen before importing
 # ssh host keys because subsequent commands will fail due to the host key
 # changing otherwise.

test/bin/service

@@ -0,0 +1,3 @@
+#!/usr/bin/env bash
+# Fake service command for tests.
+true