Dan Kenigsberg has posted comments on this change. Change subject: lvm: Set libvirt image selinux label on block devices backing vdsm images ......................................................................
Patch Set 3: (1 comment) http://gerrit.ovirt.org/#/c/33492/3//COMMIT_MSG Commit Message: Line 22: We don't know why the selinux label is lost, and lvm developers claim Line 23: that relevant code was not changed recently. This issue may be caused by Line 24: lower level components such as device mapper, multipath or iscsi. Line 25: Line 26: This patch adds a temporary solution, by updating vdsm lvm rules to set > We will open a bug for libvirt. Thanks! and mention it the the udev rule text, so that a future developer asking himself: "Is nonsense this still relevent?" would know immediately where to look. Line 27: the libvirt image selinux label on vdsm images. Line 28: Line 29: Change-Id: I95f85c7b548b2c058693b20b1fa177714a6e1a10 Line 30: Bug-Url: https://bugzilla.redhat.com/1127460 -- To view, visit http://gerrit.ovirt.org/33492 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95f85c7b548b2c058693b20b1fa177714a6e1a10 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
