So now we have ssh::server::from and ssh::server::to, hopefully making it more clear

diff --git a/modules/roles/manifests/ftp_master.pp b/modules/roles/manifests/ftp_master.pp
index 10b5bb6..c733479 100644 (file)
--- a/modules/roles/manifests/ftp_master.pp
+++ b/modules/roles/manifests/ftp_master.pp
@@ -14,7 +14,7 @@ class roles::ftp_master {
 
   # export ssh allow rules for hosts that we should be able to access
   @@ferm::rule::simple { "dsa-ssh-from-ftp_master-${::fqdn}":
-    tag         => 'ssh::server::allow::ftp_master',
+    tag         => 'ssh::server::from::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 5dc609d..ffc24c3 100644 (file)
--- a/modules/roles/manifests/ports_master.pp
+++ b/modules/roles/manifests/ports_master.pp
@@ -21,7 +21,7 @@ class roles::ports_master {
 
   # export ssh allow rules for hosts that we should be able to access
   @@ferm::rule::simple { "dsa-ssh-from-ports_master-${::fqdn}":
-    tag         => 'ssh::server::allow::ports_master',
+    tag         => 'ssh::server::from::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 702e2e2..6dd5fce 100644 (file)
--- a/modules/roles/manifests/security_master.pp
+++ b/modules/roles/manifests/security_master.pp
@@ -14,7 +14,7 @@ class roles::security_master {
 
   # export ssh allow rules for hosts that we should be able to access
   @@ferm::rule::simple { "dsa-ssh-from-security_master-${::fqdn}":
-    tag         => 'ssh::server::allow::security_master',
+    tag         => 'ssh::server::from::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 623393f..e81bf8e 100644 (file)
--- a/modules/roles/manifests/syncproxy.pp
+++ b/modules/roles/manifests/syncproxy.pp
@@ -69,7 +69,7 @@ class roles::syncproxy {
   }
 
   @@ferm::rule::simple { "dsa-ssh-from-syncproxy-${::fqdn}":
-    tag         => 'ssh::server::allow::syncproxy',
+    tag         => 'ssh::server::from::syncproxy',
     description => 'Allow ssh access from a syncproxy',
     port        => '22',
     saddr       => $base::public_addresses,
@@ -77,9 +77,9 @@ 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::from::syncproxy' or
+    tag == 'ssh::server::from::ftp_master' or
+    tag == 'ssh::server::from::ports_master' or
+    tag == 'ssh::server::from::security_master'
     |>>
 }