]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
rtstool on Ubuntu installs in /usr/local/bin
authorJohn Griffith <john.griffith@solidfire.com>
Fri, 19 Dec 2014 19:32:02 +0000 (19:32 +0000)
committerJohn Griffith <john.griffith@solidfire.com>
Fri, 19 Dec 2014 19:32:02 +0000 (19:32 +0000)
The cinder-rtstool which is required to use LIO iscsi_helper
installs in /usr/local/bin on Ubuntu and /usr/bin on RHEL
variants.  Currently the rootwrap.conf only has /usr/bin
configured, as a result LIO can't be used on Ubuntu without
modifying the rootwrap file.

This patch just adds /usr/local/bin to the rootwrap.conf; Honestly
I'm not sure if this is good, bad or doesn't matter.  May be able
to address this via documentation, or upstream in the packaging of
cinder-rtstool instead?  Not sure what's preferred.

Change-Id: I577e10df25181c7d4ca8189b10640a8e6e39b047
Closes-Bug: #1402076

etc/cinder/rootwrap.conf

index 001b90af5d7ed72a15395733bbf304b14b809a7a..beac6c4bab1a652efba24336dea5e4b2cd7b0d46 100644 (file)
@@ -10,7 +10,7 @@ filters_path=/etc/cinder/rootwrap.d,/usr/share/cinder/rootwrap
 # explicitely specify a full path (separated by ',')
 # If not specified, defaults to system PATH environment variable.
 # These directories MUST all be only writeable by root !
-exec_dirs=/sbin,/usr/sbin,/bin,/usr/bin
+exec_dirs=/sbin,/usr/sbin,/bin,/usr/bin,/usr/local/bin
 
 # Enable logging to syslog
 # Default value is False