<div dir="ltr">Hi, Alex,<div><br></div><div>Thanks for the information, I was unable to join the conference yesterday. Then lets get the dicision done before fix it.</div><div><br></div><div>BR,</div><div><br></div><div>Zheng</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 12:56 PM, Alex Xu <span dir="ltr"><<a href="mailto:hejie.xu@intel.com" target="_blank">hejie.xu@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi, Zhengyu,<div><br></div><div>We discussed this in yesterday Nova API meeting. We think it should get consistent in API-WG.</div><div><br></div><div>And there already have patch for pagination guideline <a href="https://review.openstack.org/190743" target="_blank">https://review.openstack.org/190743</a> , and there also have some discussion on limits.</div><div>So we are better waiting the guideline get consistent before fix it.</div><div><br></div><div>Thanks</div><span class="HOEnZb"><font color="#888888"><div>Alex</div></font></span><div><div class="h5"><div><br><div><blockquote type="cite"><div>On Sep 23, 2015, at 9:18 AM, Zhenyu Zheng <<a href="mailto:zhengzhenyulixi@gmail.com" target="_blank">zhengzhenyulixi@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Any thoughts on this?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 14, 2015 at 11:53 AM, Zhenyu Zheng <span dir="ltr"><<a href="mailto:zhengzhenyulixi@gmail.com" target="_blank">zhengzhenyulixi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi, Thanks for your reply, after check again and I agree with you. I think we should come up with a conclusion about how we should treat this limit=0 across nova. And that's also why I sent out this mail. I will register this topic in the API meeting open discussion section, my be a BP in M to fix this.<div><br></div><div>BR,</div><div><br></div><div>Zheng</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Sep 12, 2015 at 12:07 AM, Kevin L. Mitchell <span dir="ltr"><<a href="mailto:kevin.mitchell@rackspace.com" target="_blank">kevin.mitchell@rackspace.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Fri, 2015-09-11 at 15:41 +0800, Zhenyu Zheng wrote:<br>
> Hi, I found out that nova currently handles list with limit=0 quite<br>
> different for different objects.<br>
><br>
> Especially when list servers:<br>
><br>
> According to the code:<br>
> <a href="http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/common.py#n206" rel="noreferrer" target="_blank">http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/common.py#n206</a><br>
><br>
> when limit = 0, it should apply as max_limit, but currently, in:<br>
> <a href="http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n1930" rel="noreferrer" target="_blank">http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n1930</a><br>
><br>
> we directly return [], this is quite different with comment in the api<br>
> code.<br>
><br>
><br>
> I checked other objects:<br>
><br>
> when list security groups and server groups, it will return as no<br>
> limit has been set. And for flavors it returns []. I will continue to<br>
> try out other APIs if needed.<br>
><br>
> I think maybe we should make a rule for all objects, at least fix the<br>
> servers to make it same in api and db code.<br>
><br>
> I have reported a bug in launchpad:<br>
><br>
> <a href="https://bugs.launchpad.net/nova/+bug/1494617" rel="noreferrer" target="_blank">https://bugs.launchpad.net/nova/+bug/1494617</a><br>
><br>
><br>
> Any suggestions?<br>
<br>
</span>After seeing the test failures that showed up on your proposed fix, I'm<br>
thinking that the proposed change reads like an API change, requiring a<br>
microversion bump.  That said, I approve of increased consistency across<br>
the API, and perhaps the behavior on limit=0 is something the API group<br>
needs to discuss a guideline for?<br>
<span><font color="#888888">--<br>
Kevin L. Mitchell <<a href="mailto:kevin.mitchell@rackspace.com" target="_blank">kevin.mitchell@rackspace.com</a>><br>
Rackspace<br>
<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></div></blockquote></div><br></div>
__________________________________________________________________________<br>OpenStack Development Mailing List (not for usage questions)<br>Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" target="_blank">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div></blockquote></div><br></div></div></div></div><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>
<br></blockquote></div><br></div>