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&gt; --- 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&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com&gt; Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com&gt; Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com&gt;