Fix logging.conf: No section: 'logger_'

Caused by trailing comma on keys= line

Change-Id: Ie0a87642acbeb2a019f462889e8f8de84816377e
Closes-Bug: #1365677

CHANGELOG.md

@@ -4,6 +4,7 @@ This file is used to list changes made in each version of cookbook-openstack-com
 
 ## 10.1.0
 * Adding identity admin bind host endpoint to allow flexibility and consistency
+* Fix logging.conf No section error
 
 ## 10.0.3
 * Adding identity internal endpoint variables to support new endpoint usage in identity cookbook

spec/logging_spec.rb

@@ -74,7 +74,7 @@ describe 'openstack-common::logging' do
             ['[loggers]',
              'keys=root,ceilometer,cinder,glance,horizon,keystone,nova,'\
                'neutron,swift,trove,amqplib,sqlalchemy,boto,suds,eventletwsgi,'\
-               'nova_api_openstack_wsgi,nova_osapi_compute_wsgi_server,'],
+               'nova_api_openstack_wsgi,nova_osapi_compute_wsgi_server'],
             ['[logger_root]',
              'level=NOTSET',
              'handlers=devel'],

templates/default/logging.conf.erb

@@ -1,5 +1,5 @@
 [loggers]
-keys=<%= node["openstack"]["logging"]['loggers'].keys.join(',') %>,<%= node["openstack"]["logging"]["ignore"].map{|k,v| k.gsub(/\W/, '_')}.join(',') %>
+keys=<%= (node["openstack"]["logging"]['loggers'].keys + node["openstack"]["logging"]["ignore"].map{|k,v| k.gsub(/\W/, '_')}).join(',') %>
 
 [formatters]
 keys=<%= node["openstack"]["logging"]['formatters'].keys.join(',') %>