<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>