Reviewed: https://review.opendev.org/656307 Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8241b47967adb792a4254eeb58fda1fc55edf314 Submitter: Zuul Branch: stable/rocky
commit 8241b47967adb792a4254eeb58fda1fc55edf314 Author: Corey Bryant <corey.bry...@canonical.com> 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 <x...@ubuntu.com> and Eric Fried <openst...@fried.cc> for helping with this patch. Conflicts: nova/virt/xenapi/agent.py NOTE(coreycb): The conflict is due to Ibe2f478288db42f8168b52dfc14d85ab92ace74b not being in stable/rocky. Change-Id: I212ac2b5ccd93e00adb7b9fe102fcb70857c6073 Partial-Bug: #1771506 (cherry picked from commit 1da71fa4ab1d7d0f580cd5cbc97f2dfd2e1c378a) (cherry picked from commit 64793cf6f77c5ba7c9ea51662d936c7545ffce8c) ** Tags added: in-stable-rocky -- 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 ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs