<div dir="ltr">Hi Avishay, lbaas folks,<div><br></div><div>I've reviewed the wiki and have some questions/suggestions:</div><div><br></div><div>1) Looks like L7Policy is lacking 'name' attribute in it's description. However i see little benefit of having a name for this object</div>
<div>2) lbaas-related neutron client commands start with lb-, please fix this.<br></div><div>3) How does L7Policy specifies relation of vip and pool?</div><div>4) How default pool will be associated with the vip? Will it be a l7 rule of special kind?<br>
</div><div>It is not quite clear what does mean associating vip with pool with policy if each rule in the policy contains <font face="arial, helvetica, sans-serif" color="#000000" style="background-color:rgb(255,255,255)">'<span style="font-size:13px;line-height:20px;white-space:pre-wrap">SelectedPool' attribute.</span></font></div>
<div><font face="arial, helvetica, sans-serif" color="#000000" style="background-color:rgb(255,255,255)"><span style="font-size:13px;line-height:20px;white-space:pre-wrap">5) what is 'action'? What other actions except SELECT_POOL can be?</span></font></div>
<div><br></div><div>In fact my suggestion will be slightly different:</div><div>- Instead of having L7Policy which i believe serves only to rules grouping, we introduce VipPoolAssociation as follows:</div><div><br></div><div>
VipPoolAssociation<br></div><div>     id,</div><div>     vip_id,</div><div>     pool_id,</div><div>     default - boolean flag for default pool for the vip</div><div><br></div><div>L7Rule then will have vippoolassociation_id (it's better to shorten the attr name) just like it has policy_id in your proposal. L7Rule doesn't need SelectedPool attribute since once it attached to VipPoolAssociation, the rule then points to the pool with pool_id of that association.</div>
<div>In other words, VipPoolAssociation is almost the same as L7Policy but named closer to it's primary goal of associating vips and pools.</div><div><br></div><div>I would also suggest to add add/delete-rule operation for the association.</div>
<div><br></div><div>What do you think?</div><div><br></div><div>Thanks,</div><div>Eugene.</div><div><br></div></div>