From d4807d8033e6c048de2e6cdbe562dc791ecc92e8 Mon Sep 17 00:00:00 2001
From: Peter Palfrader <peter@palfrader.org>
Date: Sun, 22 Sep 2019 21:18:12 +0200
Subject: [PATCH] avoid duplicate ssh keygen for snapshot

---
 modules/roles/manifests/snapshot_master.pp     | 2 +-
 modules/roles/manifests/snapshot_secondary.pp  | 1 +
 modules/roles/manifests/snapshot_ssh_keygen.pp | 4 ++++
 3 files changed, 6 insertions(+), 1 deletion(-)
 create mode 100644 modules/roles/manifests/snapshot_ssh_keygen.pp

diff --git a/modules/roles/manifests/snapshot_master.pp b/modules/roles/manifests/snapshot_master.pp
index eeb74f7d9..e976cdc44 100644
--- a/modules/roles/manifests/snapshot_master.pp
+++ b/modules/roles/manifests/snapshot_master.pp
@@ -1,8 +1,8 @@
 # snapshot master
 class roles::snapshot_master {
   include roles::snapshot_base
+  include roles::snapshot_ssh_keygen
 
-  ssh::keygen {'snapshot': }
   ssh::authorized_key_add { 'roles::snapshot_master::to::farmsync_target':
     target_user => 'snapshot',
     key         => dig($facts, 'ssh_keys_users', 'snapshot', 'id_rsa.pub', 'line'),
diff --git a/modules/roles/manifests/snapshot_secondary.pp b/modules/roles/manifests/snapshot_secondary.pp
index f4ef77da0..e53c267bc 100644
--- a/modules/roles/manifests/snapshot_secondary.pp
+++ b/modules/roles/manifests/snapshot_secondary.pp
@@ -4,6 +4,7 @@
 # like sync targets or web mirrors.
 class roles::snapshot_secondary {
   include roles::snapshot_base
+  include roles::snapshot_ssh_keygen
 
   ssh::keygen {'snapshot': }
 
diff --git a/modules/roles/manifests/snapshot_ssh_keygen.pp b/modules/roles/manifests/snapshot_ssh_keygen.pp
new file mode 100644
index 000000000..d9b80cc99
--- /dev/null
+++ b/modules/roles/manifests/snapshot_ssh_keygen.pp
@@ -0,0 +1,4 @@
+# create the ssh key for snapshot
+class roles::snapshot_ssh_keygen {
+  ssh::keygen {'snapshot': }
+}
-- 
2.20.1