Comment 4 for bug 1294213

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to devstack (master)

Reviewed: https://review.openstack.org/85120
Committed: https://git.openstack.org/cgit/openstack-dev/devstack/commit/?id=410f5c0016a9d3b1fbd42b95ce1402a1c614e3d3
Submitter: Jenkins
Branch: master

commit 410f5c0016a9d3b1fbd42b95ce1402a1c614e3d3
Author: Ryota MIBU <email address hidden>
Date: Fri Apr 4 02:00:31 2014 +0900

    Fix unsubstituted filename creation

    Since merge_config_file() tries to create an unsubstituted config file,
    stack.sh fails due to permission denied in the file creation when you
    use '[[post-config|/$Q_PLUGIN_CONF_FILE]]' and the file does not exist.

    This patch deletes unnecessary 'touch' command, because the file will
    be made by 'iniset' function in the next command line with evaled
    string from 'configfile'.

    This patch also fixes merge_config_group() to use evaled 'configfile'
    when it checks the directory of the config file exists.

    Change-Id: I608847a34143b5c6a1708c180186dd88a32dd44b
    Closes-bug: #1294213