<div dir="ltr">Replies inline.<div><br><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 6, 2015 at 1:48 PM, Salvatore Orlando <span dir="ltr"><<a href="mailto:salv.orlando@gmail.com" target="_blank">salv.orlando@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">More comments inline.<div>I shall stop trying being ironic (pun intended) in my posts.</div></div></blockquote><div><br></div><div>:(</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Salvatore<br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On 5 November 2015 at 18:37, Kyle Mestery <span dir="ltr"><<a href="mailto:mestery@mestery.com" target="_blank">mestery@mestery.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Thu, Nov 5, 2015 at 10: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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On 11/04/2015 04:21 PM, Shraddha Pandhe wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hi Salvatore,<br>
<br>
Thanks for the feedback. I agree with you that arbitrary JSON blobs will<br>
make IPAM much more powerful. Some other projects already do things like<br>
this.<br>
</blockquote>
<br>
:( Actually, though "powerful" it also leads to implementation details leaking directly out of the public REST API. I'm very negative on this and would prefer an actual codified REST API that can be relied on regardless of backend driver or implementation.<br></blockquote><div><br></div></span><div>I agree with Jay here. We've had people propose similar things in Neutron before, and I've been against them. The entire point of the Neutron REST API is to not leak these details out. It dampens the strength of the logical model, and it tends to have users become reliant on backend implementations.<br></div></div></div></div></blockquote><div><br></div></span><div>I see I did not manage to convey accurately irony and sarcasm in my previous post ;)</div><div>The point was that thanks to a blooming number of extensions the Neutron API is already hardly portable. Blob attributes (or dict attributes, or key/value list attributes, or whatever does not have a precise schema) are a nail in the coffin, and also violate the only tenet Neutron has somehow managed to honour, which is being backend agnostic.</div><div>And the fact that the port binding extension is pretty much that is not a valid argument, imho.</div><div>On the other hand, I'm all in for extending DB schema and driver logic to suit all IPAM needs; at the end of the day that's what do with plugins for all sort of stuff.</div></div></div></div></div></blockquote><div><br></div><div><br></div><div>Agreed. Filed an rfe bug: <a href="https://bugs.launchpad.net/neutron/+bug/1513981">https://bugs.launchpad.net/neutron/+bug/1513981</a>. Spec coming up for review.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><span class=""><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> <br></div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
e.g. In Ironic, node has driver_info, which is JSON. it also has an<br>
'extras' arbitrary JSON field. This allows us to put any information in<br>
there that we think is important for us.<br>
</blockquote>
<br>
Yeah, and this is a bad thing, IMHO. Public REST APIs should be structured, not a Wild West free-for-all. The biggest problem with using free-form JSON blobs in RESTful APIs like this is that you throw away the ability to evolve the API in a structured, versioned way. Instead of evolving the API using microversions, instead every vendor just jams whatever they feel like into the JSON blob over time. There's no way for clients to know what the server will return at any given time.<br>
<br>
Achieving consensus on a REST API that meets the needs of a variety of backend implementations is *hard work*, yes, but it's what we need to do if we are to have APIs that are viewed in the industry as stable, discoverable, and reliably useful.<br></blockquote><div><br></div></span><div>++, this is the correct way forward.<br></div></div></div></div></blockquote><div><br></div></span><div>Cool, but let me point out that experience has thought us that anything that is a result of a compromise between several parties following different agendas is bound to failure as it does not fully satisfy the requirements of any stakeholder.</div><div>If these information are needed for making scheduling decisions based on network requirements, then it makes sense to expose this information also at the API layer (I assume there also plans for making the scheduler *seriously* network aware). However, this information should have a well-defined schema with no leeway for 'extensions; such schema can evolve over time.</div><div><div class="h5"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Thanks,<br></div><div>Kyle<br> <br></div><div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Best,<br>
-jay<br>
<br>
Best,<br>
-jay<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hoping to get some positive feedback from API and DB lieutenants too.<br>
<br>
<br>
On Wed, Nov 4, 2015 at 1:06 PM, Salvatore Orlando<br>
<<a href="mailto:salv.orlando@gmail.com" target="_blank">salv.orlando@gmail.com</a> <mailto:<a href="mailto:salv.orlando@gmail.com" target="_blank">salv.orlando@gmail.com</a>>> wrote:<br>
<br>
Arbitrary blobs are a powerful tools to circumvent limitations of an<br>
API, as well as other constraints which might be imposed for<br>
versioning or portability purposes.<br>
The parameters that should end up in such blob are typically<br>
specific for the target IPAM driver (to an extent they might even<br>
identify a specific driver to use), and therefore an API consumer<br>
who knows what backend is performing IPAM can surely leverage it.<br>
<br>
Therefore this would make a lot of sense, assuming API portability<br>
and not leaking backend details are not a concern.<br>
The Neutron team API & DB lieutenants will be able to provide more<br>
input on this regard.<br>
<br>
In this case other approaches such as a vendor specific extension<br>
are not a solution - assuming your granularity level is the<br>
allocation pool; indeed allocation pools are not first-class neutron<br>
resources, and it is not therefore possible to have APIs which<br>
associate vendor specific properties to allocation pools.<br>
<br>
Salvatore<br>
<br>
On 4 November 2015 at 21:46, Shraddha Pandhe<br>
<<a href="mailto:spandhe.openstack@gmail.com" target="_blank">spandhe.openstack@gmail.com</a> <mailto:<a href="mailto:spandhe.openstack@gmail.com" target="_blank">spandhe.openstack@gmail.com</a>>><br>
wrote:<br>
<br>
Hi folks,<br>
<br>
I have a small question/suggestion about IPAM.<br>
<br>
With IPAM, we are allowing users to have their own IPAM drivers<br>
so that they can manage IP allocation. The problem is, the new<br>
ipam tables in the database have the same columns as the old<br>
tables. So, as a user, if I want to have my own logic for ip<br>
allocation, I can't actually get any help from the database.<br>
Whereas, if we had an arbitrary json blob in the ipam tables, I<br>
could put any useful information/tags there, that can help me<br>
for allocation.<br>
<br>
Does this make sense?<br>
<br>
e.g. If I want to create multiple allocation pools in a subnet<br>
and use them for different purposes, I would need some sort of<br>
tag for each allocation pool for identification. Right now,<br>
there is no scope for doing something like that.<br>
<br>
Any thoughts? If there are any other way to solve the problem,<br>
please let me know<br>
<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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
<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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
<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>
<br>
</blockquote>
<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>
</blockquote></div></div></div><br></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></div></div><br></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></div></div></div>