<div dir="ltr">Hi Avishay,<div><br></div><div>Thanks for uploading the patch. Please see my explained comments:</div><div><br></div><div>I think there's no need to remove original plugin since it does the job and also it quite suits the goal of your "master blueprint".</div>
<div><br></div><div><i>> The noop driver is used for testing purposes and may be a good starting point for vendors who wish to implement their own driver.</i></div><div>That ok then, but in such case I'd just move noop driver in unit test directory.<br>
</div><div><div class="gmail_extra"><br></div><div class="gmail_extra"><i>> Can you please elaborate about your patch - which BP is backing it?</i></div><div class="gmail_extra">I was planning to submit my patch under your "<a href="https://blueprints.launchpad.net/quantum/+spec/multi-vendor-support-for-lbaas" target="_blank">multi-vendor-support-for-lbaas</a>" blueprint.</div>
<div class="gmail_extra">Btw, I think it's not necessary to have several separate BPs as steps could be described as work items in the "master" blueprint.</div><div class="gmail_extra"><br></div><div class="gmail_extra">
So I've not submitted my patch yet because I was waiting for consensus about some "house keeping" of services and lbaas dirs, see <a href="https://bugs.launchpad.net/quantum/+bug/1175745">https://bugs.launchpad.net/quantum/+bug/1175745</a></div>
<div class="gmail_extra">Making such change (renaming/moving) could add pain for those who maintain uncommitted patches. </div><div class="gmail_extra">I think I'll post in on review shortly, in WIP state, just to show the idea.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Eugene.<br><br><div class="gmail_quote">On Sun, May 5, 2013 at 2:40 PM, Avishay Balderman (Code Review) <span dir="ltr"><<a href="mailto:review@openstack.org" target="_blank">review@openstack.org</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">Avishay Balderman has posted comments on this change.<br>
<div class="im"><br>
Change subject: BluePrint: multi-vendor-support-for-lbaas-step0<br>
......................................................................<br>
<br>
<br>
</div>Patch Set 1:<br>
<br>
Hi Eugene<br>
As you probably know this patch is "step0" of <a href="https://blueprints.launchpad.net/quantum/+spec/multi-vendor-support-for-lbaas" target="_blank">https://blueprints.launchpad.net/quantum/+spec/multi-vendor-support-for-lbaas</a>.<br>

In the next step the original plugin will be removed and there will be only one lbaas plugin.<br>
The noop driver is used for testing purposes and may be a good starting point for vendors who wish to implement their own driver.<br>
Can you please elaborate about your patch - which BP is backing it?<br>
<div class="im"><br>
--<br>
To view, visit <a href="https://review.openstack.org/28245" target="_blank">https://review.openstack.org/28245</a><br>
To unsubscribe, visit <a href="https://review.openstack.org/settings" target="_blank">https://review.openstack.org/settings</a><br>
<br>
</div>Gerrit-MessageType: comment<br>
<div class="im">Gerrit-Change-Id: Ic5b2c46c5a74338c3fa14cc991f4420cabd7798e<br>
Gerrit-PatchSet: 1<br>
Gerrit-Project: openstack/quantum<br>
Gerrit-Branch: master<br>
Gerrit-Owner: Avishay Balderman <<a href="mailto:avishayb@radware.com">avishayb@radware.com</a>><br>
</div>Gerrit-Reviewer: Avishay Balderman <<a href="mailto:avishayb@radware.com">avishayb@radware.com</a>><br>
Gerrit-Reviewer: Jenkins<br>
Gerrit-Reviewer: enikanorov <<a href="mailto:enikanorov@mirantis.com">enikanorov@mirantis.com</a>><br>
</blockquote></div><br></div></div></div>