]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Execute mount.nfs check with absolute path
authorTom Patzig <tom.patzig@sap.com>
Mon, 26 Oct 2015 19:28:05 +0000 (20:28 +0100)
committerTom Patzig <tom.patzig@sap.com>
Wed, 4 Nov 2015 21:13:13 +0000 (22:13 +0100)
commit64172b01cd5d8eda3aa282262d7ca060066161d7
tree60ff0d816e21d74b1b74153fddb7a9341bf97776
parent321c9c901d3b456c188a5e76f4dd102bdde2c0d8
Execute mount.nfs check with absolute path

Currently the existence of mount.nfs is checked by executing
the relative binary 'mount.nfs' with a non-root user,
in this case cinder. This results, for example on SUSE, in the error:

      NfsException: mount.nfs is not installed

Because mount.nfs is located under /sbin, unprivileged users
do not have /sbin in their PATH to search for executables.
The change runs the mount.nfs check by using the absolute binary
path /sbin/mount.nfs. This seems to be common for most distributions
(SUSE, RedHat, CentOS, Ubuntu, Debian). The check can still be executed
as non privileged user, by not relying on correctly set PATH variable
and using the absolute path.

Change-Id: I3c1ecfdadd9ea492d58d69cbdf33045b002668c7
Closes-Bug: #1510150
cinder/tests/unit/test_nfs.py
cinder/volume/drivers/nfs.py