Log directory is being assumed in upstart init scripts

Bug #1460469 reported by Kevin Carter
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Kevin Carter

Bug Description

In the upstart init scripts we are assuming that the log directory will always match the name of the program. This assumption breaks down when dealing with multiple regions. To fix this we need to set the log directory per the role not attempt to dynamically map it to some arbitrary program name. The reason that this is being done is due to legacy bits that came over from the galaxification process.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (master)

Fix proposed to branch: master
Review: https://review.openstack.org/187074

Changed in openstack-ansible:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (kilo)

Fix proposed to branch: kilo
Review: https://review.openstack.org/187368

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

Reviewed: https://review.openstack.org/187074
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=69d22e926cde40cd775d9bd4070a1d8e80991e64
Submitter: Jenkins
Branch: master

commit 69d22e926cde40cd775d9bd4070a1d8e80991e64
Author: kevin <email address hidden>
Date: Sun May 31 14:52:52 2015 -0500

    Set the log directory accordingly

    This changes the log directory within the upstart init scripts
    such that they are being set throughout the roles and removes an
    assumed variable name as the name of the log directory.

    Change-Id: I32a2a76a4a73b609999886427b494f380850f12a
    Closes-Bug: #1460469

Changed in openstack-ansible:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (kilo)

Reviewed: https://review.openstack.org/187368
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=5147bd27c69f434f6fd71d6322d6534631d8bf9d
Submitter: Jenkins
Branch: kilo

commit 5147bd27c69f434f6fd71d6322d6534631d8bf9d
Author: kevin <email address hidden>
Date: Sun May 31 14:52:52 2015 -0500

    Set the log directory accordingly

    This changes the log directory within the upstart init scripts
    such that they are being set throughout the roles and removes an
    assumed variable name as the name of the log directory.

    Change-Id: I32a2a76a4a73b609999886427b494f380850f12a
    Closes-Bug: #1460469
    (cherry picked from commit 69d22e926cde40cd775d9bd4070a1d8e80991e64)

Changed in openstack-ansible:
status: Fix Committed → Fix Released
Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.11

This issue was fixed in the openstack/openstack-ansible 11.2.11 release.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.12

This issue was fixed in the openstack/openstack-ansible 11.2.12 release.

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.14

This issue was fixed in the openstack/openstack-ansible 11.2.14 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.