<div dir="ltr"><div>Hi Kyle,<br><br>thanks for your feedback. Good point! I have filled the tests section.<br><br></div>cheers,<br><br>Rossella<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 14, 2013 at 5:34 PM, Kyle Mestery (kmestery) <span dir="ltr"><<a href="mailto:kmestery@cisco.com" target="_blank">kmestery@cisco.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Nov 14, 2013, at 8:46 AM, Rossella Sblendido <<a href="mailto:rossella@midokura.com">rossella@midokura.com</a>> wrote:<br>

><br>
> Hello devs,<br>
><br>
> I'd like to start working at this blueprint:<br>
><br>
> <a href="https://blueprints.launchpad.net/neutron/+spec/port-mirroring" target="_blank">https://blueprints.launchpad.net/neutron/+spec/port-mirroring</a><br>
><br>
> I didn't receive much feedback so please have a look at it.<br>
><br>
> It was not approved yet. What's the usual workflow? Shall I wait for approval before I start implementing it? Sorry for the trivial question but I'm new here.<br>
><br>
</div></div>Welcome Rossella!<br>
<br>
One issue I have with the design document linked in the blueprint is the testing section has nothing in it all. Given we're trying hard to focus on quality and testing, I'd like you to add some specific testing information to the document before we approve it. It would be ideal to have Tempest tests which test out the new API functionality as well. That's my main issue with this BP as seen now, but I'd like others to comment on this as well.<br>

<br>
Thanks!<br>
Kyle<br>
<br>
> thanks,<br>
><br>
> Rossella<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>
<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>
</blockquote></div><br></div>