Change in vdsm[master]: make our udev rules take effect later than defualt Ubuntu lv...

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Thu Mar 7 09:46:37 UTC 2013


Zhou Zheng Sheng has uploaded a new change for review.

Change subject: make our udev rules take effect later than defualt Ubuntu lvm rules
......................................................................

make our udev rules take effect later than defualt Ubuntu lvm rules

There is a 85-lvm2.rules under Ubuntu /lib/udev/rules.d/, it sets the
owner group of LV to "disk', preventing VDSM from chown-ing, reading,
writing the LV. VDSM will chown the LV in the code and 12-vdsm-lvm.rules
does this as well. However 85-lvm2.rules are run after 12-vdsm-lvm.rules,
so the owner is always set by 85-lvm2.rules "root:disk" finally. In
Fedora, there is no such 85-lvm2.rules, so there is no such problem.

This patch rename 12-vdsm-lvm.rules to vdsm/storage/86-vdsm-lvm.rules,
so that our rules take effect finally.

The patch is on going work of porting VDSM to Ubuntu
http://www.ovirt.org/VDSM_on_Ubuntu

Change-Id: I9711c8388ad260427a511ed00465cdeb80a5ca1c
Signed-off-by: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
---
M vdsm.spec.in
R vdsm/storage/86-vdsm-lvm.rules
M vdsm/storage/Makefile.am
3 files changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/16/12816/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index ed71ab9..872e5a4 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -465,8 +465,8 @@
                  %{buildroot}%{_datadir}/%{vdsm_name}/respawn
 
 # Install the lvm rules
-install -Dm 0644 vdsm/storage/12-vdsm-lvm.rules \
-                 %{buildroot}/lib/udev/rules.d/12-vdsm-lvm.rules
+install -Dm 0644 vdsm/storage/86-vdsm-lvm.rules \
+                 %{buildroot}/lib/udev/rules.d/86-vdsm-lvm.rules
 
 install -Dm 0644 vdsm/limits.conf \
                  %{buildroot}/etc/security/limits.d/99-vdsm.conf
@@ -805,7 +805,7 @@
 %{_datadir}/%{vdsm_name}/gluster/exception.py*
 %{_datadir}/%{vdsm_name}/gluster/hostname.py*
 %{python_sitelib}/sos/plugins/vdsm.py*
-/lib/udev/rules.d/12-vdsm-lvm.rules
+/lib/udev/rules.d/86-vdsm-lvm.rules
 /etc/security/limits.d/99-vdsm.conf
 %{_mandir}/man8/vdsmd.8*
 %if 0%{?rhel}
diff --git a/vdsm/storage/12-vdsm-lvm.rules b/vdsm/storage/86-vdsm-lvm.rules
similarity index 100%
rename from vdsm/storage/12-vdsm-lvm.rules
rename to vdsm/storage/86-vdsm-lvm.rules
diff --git a/vdsm/storage/Makefile.am b/vdsm/storage/Makefile.am
index 8c5d8e2..b1c3a3f 100644
--- a/vdsm/storage/Makefile.am
+++ b/vdsm/storage/Makefile.am
@@ -67,7 +67,7 @@
 	volume.py
 
 EXTRA_DIST = \
-	12-vdsm-lvm.rules
+	86-vdsm-lvm.rules
 
 check-local:
 	PYTHONDONTWRITEBYTECODE=1 $(PYTHON) $(srcdir)/storage_exception.py


--
To view, visit http://gerrit.ovirt.org/12816
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9711c8388ad260427a511ed00465cdeb80a5ca1c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list