<div dir="ltr">Hi Mario,<div><br></div><div>Here is the problem:</div><div><div>>>> import netaddr</div><div>>>> p = netaddr.IPNetwork('2001:0db8::/64')</div><div>>>> str(p)</div><div>'2001:db8::/64'</div>

</div><div><br></div><div>Now that you are converting CIDR strings to netaddr objects and back, it's causing redundant info to be dropped. You will just have to remove the references to 2001:0db8::/64 and replace them with 2001:db8::/64.</div>

<div><br></div><div>I will comment on the review as well.</div><div><br></div><div>Cheers,</div><div>Kevin Benton</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 14, 2014 at 9:23 AM, <a href="mailto:marios@redhat.com">marios@redhat.com</a> <span dir="ltr"><<a href="mailto:mandreou@redhat.com" target="_blank">mandreou@redhat.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I am really stumped by a Jenkins failure for one of my reviews... @<br>
<a href="https://review.openstack.org/#/c/59212/" target="_blank">https://review.openstack.org/#/c/59212/</a> if any kind soul has any<br>
pointers/help I will be very grateful. The strange thing is that Jenkins<br>
+1 this patchset (Apr 2) but subsequently failed as described below:<br>
<br>
The failure is from<br>
neutron.tests.unit.test_security_groups_rpc.SGServerRpcCallBackMixinTestCase<br>
and specifically the 2 test cases<br>
test_security_group_rules_for_devices_ipv6_ingress and<br>
test_security_group_rules_for_devices_ipv6_egress<br>
<br>
The failure is on the assertion at the end of the methods:<br>
<br>
 self.assertEqual(port_rpc['security_group_rules'],expected)<br>
<br>
port_rpc['security_group_rules'] ---> {'ethertype': u'IPv6',<br>
'direction': u'egress'<br>
expected                         ---> {'ethertype': 'IPv6', 'direction':<br>
'egress'<br>
<br>
(you can see this in the logs, e.g.<br>
<a href="http://logs.openstack.org/12/59212/14/check/gate-neutron-python27/57d89d5/console.html.gz" target="_blank">http://logs.openstack.org/12/59212/14/check/gate-neutron-python27/57d89d5/console.html.gz</a>)<br>
<br>
<br>
I have *no* idea what in my code is causing this behaviour; if I just<br>
grab this review, the tests pass fine. But if I rebase against master,<br>
they fail, i.e.:<br>
<br>
git review -d I71fb8c887963a122a5bd8cfdda800026c1cd3954<br>
source .tox/py27/bin/activate<br>
./run_tests.sh -d<br>
neutron.tests.unit.test_security_groups_rpc.SGServerRpcCallBackMixinTestCase<br>
<br>
...<br>
<br>
Ran 7 tests in 1.926s<br>
OK<br>
<br>
...<br>
<br>
git rebase master<br>
./run_tests.sh -d<br>
neutron.tests.unit.test_security_groups_rpc.SGServerRpcCallBackMixinTestCase<br>
<br>
...<br>
<br>
Ran 11 tests in 2.964s<br>
FAILED (failures=2)<br>
<br>
...<br>
<br>
thanks for reading, if you came this far!<br>
<br>
marios<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><br clear="all"><div><br></div>-- <br><div>Kevin Benton</div>
</div>