3 Commits

Author SHA1 Message Date
Thang Pham
61c09b88cd Enabled non-root users to read nfs_shares.conf.
If you use the cinder NFS driver, it will create a file called
nfs_shares.conf, containing a list of NFS shares cinder can
use.  The file is created with 660 permissions and owned by
root:root.  When c-vol starts, it has problems reading
nfs_shares.conf, resulting in a "[Errno 13] Permission denied:
'/etc/cinder/nfs_shares.conf'".  This patch creates
nfs_shares.conf without using sudo, so c-vol can at least
read it.

Change-Id: Ifce2bd4b9ff2191445a1964c1ce09c823d096d68
Closes-Bug: #1297916
2014-04-08 10:10:21 -04:00
Ian Wienand
aee18c749b Enforce function declaration format in bash8
Check that function calls look like ^function foo {$ in bash8, and fix
all existing failures of that check.  Add a note to HACKING.rst

Change-Id: Ic19eecb39e0b20273d1bcd551a42fe400d54e938
2014-02-28 07:59:03 +11:00
john-griffith
d0860cc26d Replace custom cinder driver configs
The devstack/lib/cinder file has a number of third party
driver config options hard-coded in it.  Rather than add
yet another if driver== statement here let's use plugin
files and do something similar to what's already
in place for nova_hypervisors and neutron plugins.

This works the same way folks were implementing their
drivers already, the key is to use a CINDER_DRIVER
variable in your localrc file that matches the name
of the lib/cinder_plugin file to use.

The existing third party driver entries that were
in lib/cinder have been migrated to cooresponding
plugin files.

Change-Id: I4ee51ea542d5aa63879afd5297311a9df727c57f
2014-02-11 11:30:04 -08:00