From 9e758f3961ae8e202c90a67fb8ef01e1d662474d Mon Sep 17 00:00:00 2001
From: Peter Palfrader <peter@palfrader.org>
Date: Sat, 21 Sep 2019 12:26:20 +0200
Subject: [PATCH] minor naming fixes

---
 modules/roles/manifests/ftp_master.pp      | 6 +++---
 modules/roles/manifests/ports_master.pp    | 4 ++--
 modules/roles/manifests/security_master.pp | 6 +++---
 modules/roles/manifests/syncproxy.pp       | 6 +++---
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/modules/roles/manifests/ftp_master.pp b/modules/roles/manifests/ftp_master.pp
index f8d6f877e..10b5bb6f1 100644
--- a/modules/roles/manifests/ftp_master.pp
+++ b/modules/roles/manifests/ftp_master.pp
@@ -13,9 +13,9 @@ class roles::ftp_master {
   }
 
   # export ssh allow rules for hosts that we should be able to access
-  @@ferm::rule::simple { "dsa-ssh-from-syncproxy-${::fqdn}":
-    tag         => 'ssh::server::allow::ftp-master',
-    description => 'Allow ssh access from ftp-master',
+  @@ferm::rule::simple { "dsa-ssh-from-ftp_master-${::fqdn}":
+    tag         => 'ssh::server::allow::ftp_master',
+    description => 'Allow ssh access from ftp_master',
     port        => '22',
     saddr       => $base::public_addresses,
   }
diff --git a/modules/roles/manifests/ports_master.pp b/modules/roles/manifests/ports_master.pp
index 31c650303..5dc609d04 100644
--- a/modules/roles/manifests/ports_master.pp
+++ b/modules/roles/manifests/ports_master.pp
@@ -20,8 +20,8 @@ class roles::ports_master {
   }
 
   # export ssh allow rules for hosts that we should be able to access
-  @@ferm::rule::simple { "dsa-ssh-from-syncproxy-${::fqdn}":
-    tag         => 'ssh::server::allow::ports-master',
+  @@ferm::rule::simple { "dsa-ssh-from-ports_master-${::fqdn}":
+    tag         => 'ssh::server::allow::ports_master',
     description => 'Allow ssh access from ports-master',
     port        => '22',
     saddr       => $base::public_addresses,
diff --git a/modules/roles/manifests/security_master.pp b/modules/roles/manifests/security_master.pp
index 6f92d33a2..702e2e2ae 100644
--- a/modules/roles/manifests/security_master.pp
+++ b/modules/roles/manifests/security_master.pp
@@ -13,9 +13,9 @@ class roles::security_master {
   }
 
   # export ssh allow rules for hosts that we should be able to access
-  @@ferm::rule::simple { "dsa-ssh-from-syncproxy-${::fqdn}":
-    tag         => 'ssh::server::allow::security-master',
-    description => 'Allow ssh access from security-master',
+  @@ferm::rule::simple { "dsa-ssh-from-security_master-${::fqdn}":
+    tag         => 'ssh::server::allow::security_master',
+    description => 'Allow ssh access from security_master',
     port        => '22',
     saddr       => $base::public_addresses,
   }
diff --git a/modules/roles/manifests/syncproxy.pp b/modules/roles/manifests/syncproxy.pp
index 26e1c5a31..623393ffc 100644
--- a/modules/roles/manifests/syncproxy.pp
+++ b/modules/roles/manifests/syncproxy.pp
@@ -78,8 +78,8 @@ class roles::syncproxy {
   # syncproxies should be accessible from various role hosts
   Ferm::Rule::Simple <<|
     tag == 'ssh::server::allow::archvsync' or
-    tag == 'ssh::server::allow::ftp-master' or
-    tag == 'ssh::server::allow::ports-master' or
-    tag == 'ssh::server::allow::security-master'
+    tag == 'ssh::server::allow::ftp_master' or
+    tag == 'ssh::server::allow::ports_master' or
+    tag == 'ssh::server::allow::security_master'
     |>>
 }
-- 
2.20.1