Baremetal API requres that ironic options is configured in nova.conf on nova-api node

[Vasyl Saienko]

At the moment ironic section is configured from nova:compute:ironic class [0].
If nova-compute and nova-api are on the different nodes. ironic section is missing in nova.conf on
nova-api node. As result baremetal api doesn't work in nova.

The proper way to fix this issue is to configure ironic options from nova:api class as well.

[0] https://github.com/openstack/puppet-nova/blob/master/manifests/compute/ironic.pp

[Emilien Macchi (emilienm)]

it's the same issue as vncproxy, I prefer the option of using a class, let's say nova::ironic::comon (we could also have later nova::ironic::compute and nova::ironic::api for specific options), and include the common classe (+ the specific classes if needed) in nova::compute and nova::api.

What do you think?

[Denis Egorenko (degorenko)]

+1 for Emilien suggestion

[Vasyl Saienko (vsaienko)]

Sounds good

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/317593
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=2f0f24c5e7c75d160452020f0445dea122dd1ad5
Submitter: Jenkins
Branch: master

commit 2f0f24c5e7c75d160452020f0445dea122dd1ad5
Author: Andrey Shestakov <email address hidden>
Date: Tue May 17 18:49:51 2016 +0300

    Implement nova::ironic::common class

    This change introduces new class nova::ironic::common which adds common
    Ironic related parameters to nova.conf required for nova-api and nova-compute.

    Change-Id: Ia878d19cffd833e9bf95d1b26ff0c2b6c02370d0
    Closes-bug: #1581040

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/319344

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/319344
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=abb20ed24542350bc062c8295b7e9c596b2a4afb
Submitter: Jenkins
Branch: stable/mitaka

commit abb20ed24542350bc062c8295b7e9c596b2a4afb
Author: Andrey Shestakov <email address hidden>
Date: Tue May 17 18:49:51 2016 +0300

    Implement nova::ironic::common class

    This change introduces new class nova::ironic::common which adds common
    Ironic related parameters to nova.conf required for nova-api and nova-compute.

    Change-Id: Ia878d19cffd833e9bf95d1b26ff0c2b6c02370d0
    Closes-bug: #1581040
    (cherry picked from commit 2f0f24c5e7c75d160452020f0445dea122dd1ad5)

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/puppet-nova 8.1.0 release.

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/puppet-nova 9.0.0 release.