Re: [openstack-dev] [Cinder] Get volumes REST API with filters and limit

2014-03-04 Thread Duncan Thomas
Definitely file a bug... a script to reproduce would be fantastic.
Needs to be fixed... I don't think you need a blueprint if the fix is
simple, but if you're making deep changes then a blueprint always
helps.#

Thanks for pointing this out

On 28 February 2014 20:52, Steven Kaufer kau...@us.ibm.com wrote:
 I am investigating some pagination enhancements in nova and cinder (see nova
 blueprint https://blueprints.launchpad.net/nova/+spec/nova-pagination).

 In cinder, it appears that all filtering is done after the volumes are
 retrieved from the database (see the API.get_all function in
 https://github.com/openstack/cinder/blob/master/cinder/volume/api.py).
 Therefore, the usage combination of filters and limit will only work if all
 volumes matching the filters are in the page of data being retrieved from
 the database.

 For example, assume that all of the volumes with a name of foo would be
 retrieved from the database starting at index 100 and that you query for all
 volumes with a name of foo while specifying a limit of 50.  In this case,
 the query would yield 0 results since the filter did not match any of the
 first 50 entries retrieved from the database.

 Is this a known problem?
 Is this considered a bug?
 How should this get resolved?  As a blueprint for juno?

 I am new to the community and am trying to determine how this should be
 addressed.

 Thanks,

 Steven Kaufer


 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev




-- 
Duncan Thomas

___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


Re: [openstack-dev] [Cinder] Get volumes REST API with filters and limit

2014-03-04 Thread Steven Kaufer
Duncan Thomas duncan.tho...@gmail.com wrote on 03/04/2014 07:53:49 AM:

 From: Duncan Thomas duncan.tho...@gmail.com
 To: OpenStack Development Mailing List (not for usage questions)
 openstack-dev@lists.openstack.org,
 Date: 03/04/2014 08:06 AM
 Subject: Re: [openstack-dev] [Cinder] Get volumes REST API with
 filters and limit

 Definitely file a bug... a script to reproduce would be fantastic.
 Needs to be fixed... I don't think you need a blueprint if the fix is
 simple, but if you're making deep changes then a blueprint always
 helps.#

Bug created: https://bugs.launchpad.net/cinder/+bug/1287813


 Thanks for pointing this out

 On 28 February 2014 20:52, Steven Kaufer kau...@us.ibm.com wrote:
  I am investigating some pagination enhancements in nova and cinder(see
nova
  blueprint https://blueprints.launchpad.net/nova/+spec/nova-pagination).
 
  In cinder, it appears that all filtering is done after the volumes are
  retrieved from the database (see the API.get_all function in
  https://github.com/openstack/cinder/blob/master/cinder/volume/api.py).
  Therefore, the usage combination of filters and limit will only work if
all
  volumes matching the filters are in the page of data being retrieved
from
  the database.
 
  For example, assume that all of the volumes with a name of foo would
be
  retrieved from the database starting at index 100 and that you query
for all
  volumes with a name of foo while specifying a limit of 50.  In this
case,
  the query would yield 0 results since the filter did not match any of
the
  first 50 entries retrieved from the database.
 
  Is this a known problem?
  Is this considered a bug?
  How should this get resolved?  As a blueprint for juno?
 
  I am new to the community and am trying to determine how this should be
  addressed.
 
  Thanks,
 
  Steven Kaufer
 
 
  ___
  OpenStack-dev mailing list
  OpenStack-dev@lists.openstack.org
  http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
 



 --
 Duncan Thomas

 ___
 OpenStack-dev mailing list
 OpenStack-dev@lists.openstack.org
 http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Thanks,

Steven Kaufer___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


[openstack-dev] [Cinder] Get volumes REST API with filters and limit

2014-02-28 Thread Steven Kaufer


I am investigating some pagination enhancements in nova and cinder (see
nova blueprint
https://blueprints.launchpad.net/nova/+spec/nova-pagination).

In cinder, it appears that all filtering is done after the volumes are
retrieved from the database (see the API.get_all function in
https://github.com/openstack/cinder/blob/master/cinder/volume/api.py).
Therefore, the usage combination of filters and limit will only work if all
volumes matching the filters are in the page of data being retrieved from
the database.

For example, assume that all of the volumes with a name of foo would be
retrieved from the database starting at index 100 and that you query for
all volumes with a name of foo while specifying a limit of 50.  In this
case, the query would yield 0 results since the filter did not match any of
the first 50 entries retrieved from the database.

Is this a known problem?
Is this considered a bug?
How should this get resolved?  As a blueprint for juno?

I am new to the community and am trying to determine how this should be
addressed.

Thanks,

Steven Kaufer___
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev