Change in vdsm[ovirt-3.5]: multipath: add specific fail entry for emc-vnx

Federico Simoncelli fsimonce at redhat.com
Fri Nov 14 19:54:23 UTC 2014


Federico Simoncelli has uploaded a new change for review.

Change subject: multipath: add specific fail entry for emc-vnx
......................................................................

multipath: add specific fail entry for emc-vnx

As few other storage arrays also the emc-vnx is forcing the
queue_if_no_path option on the multipath device.

This patch adds a specific section for the device maintaining
the emc defaults but forcing to fail the io instead of queuing.

In case of queuing we observed that the processes were blocked
on io and among other things it prevented the storage pool to
be reconstructed.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1098769
Change-Id: Icf5f4c89eac1273bf3a8e1afa300fec7b5b68eae
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/35072
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
(cherry picked from commit 9880fdf9989718a097bb05571a6344e28c2f258b)
---
M vdsm/storage/multipath.py
1 file changed, 20 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/79/35179/1

diff --git a/vdsm/storage/multipath.py b/vdsm/storage/multipath.py
index 68502ce..34573c2 100644
--- a/vdsm/storage/multipath.py
+++ b/vdsm/storage/multipath.py
@@ -53,8 +53,9 @@
 OLD_TAGS = ["# RHAT REVISION 0.2", "# RHEV REVISION 0.3",
             "# RHEV REVISION 0.4", "# RHEV REVISION 0.5",
             "# RHEV REVISION 0.6", "# RHEV REVISION 0.7",
-            "# RHEV REVISION 0.8", "# RHEV REVISION 0.9"]
-MPATH_CONF_TAG = "# RHEV REVISION 1.0"
+            "# RHEV REVISION 0.8", "# RHEV REVISION 0.9",
+            "# RHEV REVISION 1.0"]
+MPATH_CONF_TAG = "# RHEV REVISION 1.1"
 MPATH_CONF_PRIVATE_TAG = "# RHEV PRIVATE"
 STRG_MPATH_CONF = (
     "\n\n"
@@ -82,6 +83,23 @@
     "    product                 \"Compellent Vol\"\n"
     "    no_path_retry           fail\n"
     "}\n"
+    "device {\n"
+    "    # multipath.conf.default\n"
+    "    vendor                  \"DGC\"\n"
+    "    product                 \".*\"\n"
+    "    product_blacklist       \"LUNZ\"\n"
+    "    path_grouping_policy    \"group_by_prio\"\n"
+    "    path_checker            \"emc_clariion\"\n"
+    "    hardware_handler        \"1 emc\"\n"
+    "    prio                    \"emc\"\n"
+    "    failback                immediate\n"
+    "    rr_weight               \"uniform\"\n"
+    "    # vdsm required configuration\n"
+    "    getuid_callout          \"%(scsi_id_path)s --whitelisted "
+    "--replace-whitespace --device=/dev/%%n\"\n"
+    "    features                \"0\"\n"
+    "    no_path_retry           fail\n"
+    "}\n"
     "}"
 )
 MPATH_CONF_TEMPLATE = MPATH_CONF_TAG + STRG_MPATH_CONF


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf5f4c89eac1273bf3a8e1afa300fec7b5b68eae
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>


More information about the vdsm-patches mailing list