[node-patches] Change in ovirt-node[master]: recipe: Create a vdsm-like multipath.conf

[fabiand at redhat.com]

Fabian Deutsch has uploaded a new change for review.

Change subject: recipe: Create a vdsm-like multipath.conf
......................................................................

recipe: Create a vdsm-like multipath.conf

Previously we were creating a quite sparse multipath.conf compared
to the default one created by vdsm.
With this patch we ask vdsm-tool during build to create the multipath.conf
the assumption is that this will avoid bugs when the host is doing
the switchroot from initramfs to userspace.

One problem which could occur is that the accepted devices change
because the device filters are different between the two config files.

Change-Id: I450b33b2f5e2325b5850bc05c4a9691b45df4964
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1275956
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M recipe/common-post.ks
1 file changed, 4 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/70/48470/1

diff --git a/recipe/common-post.ks b/recipe/common-post.ks
index 27c7abc..866ea2e 100644
--- a/recipe/common-post.ks
+++ b/recipe/common-post.ks
@@ -280,15 +280,10 @@
 # https://bugzilla.redhat.com/show_bug.cgi?id=1168582
 rm -vf /usr/lib64/python2.*/site-packages/backports/*
 
-# https://bugzilla.redhat.com/show_bug.cgi?id=1167620
-# Ensure that mpath is enabled and find_multipaths is y
-mpathconf --enable --find_multipaths y
-# Then ensure that getuid_callout is set for b/c
-sed -i \
-   -e "/find_multipaths / a getuid_callout \"/lib/udev/scsi_id --replace-whitespace --whitelisted --device=/dev/%n\"" \
-   -e "/^#/ d" \
-   -e "/user_friendly_names/ d" \
-   /etc/multipath.conf
+# Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1275956
+# Create a multipath config which is close to the final one to prevent
+# problems during the initramfs-userspace switchroot.
+vdsm-tool configure --module multipath --force
 
 # Also update os-release, this is where ply 7.0 is taking the name from
 # we should update both since both can be used


-- 
To view, visit https://gerrit.ovirt.org/48470
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I450b33b2f5e2325b5850bc05c4a9691b45df4964
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>