Dan Kenigsberg has submitted this change and it was merged. Change subject: lvm: Do not use udev cache for obtaining device list ......................................................................
lvm: Do not use udev cache for obtaining device list lvm is obtaining the device list from udev. When using concurrently, udev sometimes returns incomplete list, which cause lvm to think that a vg is missing, and the command may fail with "Volume group not found" error, or "Cannot change VG test while PVs are missing". lvm fixed this issue in version 2.02.100-7.el6 by disabling udev cache, setting obtain_device_list_from_udev to 0. Unfortunatlly, lvm fix is not enough for vdsm, as the fix is applied only if no lvm.conf file exists. When upgrading existing lvm installation, lvm creates a lvm.conf.rpmnew file, and the system administrator is responsible for updating lvm configuration. This patch disable udev cache using the --config option, used to override many other lvm options, ensuring proper configuration on both new and upgraded systems. Disabling udev cache may have minimal performance effect according to lvm developers. Change-Id: Ib55c8d444f3be9f63bfd23d8def60607b5b3dff0 Bug-Url: https://bugzilla.redhat.com/1014942 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/20890 Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-by: Ayal Baron <[email protected]> Reviewed-by: Allon Mureinik <[email protected]> --- M vdsm/storage/lvm.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ayal Baron: Looks good to me, approved Nir Soffer: Verified Allon Mureinik: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/20890 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib55c8d444f3be9f63bfd23d8def60607b5b3dff0 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
