kolla-ansible

deploy ceilometer+panko raise undefined variable error

Bug #1725159 reported by Jeffrey Zhang on 2017-10-20

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	kolla-ansible	Fix Released	Undecided	Jeffrey Zhang	kolla-ansible queens-2
	Pike	Fix Released	Undecided	Jeffrey Zhang	kolla-ansible 5.0.1 "pike"

Bug Description

Panko.conf depends on "service_name" variable, but when generate panko.conf in ceilometer, this variable is not set.

    {% if service_name == 'panko-api' %}
    # Force panko-api.log or will use app.wsgi
    log_file = /var/log/kolla/panko/panko-api.log
    {% endif %}

Panko is a single process service, no need add such a variable unless there are two processes in the future.

Jeffrey Zhang (jeffrey4l) on 2017-10-20

affects:	kolla → kolla-ansible
no longer affects:	kolla/pike
Changed in kolla-ansible:
milestone:	none → queens-1
assignee:	nobody → Jeffrey Zhang (jeffrey4l)

OpenStack Infra (hudson-openstack) on 2017-10-20

Changed in kolla-ansible:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-10-31: Fix merged to kolla-ansible (master)

Reviewed: https://review.openstack.org/513640
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=791bf3d5fa47729604eda9d6318c031c0b612a1a
Submitter: Zuul
Branch: master

commit 791bf3d5fa47729604eda9d6318c031c0b612a1a
Author: Jeffrey Zhang <email address hidden>
Date: Fri Oct 20 15:50:48 2017 +0800

Remove test variable for service_name in panko.conf file

panko.conf depends on "service_name" variable, but when generating
panko.conf in ceilometer, this variable is not set.

Panko is a single process service, no need add such test. So simple
remove the if clause.

Change-Id: I5ba41046e35cd3fa00b8c22e1b4435acfaff5279
Closes-Bug: #1725159

Changed in kolla-ansible:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-10-31: Fix proposed to kolla-ansible (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/516562

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2017-11-01: Fix merged to kolla-ansible (stable/pike)

Reviewed: https://review.openstack.org/516562
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=90fd8b33da9973cf3618f784bc91f8389111eb02
Submitter: Zuul
Branch: stable/pike

commit 90fd8b33da9973cf3618f784bc91f8389111eb02
Author: Jeffrey Zhang <email address hidden>
Date: Fri Oct 20 15:50:48 2017 +0800

Remove test variable for service_name in panko.conf file

panko.conf depends on "service_name" variable, but when generating
panko.conf in ceilometer, this variable is not set.

Panko is a single process service, no need add such test. So simple
remove the if clause.

    Change-Id: I5ba41046e35cd3fa00b8c22e1b4435acfaff5279
    Closes-Bug: #1725159
    (cherry picked from commit 791bf3d5fa47729604eda9d6318c031c0b612a1a)