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>
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)
Reviewed: https:/ /review. openstack. org/491835 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=c46f037a0f5 7710553b5ff8f02 bc08ebedf3c5fb
Committed: https:/
Submitter: Jenkins
Branch: stable/ocata
commit c46f037a0f57710 553b5ff8f02bc08 ebedf3c5fb
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: I4ebebcf221dfe2 9e2aa125f5956ae c10108f8fbe 8d48553233b41bf 9119d834fb)
(cherry picked from commit c2d33c327137035