Dan Kenigsberg has posted comments on this change.

Change subject: Sysctl to allow iSCSI multipath with multiple NICs in the same 
subnet
......................................................................


Patch Set 7: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/31529/7/vdsm/vdsm-sysctl.conf
File vdsm/vdsm-sysctl.conf:

Line 13: # configuration, reverse path filtering must be loosened with 
"rp_filter=2"
Line 14: # (using "default" so an admin can override it per interface if needed)
Line 15: net.ipv4.conf.default.arp_ignore = 1
Line 16: net.ipv4.conf.default.arp_announce = 2
Line 17: net.ipv4.conf.default.rp_filter = 2
According to http://www.ietf.org/rfc/rfc3704.txt (section 2.4) loose mode has 
security disadvantages. If an untrusted VM is connected to a bridge with an IP 
address and with this setting, the VM could DoS the host by sending it packages 
with spoofed random sources.

It would be more prudent to set loose mode only on specific interfaces that 
need it - though doing so properly may be a bit tedious.


-- 
To view, visit http://gerrit.ovirt.org/31529
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf93d49317c76aece764e53e58e0ff28868f16b0
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amador Pahim <apa...@redhat.com>
Gerrit-Reviewer: Amador Pahim <apa...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to