Tal Nisan has posted comments on this change. Change subject: host-deploy: Configure firewall to allow rpc.statd port ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/37602/2/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql File packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql: Line 689: # vdsm Line 690: -A INPUT -p tcp --dport @VDSM_PORT@ -j ACCEPT Line 691: # rpc.statd Line 692: -A INPUT -p tcp --dport 111 -j ACCEPT Line 693: -A INPUT -p udp --dport 111 -j ACCEPT > not the right place, see bellow. The IPTablesConfigForVirt is added on top of the IPTablesConfig? If so we can just remove the Gluster ones cause it's redundant to add them to both Line 694: # SSH Line 695: -A INPUT -p tcp --dport @SSH_PORT@ -j ACCEPT Line 696: # snmp Line 697: -A INPUT -p udp --dport 161 -j ACCEPT -- To view, visit http://gerrit.ovirt.org/37602 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I80b53b2c2e87000bb8af43b263b8c7fe1d904964 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
