Change in ovirt-engine[master]: core: NPE while adding lun disks (#869596)
Daniel Erez has submitted this change and it was merged.
Change subject: core: NPE while adding lun disks (#869596)
......................................................................
core: NPE while adding lun disks (#869596)
fixing regression that was introduced in patch with
change id I8cb77b017282733d72ff66af499b576674674e01
while adding a disk there is an attempt to check the storage domain
of the disk in order to verify whether the added disk storage needs to be
wiped after it's deletion, lun disks don't have storage domain and can't be
wiped which caused to NPE.
Change-Id: Id754bdd7b49ff0dcc80b060f5900f8e3ea313637
Bug-Url: https://bugzilla.redhat.com/869596
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java
1 file changed, 4 insertions(+), 5 deletions(-)
Approvals:
Daniel Erez: Looks good to me, approved
Allon Mureinik: Looks good to me, but someone else must approve
Vered Volansky: Looks good to me, but someone else must approve
Liron Aravot: Verified
--
To view, visit http://gerrit.ovirt.org/8821
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id754bdd7b49ff0dcc80b060f5900f8e3ea313637
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>
Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>