Change in vdsm[master]: multipath: Configure iscsi_session recovery_tmo
Alon Bar-Lev
alonbl at redhat.com
Mon Sep 8 09:17:41 UTC 2014
Alon Bar-Lev has posted comments on this change.
Change subject: multipath: Configure iscsi_session recovery_tmo
......................................................................
Patch Set 3:
(3 comments)
http://gerrit.ovirt.org/#/c/32582/3/vdsm/storage/Makefile.am
File vdsm/storage/Makefile.am:
Line 72: volume.py
Line 73:
Line 74: dist_vdsmexec_SCRIPTS = \
Line 75: curl-img-wrap \
Line 76: multipath-configure-device
> Why do we need this?
so that future diffs will change one line (the relevant one)?
Line 77:
Line 78: nodist_vdsmstorage_DATA = \
Line 79: lvm.env \
Line 80: $(NULL)
http://gerrit.ovirt.org/#/c/32582/3/vdsm/storage/multipath-configure-device.in
File vdsm/storage/multipath-configure-device.in:
Line 7: # (at your option) any later version. See the files README and
Line 8: # LICENSE_GPL_v2 which accompany this distribution.
Line 9: #
Line 10:
Line 11: set -e
> You suggest to replace this with check for each call and logging errors?
yes... proper helpful message.
just recommendation.
Line 12:
Line 13: # Ensure that multipath devices use no_path_retry fail, instead of
Line 14: # no_path_retry queue, which is hardcoded in multipath configuration for many
Line 15: # devices.
Line 24:
Line 25: timeout=5
Line 26:
Line 27: for slave in /sys${DEVPATH}/slaves/*; do
Line 28: path=$(@UDEVADM_PATH@ info --query=path --path="$slave")
> Do you mean path="$(x y)"?
yes
Line 29: session=$(echo "$path" | @SED_PATH@ -rn \
Line 30: s'|^/devices/platform/host[0-9]+/(session[0-9]+)/.+$|\1|p')
Line 31: if [ -n "$session" ]; then
Line 32: echo $timeout > "/sys/class/iscsi_session/${session}/recovery_tmo"
--
To view, visit http://gerrit.ovirt.org/32582
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaaa40fa5e6dc86beef501ef4aaefa17c4c1574c1
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list