Muli Salem has posted comments on this change.

Change subject: core: Allow Non-Required Networks Not On Host (#858654)
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsSelector.java
Line 335:     }
Line 336: 
Line 337:     /**
Line 338:     * Determine whether all required Networks are attached to the 
Host's Nics. A required Network, depending on
Line 339:     * ConfigValue.OnlyRequiredNetworksMandatoryForVdsSelection, is 
defined as: 
No tws here.
Line 340:     * 1. false: any network that is defined on an Active vNic of the 
VM.
Line 341:     * 2. true: a Cluster-Required Network that is defined on an 
Active vNic of the VM.
Line 342:     * @param vdsId
Line 343:     *            The Host id.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4909a658bf729d839c9f01268f1295e43391a2c3
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <msa...@redhat.com>
Gerrit-Reviewer: Itzik Brown <itz...@mellanox.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Muli Salem <msa...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to