Move backup to its own class

This commit moves the backup cronjob to its own class. This ensures that
backup configuration changes do not trigger a Gitlab service restart.

Fixes #204.

manifests/backup.pp

@@ -0,0 +1,24 @@
+# == Class gitlab::backup
+#
+# This class is called from gitlab for backup config.
+#
+class gitlab::backup {
+  $rake_exec = $::gitlab::rake_exec
+  $backup_cron_enable = $::gitlab::backup_cron_enable
+  $backup_cron_minute = $::gitlab::backup_cron_minute
+  $backup_cron_hour = $::gitlab::backup_cron_hour
+  if empty($::gitlab::backup_cron_skips) {
+    $backup_cron_skips = ''
+  } else {
+    $_backup_cron_skips = join($::gitlab::backup_cron_skips, ',')
+    $backup_cron_skips = "SKIP=${_backup_cron_skips}"
+  }
+
+  if $backup_cron_enable {
+    cron {'gitlab backup':
+      command => "${rake_exec} gitlab:backup:create CRON=1 ${backup_cron_skips}",
+      hour    => $backup_cron_hour,
+      minute  => $backup_cron_minute,
+    }
+  }
+}

manifests/init.pp

@@ -435,10 +435,12 @@
   class { '::gitlab::install': }
   -> class { '::gitlab::config': }
   ~> class { '::gitlab::service': }
+  -> class { '::gitlab::backup': }
 
   contain gitlab::install
   contain gitlab::config
   contain gitlab::service
+  contain gitlab::backup
 
   $custom_hooks.each |$name, $options| {
     gitlab::custom_hook { $name: