Change in ovirt-engine[ovirt-engine-4.1]: core: Initialize SparsifyImageCommand#vmsForDisk

Code Review gerrit at ovirt.org
Sun Apr 9 14:19:22 UTC 2017


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

Tal Nisan has submitted this change and it was merged.

Change subject: core: Initialize SparsifyImageCommand#vmsForDisk
......................................................................


core: Initialize SparsifyImageCommand#vmsForDisk

Initialize SparsifyImageCommand#vmsForDisk field on demand. Previously
this field was always initialized to an empty list which caused
SparsifyImageCommand to ignore status of VMs the disk is attached to and
to fail to lock these VMs.

Change-Id: I3d87eb858d144de2f4e855709b8638c783ed5640
Bug-Url: https://bugzilla.redhat.com/1431596
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/SparsifyImageCommand.java
1 file changed, 17 insertions(+), 5 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Shmuel Leib Melamud: Verified
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d87eb858d144de2f4e855709b8638c783ed5640
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.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>


More information about the Engine-commits mailing list