Public bug reported:

The placement/handlers/aggregate.py handler imports nova.exception and
then tries to use exception.ConcurrentUpdateDetected. Since the move of
exceptions to the placement namespace, that exception is no longer in
the module.

However, we're not seeing an error because this piece of code is very
very rarely called and we don't have tests (can) cover it.

I just happened to notice while doing a readthrough.

** Affects: nova
     Importance: Medium
     Assignee: Chris Dent (cdent)
         Status: In Progress


** Tags: placement

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

Title:
  placement aggregate handler imports wrong exception module

Status in OpenStack Compute (nova):
  In Progress

Bug description:
  The placement/handlers/aggregate.py handler imports nova.exception and
  then tries to use exception.ConcurrentUpdateDetected. Since the move
  of exceptions to the placement namespace, that exception is no longer
  in the module.

  However, we're not seeing an error because this piece of code is very
  very rarely called and we don't have tests (can) cover it.

  I just happened to notice while doing a readthrough.

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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to