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(a)redhat.com&gt; --- 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(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server