Mike Kolesnik has submitted this change and it was merged. Change subject: core: adding AbstractVmInterfaceCommand ...................................................................... core: adding AbstractVmInterfaceCommand This command will contain common logic of VmIntrefaceCommands. For now, the logic of internaly calling activateNic (called from AddVmInterfaceCommnad and will be called from UpdateVmInterfaceCommanf in the future) was moved to AbstractVmInterfaceCommand. In the future other common logic can be placed here. Change-Id: I15acf7c715703582e96274f4621239b33d263172 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com&gt; --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AbstractVmInterfaceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmInterfaceCommand.java 2 files changed, 45 insertions(+), 32 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9667 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15acf7c715703582e96274f4621239b33d263172 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com&gt; Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com&gt; Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com&gt; Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com&gt;