<div dir="ltr">Thanks for the information, really hope these two can get merged for Newton:<div> <a href="https://review.openstack.org/#/c/240401/" rel="noreferrer" target="_blank" style="font-size:12.8px">https://review.openstack.org/#/c/240401/</a></div><div> <a href="https://review.openstack.org/#/c/239869/" rel="noreferrer" target="_blank" style="font-size:12.8px">https://review.openstack.org/#/c/239869/</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 21, 2016 at 5:55 AM, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 on all your suggestions below, Sean.<span class="HOEnZb"><font color="#888888"><br>
<br>
-jay</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 05/20/2016 08:05 AM, Sean Dague wrote:<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>
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>
<br>
</blockquote>
<br></div></div><div class="HOEnZb"><div class="h5">
__________________________________________________________________________<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>
</div></div></blockquote></div><br></div>