Alexandros Kosiaris has submitted this change and it was merged.
Change subject: tftp is udp not tcp
......................................................................
tftp is udp not tcp
Wrong ferm rule all along
Change-Id: I6187acf53d737cee72d98f10c0f112b2ef6ef87b
---
M manifests/role/install-server.pp
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Alexandros Kosiaris: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/role/install-server.pp b/manifests/role/install-server.pp
index 091459e..e6e074e 100644
--- a/manifests/role/install-server.pp
+++ b/manifests/role/install-server.pp
@@ -39,7 +39,7 @@
include install-server::tftp-server
ferm::rule { 'tftp':
- rule => 'proto tcp dport tftp { saddr $ALL_NETWORKS ACCEPT; }'
+ rule => 'proto udp dport tftp { saddr $ALL_NETWORKS ACCEPT; }'
}
include install-server::caching-proxy
@@ -118,6 +118,6 @@
include install-server::tftp-server
ferm::rule { 'tftp':
- rule => 'proto tcp dport tftp { saddr $ALL_NETWORKS ACCEPT; }'
+ rule => 'proto udp dport tftp { saddr $ALL_NETWORKS ACCEPT; }'
}
}
--
To view, visit https://gerrit.wikimedia.org/r/101200
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6187acf53d737cee72d98f10c0f112b2ef6ef87b
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
