openstack/kolla-ansible
Code Issues Proposed changes

Remove test variable for service_name in panko.conf file

Browse Source 
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
This commit is contained in:
Jeffrey Zhang 2017-10-20 15:50:48 +08:00
parent 175c65e490
commit 791bf3d5fa
2 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ansible/roles/panko/tasks/config.yml
View File

@ -22,8 +22,6 @@
- Restart panko-api container - Restart panko-api container
- name: Copying over panko.conf - name: Copying over panko.conf
vars:
service_name: "{{ item.key }}"
merge_configs: merge_configs:
sources: sources:
- "{{ role_path }}/templates/panko.conf.j2" - "{{ role_path }}/templates/panko.conf.j2"

2
ansible/roles/panko/templates/panko.conf.j2
View File

@ -3,10 +3,8 @@
log_dir = /var/log/kolla/panko log_dir = /var/log/kolla/panko
debug = {{ panko_logging_debug }} debug = {{ panko_logging_debug }}
{% if service_name == 'panko-api' %}
# Force panko-api.log or will use app.wsgi # Force panko-api.log or will use app.wsgi
log_file = /var/log/kolla/panko/panko-api.log log_file = /var/log/kolla/panko/panko-api.log
{% endif %}
[database] [database]