Reviewed: https://review.opendev.org/656308 Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=5b0adaa0ca5f757bb224d1ffac0c6705b03ee2ed Submitter: Zuul Branch: stable/queens
commit 5b0adaa0ca5f757bb224d1ffac0c6705b03ee2ed Author: Corey Bryant <[email protected]> Date: Thu Feb 7 10:12:54 2019 -0500 xenapi/agent: Change openssl error handling Prior to this patch, if the openssl command returned a zero exit code and wrote details to stderr, nova would raise a RuntimeError exception. This patch changes the behavior to only raise a RuntimeError exception when openssl returns a non-zero exit code. Regardless of the exit code a warning will always be logged with stderr details if stderr is not None. Note that processutils.execute will now raise a processutils.ProcessExecutionError exception for any non-zero exit code since we are passing check_exit_code=True, which we convert to a Runtime error. Thanks to Dimitri John Ledkov <[email protected]> and Eric Fried <[email protected]> for helping with this patch. Conflicts: nova/virt/xenapi/agent.py NOTE(coreycb): The conflict is due to Ibe2f478288db42f8168b52dfc14d85ab92ace74b not being in stable/queens. Change-Id: I212ac2b5ccd93e00adb7b9fe102fcb70857c6073 Partial-Bug: #1771506 (cherry picked from commit 1da71fa4ab1d7d0f580cd5cbc97f2dfd2e1c378a) (cherry picked from commit 64793cf6f77c5ba7c9ea51662d936c7545ffce8c) (cherry picked from commit 82de38ad4ce86c5398538a8635713a86407216d0) ** Tags added: in-stable-queens -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1771506 Title: Unit test failure with OpenSSL 1.1.1 To manage notifications about this bug go to: https://bugs.launchpad.net/cloud-archive/+bug/1771506/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
