Filippo Giunchedi has submitted this change and it was merged.
Change subject: graphite: permit line protocol traffic from ALL_NETWORKS
......................................................................
graphite: permit line protocol traffic from ALL_NETWORKS
following I68f585093d we'll need to grant access from public networks too
Bug: T121861
Change-Id: I3b8248fcd6641923db8dc25d637317e578ae9329
---
M modules/role/manifests/graphite/base.pp
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Filippo Giunchedi: Verified; Looks good to me, approved
Muehlenhoff: Looks good to me, but someone else must approve
diff --git a/modules/role/manifests/graphite/base.pp
b/modules/role/manifests/graphite/base.pp
index e8651d4..3231799 100644
--- a/modules/role/manifests/graphite/base.pp
+++ b/modules/role/manifests/graphite/base.pp
@@ -229,13 +229,13 @@
ferm::service { 'carbon_c_relay-frontend_relay_udp':
proto => 'udp',
port => '2003',
- srange => '$INTERNAL',
+ srange => '$ALL_NETWORKS',
}
ferm::service { 'carbon_c_relay-frontend_relay_tcp':
proto => 'tcp',
port => '2003',
- srange => '$INTERNAL',
+ srange => '$ALL_NETWORKS',
}
ferm::service { 'graphite-http':
--
To view, visit https://gerrit.wikimedia.org/r/282706
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3b8248fcd6641923db8dc25d637317e578ae9329
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
