From 61c09b88cd1e26e54e44408d3888e9b563d8f49f Mon Sep 17 00:00:00 2001 From: Thang Pham Date: Wed, 26 Mar 2014 11:57:04 -0400 Subject: [PATCH] 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 --- lib/cinder_plugins/nfs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/cinder_plugins/nfs b/lib/cinder_plugins/nfs index 2d9d875245..5f4cc5369a 100644 --- a/lib/cinder_plugins/nfs +++ b/lib/cinder_plugins/nfs @@ -30,8 +30,7 @@ set +o xtrace function configure_cinder_driver { iniset $CINDER_CONF DEFAULT volume_driver "cinder.volume.drivers.nfs.NfsDriver" iniset $CINDER_CONF DEFAULT nfs_shares_config "$CINDER_CONF_DIR/nfs_shares.conf" - echo "$CINDER_NFS_SERVERPATH" | sudo tee "$CINDER_CONF_DIR/nfs_shares.conf" - sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf + echo "$CINDER_NFS_SERVERPATH" | tee "$CINDER_CONF_DIR/nfs_shares.conf" } # Restore xtrace