diff options
author | D G Lee <yqli2016@fiberhome.com> | 2017-06-16 10:55:48 +0800 |
---|---|---|
committer | Sam Betts <sam@code-smash.net> | 2017-09-18 12:53:31 +0000 |
commit | 56b8eae918fe525b16979eed5fbba1c525356030 (patch) | |
tree | 14add48358c021fdfbb6dc21a114cd39affd1819 /releasenotes/notes/bug-1696296-a972c8d879b98940.yaml | |
parent | 5ea8d9f3540461bfbdc5c4c62753f944ff596158 (diff) | |
download | ironic-56b8eae918fe525b16979eed5fbba1c525356030.tar.gz |
Adds more exception handling for ironic-conductor heartbeat
When heartbeat thread of ironic-conductor server is reporting heartbeat,
it will be interrupted by database exceptions except 'DBConnectionError'.
So add 'Exception' in _conductor_service_record_keepalive to catch all
possible exceptions raised from database to ensure the heartbeat thread
not to exit. And also log the exception information. When the database
recovers from an exception, heartbeat thread will continue to report
heartbeat.
Change-Id: I0dc3ada945275811ef7272d500823e0a57011e8f
Closes-Bug: #1696296
Diffstat (limited to 'releasenotes/notes/bug-1696296-a972c8d879b98940.yaml')
-rw-r--r-- | releasenotes/notes/bug-1696296-a972c8d879b98940.yaml | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/releasenotes/notes/bug-1696296-a972c8d879b98940.yaml b/releasenotes/notes/bug-1696296-a972c8d879b98940.yaml new file mode 100644 index 000000000..eb9f19212 --- /dev/null +++ b/releasenotes/notes/bug-1696296-a972c8d879b98940.yaml @@ -0,0 +1,9 @@ +--- +fixes: + - | + Fixes an issue where an ironic-conductor service was deemed dead because + the service could not report its heartbeat due to the database connection + experiencing an unexpected failure. Full tracebacks of these exceptions are + now logged, and if the database connection recovers in a reasonable amount + of time the service will still be available. + See https://bugs.launchpad.net/ironic/+bug/1696296.
\ No newline at end of file |