yahoo-eng-team team mailing list archive

Thread
Date

[Bug 883320] Re: nova.crypto. revoke_certs_by_user should handle ProcessExecutionError

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: Russell Bryant <883320@xxxxxxxxxxxxxxxxxx>
Date: Wed, 23 Jul 2014 14:58:28 -0000
Reply-to: Bug 883320 <883320@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

** Changed in: nova
       Status: Fix Committed => Fix Released

** Changed in: nova
    Milestone: None => juno-2

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/883320

Title:
  nova.crypto. revoke_certs_by_user should handle ProcessExecutionError

Status in OpenStack Compute (Nova):
  Fix Released

Bug description:
  
  revoke_cert may throw ProcessExecutionError.
  https://github.com/openstack/nova/blob/master/nova/crypto.py#L164

  This is bad pattern. 
  (See Policy of Exception Handling
  https://blueprints.launchpad.net/openstack-qa/+spec/nova-exception-policy)

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/883320/+subscriptions