openstack/ironic
Code Issues Proposed changes

Don't depend on existing file perm for qemu hook

Browse Source 
For defensive purposes, lets make sure the hooks/qemu file is set to be
executable. Rather than relying on the source file permissions.

Change-Id: I2130adea4855f6e989dd8fb995cb830db76ba2a1
This commit is contained in:
John L. Villalovos
2016-03-17 09:25:50 -07:00
parent cb1a83f740
commit a77b24219c
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
devstack/lib/ironic
View File

@@ -601,6 +601,7 @@ function setup_qemu_log_hook {
# Copy the qemu hook to the right directory
sudo cp $IRONIC_DEVSTACK_FILES_DIR/hooks/qemu $IRONIC_LIBVIRT_HOOKS_PATH/qemu
sudo chmod -v +x $IRONIC_LIBVIRT_HOOKS_PATH/qemu
sudo sed -e "
s|%LOG_DIR%|$IRONIC_VM_LOG_DIR|g;
" -i $IRONIC_LIBVIRT_HOOKS_PATH/qemu