Comment 11 for bug 1706772

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/ocata)

Reviewed: https://review.openstack.org/491835
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c46f037a0f57710553b5ff8f02bc08ebedf3c5fb
Submitter: Jenkins
Branch: stable/ocata

commit c46f037a0f57710553b5ff8f02bc08ebedf3c5fb
Author: EdLeafe <email address hidden>
Date: Wed Jul 26 23:07:38 2017 +0000

    Handle ironicclient failures in Ironic driver

    The ironic driver uses ironicclient to get the list of nodes from the
    Ironic service. If that service is not running, or something otherwise
    prevents the ironicclient's call from succeeding, and unhandled
    exception will occur. This patch adds a generic exception handler for
    this call, logging an error and continuing.

    Co-Authored-By: Lucas Alvares Gomes <email address hidden>

    Closes-Bug: #1706772

    Conflicts:
          nova/tests/unit/virt/ironic/test_driver.py

    NOTE(mriedem): The conflicts in the tests can be ignored since they
    were just around some boot from volume and flavor migration tests
    in Pike which don't exist in Ocata. Also, the _LE translation marker
    has to be added back in for LOG.error in Ocata due to hacking checks.

    Change-Id: I4ebebcf221dfe29e2aa125f5956aec10108f8fbe
    (cherry picked from commit c2d33c3271370358d48553233b41bf9119d834fb)