Change in ovirt-engine[master]: core: Extract Validator creation and apply usages

amureini at redhat.com amureini at redhat.com
Tue Jul 29 13:21:18 UTC 2014


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Extract Validator creation and apply usages
......................................................................


core: Extract Validator creation and apply usages

There are several places creating a MultipleStorageDomainsValidator
based on a VM and DisksImage List in the same manner.
Some new code for bellow bzs does the same, as will other patches. This
patch removed future code duplicity by extracting the
duplicated code into VmCommand.

Change-Id: I3a6d04fe1a670c814b44bad5304a894faa8f8510
Relatee-To: https://bugzilla.redhat.com/1054175
Signed-off-by: Vered Volansky <vvolansk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java
2 files changed, 7 insertions(+), 5 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Vered Volansky: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3a6d04fe1a670c814b44bad5304a894faa8f8510
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list