<div dir="ltr"><div><div><div><div><div>FYI,<br><br></div>numan created a bug [1] about being able to run functional test job in neutron-vpnaas CI<br></div>I've proposed a patch [2] which simply add <span class="">neutron</span><span class="">-</span><span class="">dsvm</span><span class="">-</span><span class="">functional job in check and gate queue of neutron-vpnaas.<br><br></span></div><span class="">Unfortunately, as discussed with marun on IRC, this won't be enough, since this job depends on hook scripts hosted in the neutron repository</span> [3].<br><br></div>This issue will impact all advanced services which want to run functional test.<br><br></div><div>I will try to investigate this deeper, but any though on this issue would be appreciated<br></div><div><div><div><div><br>[1]<a href="https://bugs.launchpad.net/openstack-ci/+bug/1412770">https://bugs.launchpad.net/openstack-ci/+bug/1412770</a><br>[2]<a href="https://review.openstack.org/#/c/148616/">https://review.openstack.org/#/c/148616/</a><br>[3]<a href="https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/neutron-functional.yaml">https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/neutron-functional.yaml</a><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 20, 2015 at 9:02 PM, Kevin Benton <span dir="ltr"><<a href="mailto:blak111@gmail.com" target="_blank">blak111@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I don't believe we have any unit tests that create namespaces or veth pairs. This sounds like it belongs with functional tests.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Tue, Jan 20, 2015 at 10:20 AM, Numan Siddique <span dir="ltr"><<a href="mailto:numan.siddique@enovance.com" target="_blank">numan.siddique@enovance.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
<div bgcolor="#FFFFFF" text="#000000">
<font face="monospace">Hello,<br>
<br>
I am working on a bug [1] on neutron vpnaas and submitted the
patch here [2].<br>
<br>
The test code to test the fix does the following<br>
- creates a namespace<br>
- creates a veth pair and add one interface into the namespace<br>
- configures the interface with an ip address and<br>
- adds a default gateway<br>
- and of course tests the code.<br>
<br>
This test code only tests a specific function (</font><font face="monospace">
<span style="color:rgb(102,0,102);font-family:monospace;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;display:inline!important;float:none;background-color:rgb(255,255,255)">OpenSwanProcess.</span>_get_nexthop())<br>
</font><font face="monospace"><font face="monospace"><br>
Reviewers of this patch are not clear if this should be part of
functional tests or unit tests.<br>
</font>Can unit tests create linux namespaces, interfaces etc or
it falls under functional tests?<br>
<br>
Please let me know your thoughts on this.<br>
<br>
[1] - <a href="https://bugs.launchpad.net/neutron/+bug/1405413" target="_blank">https://bugs.launchpad.net/neutron/+bug/1405413</a><br>
[2] - <a href="https://review.openstack.org/#/c/145005/5" target="_blank">https://review.openstack.org/#/c/145005/5</a><br>
<br>
<br>
Regards<span><font color="#888888"><br>
Numan<br>
<br>
</font></span></font>
</div>
<br></div></div>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><div>Kevin Benton</div></div>
</font></span></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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></blockquote></div><br></div></div>