<div dir="ltr">Hi folks,<div><br></div><div>That actually going in opposite direction to what flavor framework is trying to do (and for dispatching it's doing the same as providers). REST call dispatching should really go via the root object.</div>
<div><br></div><div>I don't quite get the issue with health monitors. If HM is incorrectly configured prior to association with a pool - API layer should handle that.</div><div>I don't think driver implementations should be different at constraints to HM parameters.</div>
<div><br></div><div>So I'm -1 on adding provider (or flavor) to each entity. After all, it looks just like data denormalization which actually will affect lots of API aspects in negative way.</div><div><br></div><div>
Thanks,</div><div>Eugene.</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 11, 2014 at 11:20 PM, Vijay Venkatachalam <span dir="ltr"><<a href="mailto:Vijay.Venkatachalam@citrix.com" target="_blank">Vijay.Venkatachalam@citrix.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Yes, the point was to say "the plugin need not restrict and let driver decide what to do with the API".<br>
<br>
Even if the call was made to driver instantaneously, I understand, the driver might decide to ignore<br>
first and schedule later. But, if the call is present, there is scope for validation.<br>
Also, the driver might be scheduling an async-api to backend, in which case  deployment error<br>
cannot be shown to the user instantaneously.<br>
<br>
W.r.t. identifying a provider/driver, how would it be to make tenant the default "root" object?<br>
"tenantid" is already associated with each of these entities, so no additional pain.<br>
For the tenant who wants to override let him specify provider in each of the entities.<br>
If you think of this in terms of the UI, let's say if the loadbalancer configuration is exposed<br>
as a single wizard (which has loadbalancer, listener, pool, monitor properties) then provider<br>
 is chosen only once.<br>
<br>
Curious question, is flavour framework expected to address this problem?<br>
<br>
Thanks,<br>
Vijay V.<br>
<div class="im HOEnZb"><br>
-----Original Message-----<br>
From: Doug Wiegley [mailto:<a href="mailto:dougw@a10networks.com">dougw@a10networks.com</a>]<br>
</div><div class="HOEnZb"><div class="h5">Sent: 11 August 2014 22:02<br>
To: OpenStack Development Mailing List (not for usage questions)<br>
Subject: Re: [openstack-dev] [Neutron][LBaaS] Continuing on "Calling driver interface on every API request"<br>
<br>
Hi Sam,<br>
<br>
Very true.  I think that Vijay’s objection is that we are currently imposing a logical structure on the driver, when it should be a driver decision.  Certainly, it goes both ways.<br>
<br>
And I also agree that the mechanism for returning multiple errors, and the ability to specify whether those errors are fatal or not, individually, is currently weak.<br>
<br>
Doug<br>
<br>
<br>
On 8/11/14, 10:21 AM, "Samuel Bercovici" <SamuelB@Radware.com> wrote:<br>
<br>
>Hi Doug,<br>
><br>
>In some implementations Driver !== Device. I think this is also true<br>
>for HA Proxy.<br>
>This might mean that there is a difference between creating a logical<br>
>object and when there is enough information to actually schedule/place<br>
>this into a device.<br>
>The ability to express such errors (detecting an error on a logical<br>
>object after it was created but when it actually get scheduled) should<br>
>be discussed and addressed anyway.<br>
><br>
>-Sam.<br>
><br>
><br>
>-----Original Message-----<br>
>From: Doug Wiegley [mailto:<a href="mailto:dougw@a10networks.com">dougw@a10networks.com</a>]<br>
>Sent: Monday, August 11, 2014 6:55 PM<br>
>To: OpenStack Development Mailing List (not for usage questions)<br>
>Subject: Re: [openstack-dev] [Neutron][LBaaS] Continuing on "Calling<br>
>driver interface on every API request"<br>
><br>
>Hi all,<br>
><br>
>> Validations such as ³timeout > delay² should be performed on the API<br>
>>level before it reaches the driver.<br>
>For a configuration tree (lb, listeners, pools, etc.), there should be<br>
>one provider.<br>
><br>
>You¹re right, but I think the point of Vijay¹s example was to highlight<br>
>the combo error problem with populating all of the driver objects at<br>
>once (in short, the driver interface isn¹t well suited to that model.)<br>
>That his one example can be covered by API validators is irrelevant.<br>
>Consider a backend that does not support APP_COOKIE¹s, or<br>
>HTTPS_TERMINATED (but has multiple listeners) instead.  Should the<br>
>entire load balancer create fail, or should it offer degraded service?<br>
>Do all drivers have to implement a transaction rollback; wait, the<br>
>interface makes that very hard.  That¹s his point.  The driver is no<br>
>longer just glue code between interfaces; it¹s now a mini-object error handler.<br>
><br>
><br>
>> Having provider defined in multiple places does not make sense.<br>
><br>
>Channeling Brandon, who can yell if I get this wrong, the point is not<br>
>to have a potentially different provider on each object.  It¹s to allow<br>
>a provider to be assigned when the first object in the tree is created,<br>
>so that future related objects will always get routed to the same provider.<br>
>Not knowing which provider should get all the objects is why we have to<br>
>wait until we see a LoadBalancer object.<br>
><br>
><br>
>All of this sort of edge case nonsense is because we (the royal we, the<br>
>community), wanted all load balancer objects to be ³root² objects, even<br>
>though only one of them is an actual root today, to support<br>
>many-to-many relationships among all of them, at some future date,<br>
>without an interface change.  If my bias is showing that I¹m not a fan<br>
>of adding this complexity for that, I¹m not surprised.<br>
><br>
>Thanks,<br>
>doug<br>
><br>
><br>
>On 8/11/14, 7:57 AM, "Samuel Bercovici" <SamuelB@Radware.com> wrote:<br>
><br>
>>Hi,<br>
>><br>
>>Validations such as ³timeout > delay² should be performed on the API<br>
>>level before it reaches the driver.<br>
>><br>
>>For a configuration tree (lb, listeners, pools, etc.), there should be<br>
>>one provider.<br>
>><br>
>>Having provider defined in multiple places does not make sense.<br>
>><br>
>><br>
>>-San.<br>
>><br>
>><br>
>>From: Vijay Venkatachalam [mailto:<a href="mailto:Vijay.Venkatachalam@citrix.com">Vijay.Venkatachalam@citrix.com</a>]<br>
>><br>
>>Sent: Monday, August 11, 2014 2:43 PM<br>
>>To: OpenStack Development Mailing List<br>
>>(<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>)<br>
>>Subject: [openstack-dev] [Neutron][LBaaS] Continuing on "Calling<br>
>>driver interface on every API request"<br>
>><br>
>><br>
>><br>
>>Hi:<br>
>><br>
>>Continuing from last week¹s LBaaS meetingŠ<br>
>><br>
>>Currently an entity cannot be sent to driver unless it is linked to<br>
>>loadbalancer because loadbalancer is the root object and driver<br>
>>information is only available with loadbalancer.<br>
>><br>
>><br>
>>The request to the driver is delayed because of which error<br>
>>propagation becomes tricky.<br>
>><br>
>>Let¹s say a monitor was configured with timeout > delay there would be<br>
>>no error then.<br>
>>When a listener is configured there will be a monitor<br>
>>creation/deployment error like ³timeout configured greater than delay².<br>
>><br>
>>Unless the error is very clearly crafted the user won¹t be able to<br>
>>understand the error.<br>
>><br>
>>I am half-heartedly OK with current approach.<br>
>><br>
>><br>
>>But, I would prefer Brandon¹s Solution ­ make provider an attribute in<br>
>>each of the entities to get rid of this problem.<br>
>><br>
>><br>
>>What do others think?<br>
>><br>
>>Thanks,<br>
>>Vijay V.<br>
>><br>
><br>
><br>
>_______________________________________________<br>
>OpenStack-dev mailing list<br>
><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><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>
><br>
>_______________________________________________<br>
>OpenStack-dev mailing list<br>
><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><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>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><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>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><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></div></blockquote></div><br></div>