Revert "ssh_upload -> hiera role"
authorPeter Palfrader <peter@palfrader.org>
Sun, 15 Sep 2019 15:06:41 +0000 (17:06 +0200)
committerPeter Palfrader <peter@palfrader.org>
Sun, 15 Sep 2019 15:06:41 +0000 (17:06 +0200)
This reverts commit fe6c9108858446d5081a4d3505280bafe018cbdc.

This change is not a nop as expected.  Revert until we figure out why.

hieradata/common.yaml
hieradata/nodes/coccia.debian.org.yaml
hieradata/nodes/suchon.debian.org.yaml [deleted file]
hieradata/nodes/usper.debian.org.yaml
modules/roles/manifests/init.pp

index c63ec9a..2e4b0e6 100644 (file)
@@ -112,6 +112,10 @@ roles:
     - soriano.debian.org
   security_upload:
     - suchon.debian.org
+  ssh.upload.d.o:
+    - coccia.debian.org
+    - suchon.debian.org
+    - usper.debian.org
   sso:
     - diabelli.debian.org
   # single sign on relying party (host) - also required apache2 module enabled on that host via other means
index 17868ed..dfa719b 100644 (file)
@@ -2,5 +2,4 @@
 classes:
   - roles::api_ftp_master
   - roles::ftp_upload
-  - roles::ssh_upload
   - roles::static_source
diff --git a/hieradata/nodes/suchon.debian.org.yaml b/hieradata/nodes/suchon.debian.org.yaml
deleted file mode 100644 (file)
index 49ef5b0..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
----
-classes:
-  - roles::ssh_upload
index 3883d97..7d70f59 100644 (file)
@@ -1,4 +1,3 @@
 ---
 classes:
   - roles::ftp_upload
-  - roles::ssh_upload
index 83ddc6c..f1e24a7 100644 (file)
@@ -34,6 +34,9 @@ class roles {
                include roles::dakmaster
                include roles::signing
        }
+       if has_role('ssh.upload.d.o') {
+               include roles::ssh_upload
+       }
        if has_role('security_upload') {
                include roles::security_upload
        }