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

iheim at redhat.com iheim at redhat.com
Sun Mar 16 13:30:56 UTC 2014


Itamar Heim 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
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
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1d0f5f92288ea92c114526e0d7d567a7ece45fbd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
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: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list