Allon Mureinik has submitted this change and it was merged. Change subject: core: persist LUN with correct volume_group_id ...................................................................... core: persist LUN with correct volume_group_id When saving a LUN object to the db, we specify what is the needed volume group. In case of an existing LUN - its volume group id would be updated to the new given value. In case that the given LUN isn't in the DB, it'll be saved. This patch ensures that the saved LUN would be saved with the given volume_group_id. NOTES: The motivation for this change is that in cases in which we get the LUN info from VDSM and the LUN is part of an existing VG, saving it with unknown VG to the system will prevent removal of the LUN when it won't be used anymore by the system and might cause confusion and inconsistency. Change-Id: Ia4404b0d17b107208e5ec00c688db305ba0e65fe Signed-off-by: Liron Aravot <laravot(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageDomainCommandBase.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Liron Aravot: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10630 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4404b0d17b107208e5ec00c688db305ba0e65fe Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Daniel Paikov <paikov(a)gmail.com&gt; Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt; Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt;