Martin Mucha has posted comments on this change.

Change subject: engine: Introduce HostSetupNetworks command
......................................................................


Patch Set 26:

(1 comment)

https://gerrit.ovirt.org/#/c/33333/26/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java:

Line 273:                     passed = false;
Line 274:                     continue;
Line 275:                 }
Line 276: 
Line 277:                 String slaveBondName = slave.getBondName();
> 1. This validation blocks the possibility of moving slave from one bond to 
added method verifying, that previous slaves master is updated in current 
request and does not claims this slaves as his own 
(bondIsUpdatedAndDoesNotContainCertainSlave)

I think there should be && instead || so I changed it.

added tests for this scenario.

Done.
Line 278:                 if (slave.isBondSlave() && 
(!slaveBondName.equals(modifiedBond.getName())
Line 279:                         || !slaveUsedByRemovedBond(slaveBondName))) {
Line 280:                     
addViolation(VdcBllMessages.NETWORK_INTERFACE_ALREADY_IN_BOND, slaveName);
Line 281:                     passed = false;


-- 
To view, visit https://gerrit.ovirt.org/33333
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I60077019f308f371f21fb7b5545ba48adb38bd06
Gerrit-PatchSet: 26
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmu...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to