script tools/config/generate_sample.sh not working with error: Error in option "pecan_debug"

[Vadim Rovachev]

Currently config generator will fail for the ceilometer due to the fact we're using global debug option value interpolation. Generator does not understand that type of this interpolated value is Bool and raises the error.

If added 'set -x' to script, then output is:
http://paste.openstack.org/show/114918/

If added 'raise' to 305 line in ceilometer/openstack/common/config/generator.py and run:
---------------------------------
python -m ceilometer.openstack.common.config.generator -m keystoneclient.middleware.auth_token -l oslo.messaging ceilometer/alarm/notifier/rest.py ceilometer/alarm/rpc.py ceilometer/alarm/service.py ceilometer/api/app.py ceilometer/api/controllers/v2.py ceilometer/api/__init__.py ceilometer/central/manager.py ceilometer/cli.py ceilometer/cmd/alarm.py ceilometer/collector.py ceilometer/compute/discovery.py ceilometer/compute/notifications/__init__.py ceilometer/compute/util.py ceilometer/compute/virt/inspector.py ceilometer/compute/virt/libvirt/inspector.py ceilometer/compute/virt/vmware/inspector.py ceilometer/compute/virt/xenapi/inspector.py ceilometer/coordination.py ceilometer/data_processing/notifications.py ceilometer/dispatcher/file.py ceilometer/dispatcher/__init__.py ceilometer/energy/kwapi.py ceilometer/event/converter.py ceilometer/hardware/discovery.py ceilometer/identity/notifications.py ceilometer/image/glance.py ceilometer/image/notifications.py ceilometer/ipmi/notifications/ironic.py ceilometer/ipmi/platform/intel_node_manager.py ceilometer/middleware.py ceilometer/network/notifications.py ceilometer/neutron_client.py ceilometer/notification.py ceilometer/nova_client.py ceilometer/objectstore/swift.py ceilometer/openstack/common/eventlet_backdoor.py ceilometer/openstack/common/lockutils.py ceilometer/openstack/common/log.py ceilometer/openstack/common/policy.py ceilometer/orchestration/notifications.py ceilometer/pipeline.py ceilometer/profiler/notifications.py ceilometer/publisher/messaging.py ceilometer/publisher/utils.py ceilometer/sample.py ceilometer/service.py ceilometer/storage/__init__.py ceilometer/utils.py ceilometer/volume/notifications.py
---------------------------------
then outus is:
http://paste.openstack.org/show/114924/

This bug reproduced, because in https://github.com/openstack/ceilometer/blob/1ac00d4a705d0a0adc761d7f9559c69bf3748f07/ceilometer/api/app.py#L45
default parameter is not boot variable.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/123914

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/123705
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=f92ddd636d5d764949202d376597cb85c399d467
Submitter: Jenkins
Branch: master

commit f92ddd636d5d764949202d376597cb85c399d467
Author: Dina Belova <email address hidden>
Date: Wed Sep 24 15:26:24 2014 +0400

    Fix bug with wrong bool opt value interpolation

    Other way of global debug opt value interpolation is needed
    for the pecan_debug option, as bool opt interpolation
    seems to be not working in the oslo.config

    Closes-Bug: 1373360
    Change-Id: I77c4d02b0821b559d7de7329acf11c6d490f51b1

[Davanum Srinivas (DIMS) (dims-v)]

The bug shows up in oslo-incubator version of generator.py, at the moment we are so close to juno and this fix can go only in stable/juno branch. However, the ceilometer team has a work around and no one else needs it. So marking this as invalid in oslo-incubator unless someone else hollers.

Related note, oslo.config's copy of generator.py is *very* lenient and has to be tightened up to validate the default values properly AND as a consequence the template string has to be handled as well. I'll be doing this in:
https://review.openstack.org/123914

[OpenStack Infra (hudson-openstack)]

Change abandoned by Davanum Srinivas (dims) (<email address hidden>) on branch: master
Review: https://review.openstack.org/123914

[Doug Hellmann (doug-hellmann)]

Is this still a problem with the latest config generator code in oslo.config?

[Ben Nemec (bnemec)]

This was reported years ago against the incubator version of the generator, and there's no report I can see that says it still exists in oslo.config. I'm going to close this, but feel free to reopen if this is still happening.