<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-05-20 20:05 GMT+08:00 Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There are a number of changes up for spec reviews that add parameters to<br>
LIST interfaces in Newton:<br>
<br>
* keypairs-pagination (MERGED) -<br>
<a href="https://github.com/openstack/nova-specs/blob/8d16fc11ee6d01b5a9fe1b8b7ab7fa6dff460e2a/specs/newton/approved/keypairs-pagination.rst#L2" rel="noreferrer" target="_blank">https://github.com/openstack/nova-specs/blob/8d16fc11ee6d01b5a9fe1b8b7ab7fa6dff460e2a/specs/newton/approved/keypairs-pagination.rst#L2</a><br>
* os-instances-actions - <a href="https://review.openstack.org/#/c/240401/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/240401/</a><br>
* hypervisors - <a href="https://review.openstack.org/#/c/240401/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/240401/</a><br>
* os-migrations - <a href="https://review.openstack.org/#/c/239869/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/239869/</a><br>
<br>
I think that limit / marker is always a legit thing to add, and I almost<br>
wish we just had a single spec which is "add limit / marker to the<br>
following APIs in Newton"<br>
<br></blockquote><div><br></div><div>Are you looking for code sharing or one microversion? For code sharing, it sounds ok if people have some co-work. Probably we need a common pagination supported model_query function for all of those. For one microversion, i'm a little hesitate, we should keep one small change, or enable all in one microversion. But if we have some base code for pagination support, we probably can make the pagination as default thing support for all list method?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Most of these came in with sort_keys as well. We currently don't have<br>
schema enforcement on sort_keys, so I don't think we should add any more<br>
instances of it until we scrub it. Right now sort_keys is mostly a way<br>
to generate a lot of database load because users can sort by things not<br>
indexed in your DB. We really should close that issue in the future, but<br>
I don't think we should make it any worse. I have -1s on<br>
os-instance-actions and hypervisors for that reason.<br>
<br>
os-instances-actions and os-migrations are time based, so they are<br>
proposing a changes-since. That seems logical and fine. Date seems like<br>
the natural sort order for those anyway, so it's "almost" limit/marker,<br>
except from end not the beginning. I think that in general changes-since<br>
on any resource which is time based should be fine, as long as that<br>
resource is going to natural sort by the time field in question.<br>
<br>
So... I almost feel like this should just be soft policy at this point:<br>
<br>
limit / marker - always ok<br>
sort_* - no more until we have a way to scrub sort (and we fix weird<br>
sort key issues we have)<br>
changes-since - ok on any resource that will natural sort with the<br>
updated time<br>
<br>
<br>
That should make proposing these kinds of additions easier for folks,<br>
<br>
        -Sean<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</a><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br></div></div>