Change in ovirt-engine[ovirt-engine-4.1]: core: Set disks from db when init CreateAllTemplateDisksFrom...

Code Review gerrit at ovirt.org
Tue May 23 13:51:32 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: Set disks from db when init CreateAllTemplateDisksFromSnapshotCommand
......................................................................


core: Set disks from db when init CreateAllTemplateDisksFromSnapshotCommand

Before this patch, the init of CreateAllTemplateDisksFromSnapshotCommand
lacks the proccess of populate vm disks from db to command context's getVm(),
thus caused the following filterImageDisks(...) always operate on an empty
diskmap.

Change-Id: I57be9e653c825db21f66c640437544481c94fc13
Bug-Url: https://bugzilla.redhat.com/1453010
Signed-off-by: plysan <plysab at gmail.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/CreateAllTemplateDisksFromSnapshotCommand.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  plysan: Verified; Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/77076
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I57be9e653c825db21f66c640437544481c94fc13
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: plysan <plysab at gmail.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-Reviewer: plysan <plysab at gmail.com>


More information about the Engine-commits mailing list