<div dir="ltr"><div>Thanks a lot for the reply.</div><div><br></div>I have already registered a BP for this, and will submit a spec for N, we can discuss the details in the spec then.<div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 6, 2016 at 2:01 AM, Matt Riedemann <span dir="ltr"><<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 3/5/2016 9:48 AM, Adam Young wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 03/05/2016 12:27 AM, Chris Friesen wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 03/04/2016 03:42 PM, Matt Riedemann wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
On 3/3/2016 9:14 PM, Zhenyu Zheng wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hm, I found out the reason:<br>
<a href="https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L1139-L1145" rel="noreferrer" target="_blank">https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L1139-L1145</a><br>
<br>
<br>
here we filtered out parameters like "deleted", and that's why the API<br>
behavior is like above mentioned.<br>
<br>
So should we simple add "deleted" to the tuple or a microversion is<br>
needed?<br>
</blockquote></blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Nice find. This is basically the same as the ip6 case which required<br>
microversion 2.5 [1]. So I think this is going to require a<br>
microversion in<br>
Newton to fix it (which means a blueprint and a spec since it's an<br>
API change).<br>
But it's pretty trivial, the paperwork is the majority of the work.<br>
<br>
[1] <a href="https://review.openstack.org/#/c/179569/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/179569/</a><br>
</blockquote>
<br>
Does it really need a spec given that microversions are documented in<br>
the codebase?<br>
<br>
That almost seems like paperwork for the sake of following the rules<br>
rather than to serve any useful purpose.<br>
<br>
Is anyone really going to argue about the details?<br>
<br>
</blockquote>
<br>
I've been lurking on this discussion. I was worried that you were going<br>
to try to hard code "admin" somewhere in here.<br>
<br>
If the only change is that the currently accepted set of params is<br>
enforced with the current set of policy rules, just in a slightly<br>
different place, it will not break people, and thus I would think no<br>
microversion is essential.  However, if the the user might need to test<br>
which way policy is enforced in order to use the right code path when<br>
doing a client call, then a microversion would be needed.<br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Chris<br>
<br>
<br>
__________________________________________________________________________<br>
<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe:<br>
<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>
</blockquote>
<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>
<br>
</blockquote>
<br></div></div>
The ip6 case and microversion 2.5 is exactly the same scenario and sets precedent here, so yes we need a microversion which makes it an API change which according to our policy requires a spec. I realize it's trivial, but them's the rules.<br>
<br>
As far as I can tell, this is latent behavior since forever and no one has freaked out about it before, so I don't think doing things by the book and doing that in Newton is going to cause any problems.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt Riedemann</font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div>