Ironic python agent Conf should modify

[jiangfei]

ironic_python_agent.cmd.agent:run start. the CONF's api_url, is not find, also advertise_port ,listen_host etc.

I think the best CONF unity with other components openstack, so the CONF should be modify

[OpenStack Infra (hudson-openstack)]

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

[Jay Faulkner (jason-oldos)]

I believe this bug is saying that our config variables are currently delineated with a - (i.e. api-url) whereas most Openstack projects use _ (i.e. api_url).

I'm not sure this should actually count as a bug, but if we do fix it we must ensure to keep backwards compatibility with existing agents.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/131632
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=597d6245b159ecd46e27e60c5df6469db48b9700
Submitter: Jenkins
Branch: master

commit 597d6245b159ecd46e27e60c5df6469db48b9700
Author: jiangfei <email address hidden>
Date: Wed Oct 29 12:28:59 2014 +0800

    Use _ instead of - for config options

    Config options containing "-" don't work when reading
    configuration from a file; change these to "_". Leave
    the old options for backwards compatibility.

    Closes-Bug: #1386486

    Change-Id: Ie10a548191effa2a505309d9d4ee32003f9676a2

[OpenStack Infra (hudson-openstack)]

Change abandoned by jiangfei (<email address hidden>) on branch: master
Review: https://review.openstack.org/137550