yahoo-eng-team team mailing list archive

[Subhadeep De <subhadeep_de@xxxxxxxxxxxx>]

Public bug reported:

The get_all api to fetch all volumes does not pass the search options
parameter to the cinder API call. Here is the stub:

----------------
def get_all(self, context, search_opts=None):
    search_opts = search_opts or {}
    items = cinderclient(context).volumes.list(detailed=True)
    . . .
-----------------

It should have been:
------------------
def get_all(self, context, search_opts=None):
    search_opts = search_opts or {}
    items = cinderclient(context).volumes.list(detailed=True, search_opts=search_opts)
    . . .

------------------

** Affects: nova
     Importance: Undecided
     Assignee: Subhadeep De (subhadeep-de)
         Status: New

** Changed in: nova
     Assignee: (unassigned) => Subhadeep De (subhadeep-de)

** Description changed:

  The get_all api to fetch all volumes does not pass the search options
  parameter to the cinder API call. Here is the stub:
  
  ----------------
  def get_all(self, context, search_opts=None):
-     search_opts = search_opts or {}
-     items = cinderclient(context).volumes.list(detailed=True)
-     . . .
+     search_opts = search_opts or {}
+     items = cinderclient(context).volumes.list(detailed=True)
+     . . .
  -----------------
  
- Its should have been:
+ It should have been:
  ------------------
  def get_all(self, context, search_opts=None):
-     search_opts = search_opts or {}
-     items = cinderclient(context).volumes.list(detailed=True, search_opts=search_opts)
-     . . .
+     search_opts = search_opts or {}
+     items = cinderclient(context).volumes.list(detailed=True, search_opts=search_opts)
+     . . .
  
  ------------------

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

Title:
  volume api for fetching volumes does not honor search filter

Status in OpenStack Compute (Nova):
  New

Bug description:
  The get_all api to fetch all volumes does not pass the search options
  parameter to the cinder API call. Here is the stub:

  ----------------
  def get_all(self, context, search_opts=None):
      search_opts = search_opts or {}
      items = cinderclient(context).volumes.list(detailed=True)
      . . .
  -----------------

  It should have been:
  ------------------
  def get_all(self, context, search_opts=None):
      search_opts = search_opts or {}
      items = cinderclient(context).volumes.list(detailed=True, search_opts=search_opts)
      . . .

  ------------------

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