<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle" type="text/css">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Alright I broke it up into smaller chunks. One problem I ran into was that the tests basically require the extension, plugin, db, and noop driver to exist so that is why the tests
are dependents of those reviews.<br>
<br>
Even though you can navigate in order through gerrit using the depdencies section, here is a list of links in order:<br>
<br>
<a href="https://review.openstack.org/#/c/105331" target="_blank">https://review.openstack.org/#/c/105331/</a><br>
<a href="https://review.openstack.org/#/c/105609/" target="_blank">https://review.openstack.org/#/c/105609/</a><br>
<a href="https://review.openstack.org/#/c/105610/" target="_blank">https://review.openstack.org/#/c/105610/</a><br>
<a href="https://review.openstack.org/#/c/105617/" target="_blank">https://review.openstack.org/#/c/105617/</a><br>
<br>
Another note: That third one is still pretty big, around 2k lines. This is because one of the new test files is around 1500 lines.<br>
<br>
Pulling down the code from the very last review should give you all the code needed to get this running (assuming the neutron config's values are pointing to the correct values).<br>
<br>
Thanks,<br>
Brandon<br>
<br>
<br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div style="direction: ltr;" id="divRpF916796"><font color="#000000" face="Tahoma" size="2"><b>From:</b> Brandon Logan [brandon.logan@RACKSPACE.COM]<br>
<b>Sent:</b> Tuesday, July 08, 2014 1:53 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [Neutron][LBaaS] LBaaS API Version 2 WIP in gerrit<br>
</font><br>
</div>
<div></div>
<div>
<div style="direction:ltr; font-family:Tahoma; color:#000000; font-size:10pt">Avishay,<br>
You're probably right about breaking it up but I wanted to get this up in gerrit ASAP. Also, I'd like to get Kyle and Mark's ideas on breaking it up.<br>
<br>
Thanks,<br>
Brandon<br>
<div style="font-family:Times New Roman; color:#000000; font-size:16px">
<hr tabindex="-1">
<div id="divRpF709301" style="direction:ltr"><font color="#000000" face="Tahoma" size="2"><b>From:</b> Susanne Balle [sleipnir012@gmail.com]<br>
<b>Sent:</b> Tuesday, July 08, 2014 9:24 AM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [Neutron][LBaaS] LBaaS API Version 2 WIP in gerrit<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr">Will take a look :-) Thanks for the huge amount of work put into this. </div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On Tue, Jul 8, 2014 at 8:48 AM, Avishay Balderman <span dir="ltr">
<<a href="mailto:AvishayB@radware.com" target="_blank">AvishayB@radware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
Hi Brandon<br>
I think the patch should be broken into few standalone sub patches.<br>
As for now it is huge and review is a challenge :)<br>
Thanks<br>
<span class="HOEnZb"><font color="#888888">Avishay<br>
</font></span>
<div class="HOEnZb">
<div class="h5"><br>
<br>
-----Original Message-----<br>
From: Brandon Logan [mailto:<a href="mailto:brandon.logan@RACKSPACE.COM" target="_blank">brandon.logan@RACKSPACE.COM</a>]<br>
Sent: Tuesday, July 08, 2014 5:26 AM<br>
To: OpenStack Development Mailing List (not for usage questions)<br>
Subject: [openstack-dev] [Neutron][LBaaS] LBaaS API Version 2 WIP in gerrit<br>
<br>
<a href="https://review.openstack.org/#/c/105331" target="_blank">https://review.openstack.org/#/c/105331</a><br>
<br>
It's a WIP and the shim layer still needs to be completed. Its a lot of code, I know. Please review it thoroughly and point out what needs to change.<br>
<br>
Thanks,<br>
Brandon<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">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" target="_blank">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>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>