yahoo-eng-team team mailing list archive

[melanie witt <1561056@xxxxxxxxxxxxxxxxxx>]

Proposed patch: https://review.openstack.org/#/c/296543/

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

** Changed in: nova
     Assignee: (unassigned) => Corey Wright (coreywright)

** Changed in: nova
   Importance: Undecided => Medium

** Changed in: nova
       Status: New => In Progress

** Changed in: nova
     Assignee: Corey Wright (coreywright) => (unassigned)

** Tags added: volumes

** Changed in: nova
     Assignee: (unassigned) => Corey Wright (coreywright)

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

Title:
  cinder volume driver's detach() causes TypeError exception on v1
  cinder client

Status in OpenStack Compute (nova):
  In Progress
Status in nova package in Ubuntu:
  New

Bug description:
  Nova version: git master branch's HEAD (as of today)
  Expected behavior: cinderclient v1 detach() called with accepted argument
  Actual behavior: cinderclient v1 detach() called with too many arguments

  Change I3cdc4992 indiscriminately passes both volume_id and
  attachment_id to the Cinder client regardless of its version even
  though Cinder client v2 supports passing volume_id and optionally
  attachment_id to its volume manager's detach() method, but v1 does
  not, only accepting volume_id.

  Calling Cinder client v1 detach() with both volume_id and
  attachment_id results in "TypeError: detach() takes exactly 2
  arguments (3 given)"

  Full traceback and proposed bug fix to follow.

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