Empty or trailing comma on enabled_drivers cause Ironic-conductor to fail to start

Bug #1647783 reported by Lucas Alvares Gomes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Low
Lucas Alvares Gomes

Bug Description

Reported at: https://bugzilla.redhat.com/show_bug.cgi?id=1377103

If you have a trailing comma or an empty comma in the "enabled_drivers" configuration option the Ironic-conductor service will fail to start.

We should make it more resilient to failure.

Changed in ironic:
importance: Undecided → Low
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

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

Changed in ironic:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/407596
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=a581c1eafcf7be0663439011a24dcece2e16f04d
Submitter: Jenkins
Branch: master

commit a581c1eafcf7be0663439011a24dcece2e16f04d
Author: Lucas Alvares Gomes <email address hidden>
Date: Tue Dec 6 16:20:48 2016 +0000

    Make "enabled_drivers" config option more resilient to failures

    Do not fail to start the ironic-conductor service if there's a
    trailing comma or an empty value as part of the "enabled_drivers" or
    "enabled_network_interfaces" configuration option, instead, log a warning
    alerting the operation of the wrong syntax.

    Change-Id: Id9141c014263de5e72641613d970fd481a6aaa74
    Closes-Bug: #1647783

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 7.0.0

This issue was fixed in the openstack/ironic 7.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.