Moti Asayag has posted comments on this change.
Change subject: engine: Modify Vm Interface commands to work with VmNic
......................................................................
Patch Set 22:
(4 comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/AddVmInterfaceCommand.java
Line 194: if (getInterface() != null &&
getInterface().getVnicProfileId() != null && getVm() != null) {
Line 195:
Line 196: VnicProfile profile =
getVnicProfileDao().get(getInterface().getVnicProfileId());
Line 197:
Line 198: if (profile != null && profile.isPortMirroring()) {
Done
Line 199: permissionList.add(new
PermissionSubject(profile.getId(),
Line 200: VdcObjectType.VnicProfile,
Line 201: ActionGroup.PORT_MIRRORING));
Line 202: } else {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java
Line 87: public Void runInTransaction() {
Line 88: getCompensationContext().snapshotEntity(oldIface);
Line 89:
getCompensationContext().snapshotEntity(oldVmDevice);
Line 90: getVmNicDao().update(getInterface());
Line 91: getDbFacade().getVmDeviceDao().update(oldVmDevice);
you're 100% right. plug/unplugged are done in the internal command, so there no
need to take care of the oldVmDevice in this context.
Line 92: getCompensationContext().stateChanged();
Line 93: return null;
Line 94: }
Line 95: });
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java
Line 444: public static VmDevice addNetworkInterfaceDevice(VmDeviceId id,
boolean plugged, Guid vnicProfileId) {
Line 445: Map<String, String> customProperties = null;
Line 446: if (vnicProfileId != null) {
Line 447: VnicProfile profile =
DbFacade.getInstance().getVnicProfileDao().get(vnicProfileId);
Line 448: customProperties = profile == null ? null :
profile.getCustomProperties();
Done
Line 449: }
Line 450:
Line 451: return addManagedDevice(id,
Line 452: VmDeviceGeneralType.INTERFACE,
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HotPlugNicVDSCommand.java
Line 43: Network network = null;
Line 44: if (nic.getVnicProfileId() != null) {
Line 45: vnicProfile =
getDbFacade().getVnicProfileDao().get(nic.getVnicProfileId());
Line 46: if (vnicProfile != null) {
Line 47: network =
getDbFacade().getNetworkDao().get(vnicProfile.getNetworkId());
since the DAL is accessible from the vdsbroker, it is achievable.
does all of the command gathers all the information before they pass it to the
vdsbroker ? should they even do that ?
let's think about that later. the downside might be a parameters horror.
Line 48: }
Line 49: }
Line 50: map.put(VdsProperties.Type, vmDevice.getType().getValue());
Line 51: map.put(VdsProperties.Device, VmDeviceType.BRIDGE.getName());
--
To view, visit http://gerrit.ovirt.org/17229
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I92480426f2e1d49d19a1859d4ef1008699568402
Gerrit-PatchSet: 22
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Mike Kolesnik <[email protected]>
Gerrit-Reviewer: Moti Asayag <[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
