Change in ovirt-engine[master]: core: Removing the use of getDisks from RestoreCommand

mlipchuk at redhat.com mlipchuk at redhat.com
Thu Mar 6 16:11:18 UTC 2014


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Removing the use of getDisks from RestoreCommand
......................................................................


core: Removing the use of getDisks from RestoreCommand

There is no place that initializes the disks in the
parameter class when RestoreAllSnapshot is called.
Since there is no use of it whatsoever, this patch removes the use of
it in the command, so we can decouple RestoreAllSnapshotsParameters from
TryBackToAllSnapshotsOfVmParameters

Change-Id: I1d0f5f92288ea92c114526e0d7d567a7ece45fbd
Related to Bug-Url: https://bugzilla.redhat.com/867339
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreAllSnapshotsCommand.java
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1d0f5f92288ea92c114526e0d7d567a7ece45fbd
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list