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: Tom Fifield <883320@xxxxxxxxxxxxxxxxxx>
Date: Sat, 31 May 2014 13:37:12 -0000
Reply-to: Bug 883320 <883320@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

** No longer affects: openstack-qa

-- 
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):
  Confirmed

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