Alona Kaplan has posted comments on this change.
Change subject: core: HotPlugNicVDSCommand should send "linkState" to the vdsm.
......................................................................
Patch Set 17: (1 inline comment)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateDeactivateVmNicCommand.java
Line 42: // HotPlug in the host needs to be called only if the Vm
is UP
Line 43: if (hotPlugVmNicRequired(getVm().getStatus())) {
Line 44: setVdsId(getVm().getRunOnVds().getValue());
Line 45: returnValue = canPerformHotPlug();
Line 46: if (returnValue && (getNetworkName() != null &&
!networkAttachedToVds(getNetworkName(), getVdsId()))) {
Removing it from here will cause a bug. Because networkAttachedToVds returns
false if the network is null.
So if the network is null, you will get and error message.
Line 47:
addCanDoActionMessage(VdcBllMessages.ACTIVATE_DEACTIVATE_NETWORK_NOT_IN_VDS);
Line 48: returnValue = false;
Line 49: }
Line 50: }
--
To view, visit http://gerrit.ovirt.org/9519
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I127fc3202702f8cb1920990eb77d9fbccdb9804b
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.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
