<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Thanks for checking the changes and confirming!<div><br></div><div>BTW, my change set had a fix from Ihar (141405). Rather than approve this as-is, which would void Ihar's, I approved Ihar’s change, and then will rebase mine on his, once it is upstream (wanted to give him credit for his changes).</div><div><br></div><div>Regards,</div><div><br></div><div><br></div><div><div><div><div><div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div><div>PCM (Paul Michali)</div><div><br></div><div>MAIL …..…. <a href="mailto:pcm@cisco.com">pcm@cisco.com</a></div><div>IRC ……..… pc_m (<a href="http://irc.freenode.com">irc.freenode.com</a>)</div><div>TW ………... @pmichali</div><div>GPG Key … 4525ECC253E31A83</div><div>Fingerprint .. 307A 96BB 1A4C D2C7 931D 8D2D 4525 ECC2 53E3 1A83</div></div><div><br></div></div><br class="Apple-interchange-newline"><br class="Apple-interchange-newline">
</div>
<br><div><div>On Dec 14, 2014, at 12:31 AM, Brandon Logan <<a href="mailto:brandon.logan@RACKSPACE.COM">brandon.logan@RACKSPACE.COM</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Paul,<br>It looks like you put that method call in all the right places.  You<br>would know if you didn't, because the unit tests would fail bc of the<br>policy.json.  <br><br>Not sure on the tempests tests.  I'm sure Doug and Kyle know more about<br>that, so hopefully they can chime in.<br><br>Thanks,<br>Brandon<br><br>On Sat, 2014-12-13 at 19:06 +0000, Paul Michali (pcm) wrote:<br><blockquote type="cite">For the new VPNaaS repo, I have<br>created <a href="https://review.openstack.org/#/c/141532/">https://review.openstack.org/#/c/141532/</a> to move the tests<br>from tests.skip and modify the imports. This has Brandon’s change to<br>setup policy.json, and Ihar’s one-liner for moving get_admin_context()<br>in one test (should we upstream his, and I rebase mine?).<br><br><br>Please look it over, as I’m not sure if I put the override_nvalues()<br>calls in the right places or not.<br><br><br>It passes unit tests in Jenkins. The Tempest tests all fail. Is that<br>expected? What’s the plan for functional and tempest tests with these<br>other repos?<br><br><br>Thanks!<br><br><br><br><br>PCM (Paul Michali)<br><br><br>MAIL …..…. <a href="mailto:pcm@cisco.com">pcm@cisco.com</a><br>IRC ……..… pc_m (<a href="http://irc.freenode.com">irc.freenode.com</a>)<br>TW ………... @pmichali<br>GPG Key … 4525ECC253E31A83<br>Fingerprint .. 307A 96BB 1A4C D2C7 931D 8D2D 4525 ECC2 53E3 1A83<br><br><br><br><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>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev<br></blockquote><br>_______________________________________________<br>OpenStack-dev mailing list<br><a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev<br></blockquote></div><br></div></div></div></div></body></html>