yahoo-eng-team team mailing list archive

[Eric Harney <1211338@xxxxxxxxxxxxxxxxxx>]

Fixed in Cinder master (in 2013.2):
https://review.openstack.org/#/c/44532/

** Also affects: cinder
   Importance: Undecided
       Status: New

** Changed in: cinder
       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/1211338

Title:
  "Direct" vs. "direct" in impl_qpid

Status in Cinder:
  Fix Released
Status in OpenStack Compute (Nova):
  Fix Released
Status in OpenStack Compute (nova) grizzly series:
  New
Status in Oslo - a Library of Common OpenStack Code:
  Fix Released

Bug description:
  impl_qpid.py has {"type": "Direct"} (with a capital D) in one place.
  "direct" (lowercase) in others.  It appears that qpid is case-
  sensitive about exchange types, so the version with the capital D is
  invalid.  This ends up causing qpid to throw an error like:

  >> "/usr/lib/python2.6/site-packages/qpid/messaging/endpoints.py",
  >> line 567, in _ewait\n    self.check_error()\n', '  File
  >> "/usr/lib/python2.6/site-packages/qpid/messaging/endpoints.py",
  >> line 556, in check_error\n    raise self.error\n', 'NotFound:
  >> not-found: Exchange type not implemented: Direct
  >> (qpid/broker/SessionAdapter.cpp:117)(404)\n']

  It should be a one-character fix.

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