Alexandros Kosiaris has submitted this change and it was merged.
Change subject: beta: remove duplicate ferm::rule
......................................................................
beta: remove duplicate ferm::rule
The Parsoid instance on the beta cluster is a Jenkins slave which is
setup via role::ci::slave::labs::common which in turns adds a ferm rule
to allow ssh from Jenkins master (role::ci::slave::labs::common).
Drop the redundant ferm rule defined in the Parsoid manifest.
Change-Id: I41ef6a4c87dc6aec163ce6043ed3a913c94b715e
---
M manifests/role/parsoid.pp
1 file changed, 0 insertions(+), 5 deletions(-)
Approvals:
Hashar: Looks good to me, but someone else must approve
Alexandros Kosiaris: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/parsoid.pp b/manifests/role/parsoid.pp
index e45287c..026c804 100644
--- a/manifests/role/parsoid.pp
+++ b/manifests/role/parsoid.pp
@@ -233,9 +233,4 @@
# Also need the slave scripts for multi-git.sh
include contint::slave-scripts
- # And thus allow ssh access from the Jenkins master (gallium)
- ferm::rule { 'ssh-from-gallium':
- rule => 'proto tcp dport ssh { saddr 208.80.154.135 ACCEPT; }',
- }
-
}
--
To view, visit https://gerrit.wikimedia.org/r/140891
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I41ef6a4c87dc6aec163ce6043ed3a913c94b715e
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: KartikMistry <kartik.mis...@gmail.com>
Gerrit-Reviewer: Alexandros Kosiaris <akosia...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Nikerabbit <niklas.laxst...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
