← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1801713] Re: Improper error message handling

 

Reviewed:  https://review.openstack.org/615560
Committed: https://git.openstack.org/cgit/openstack/compute-hyperv/commit/?id=2757ac74b7d2c85a257ed5accced1e2c7e622827
Submitter: Zuul
Branch:    master

commit 2757ac74b7d2c85a257ed5accced1e2c7e622827
Author: Lucian Petrut <lpetrut@xxxxxxxxxxxxxxxxxxxxxx>
Date:   Mon Nov 5 15:37:54 2018 +0200

    Trivial: use string for exception message
    
    Exception messages are expected to be strings. Some Nova exception
    handlers will fail when encountering wrapped exceptions.
    
    This change fixes the only occurrence of this issue.
    
    Change-Id: If35f7b99f9c0b9f0edd614dc96842549e6ac19c1
    Closes-Bug: #1801713


** Changed in: compute-hyperv
       Status: New => Fix Released

-- 
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/1801713

Title:
  Improper error message handling

Status in compute-hyperv:
  Fix Released
Status in OpenStack Compute (nova):
  In Progress

Bug description:
  Exception messages are expected to be strings, but it's not always the
  case. Some exceptions are wrapped and passed as new exception
  messages, which affects some exception handlers.

  Trace: http://paste.openstack.org/show/733939/

  Finding all the occurrences may be difficult. For now, it may be
  easier to just convert the exception messages to strings in the
  NovaException class.

To manage notifications about this bug go to:
https://bugs.launchpad.net/compute-hyperv/+bug/1801713/+subscriptions


References