openstack/devstack
Code Issues Proposed changes

chown -R to ensure that stack user owns data_dir

Browse Source 
The fix for bug #1129748 tightens some permissions on image
directories.  This breaks the devstack gate jobs, which were
relying on wide-open permissions.  chowning recursively should
ensure that all the directories are owned by the stack user,
so the gate tests work with tighter permissions.

Change-Id: I09be54bff6d8f9c3f7f599d6e5ada6431beb7c31
This commit is contained in:
David Ripton
2013-03-01 11:48:27 -05:00
parent b057f4f75c
commit 1bb758758b
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
stack.sh
View File

@@ -197,9 +197,7 @@ fi
# Create the destination directory and ensure it is writable by the user # Create the destination directory and ensure it is writable by the user
sudo mkdir -p $DEST sudo mkdir -p $DEST
if [ ! -w $DEST ]; then sudo chown -R $STACK_USER $DEST
sudo chown $STACK_USER $DEST
fi
# Set ``OFFLINE`` to ``True`` to configure ``stack.sh`` to run cleanly without # Set ``OFFLINE`` to ``True`` to configure ``stack.sh`` to run cleanly without
# Internet access. ``stack.sh`` must have been previously run with Internet # Internet access. ``stack.sh`` must have been previously run with Internet
@@ -214,7 +212,7 @@ ERROR_ON_CLONE=`trueorfalse False $ERROR_ON_CLONE`
# Destination path for service data # Destination path for service data
DATA_DIR=${DATA_DIR:-${DEST}/data} DATA_DIR=${DATA_DIR:-${DEST}/data}
sudo mkdir -p $DATA_DIR sudo mkdir -p $DATA_DIR
sudo chown $STACK_USER $DATA_DIR sudo chown -R $STACK_USER $DATA_DIR
# Common Configuration # Common Configuration