Turn apache2_security_mirror into a security_mirror role

diff --git a/hieradata/common.yaml b/hieradata/common.yaml
index 75a759a..ad6ccb1 100644 (file)
--- a/hieradata/common.yaml
+++ b/hieradata/common.yaml
@@ -110,6 +110,18 @@ roles:
     - wolkenstein.debian.org
   search_frontend:
     - cgi-grnet-01.debian.org
+  security_mirror:
+    - mirror-anu.debian.org
+    - mirror-csail.debian.org
+    - mirror-isc.debian.org
+    - mirror-umn.debian.org
+    - lobos.debian.org
+    - santoro.debian.org
+    - setoguchi.debian.org
+    - sechter.debian.org
+    - steffani.debian.org
+    - villa.debian.org
+    - wieck.debian.org
   ssh.upload.d.o:
     - coccia.debian.org
   sso:

diff --git a/modules/debian-org/misc/local.yaml b/modules/debian-org/misc/local.yaml
index 008ffd3..7795a69 100644 (file)
--- a/modules/debian-org/misc/local.yaml
+++ b/modules/debian-org/misc/local.yaml
@@ -141,18 +141,6 @@ host_settings:
     - tye.debian.org
     - vento.debian.org
     - wuiet.debian.org
-  apache2_security_mirror:
-    - mirror-anu.debian.org
-    - mirror-csail.debian.org
-    - mirror-isc.debian.org
-    - mirror-umn.debian.org
-    - lobos.debian.org
-    - santoro.debian.org
-    - setoguchi.debian.org
-    - sechter.debian.org
-    - steffani.debian.org
-    - villa.debian.org
-    - wieck.debian.org
   not-bacula-client:
     # decommissioning
     - rautavaara.debian.org

diff --git a/modules/roles/manifests/init.pp b/modules/roles/manifests/init.pp
index 2cd90de..a6d6647 100644 (file)
--- a/modules/roles/manifests/init.pp
+++ b/modules/roles/manifests/init.pp
@@ -71,11 +71,9 @@ class roles {
                }
        }
 
-       # XXX: turn this into a real role
-       if getfromhash($site::nodeinfo, 'apache2_security_mirror') {
+       if has_role('security_mirror') {
                include roles::security_mirror
        }
-
        if has_role('mirror_debug') {
                include roles::debug_mirror
        }