netlink test_list_entries failing with mismatch
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| neutron |
High
|
Cuong Nguyen |
Bug Description
Spotted in check job that has no relationship to netlink.
Traceback (most recent call last):
File "neutron/
return f(self, *args, **kwargs)
File "neutron/
self.
File "/opt/stack/
self.
File "/opt/stack/
raise mismatch_error
testtools.
reference = ((4, 'icmp', 8, 0, '1.1.1.1', '2.2.2.2', 3333, 10),
(4, 'tcp', 1, 2, '1.1.1.1', '2.2.2.2', 10),
(4, 'udp', 4, 5, '1.1.1.1', '2.2.2.2', 10))
actual = ((4, 'icmp', 8, 0, '1.1.1.1', '2.2.2.2', 3333, 10),
(4, 'tcp', 1, 2, '1.1.1.1', '2.2.2.2', 10),
(4, 'udp', 4, 5, '1.1.1.1', '2.2.2.2', 10),
(4, 'tcp', 40364, 12345, '192.168.0.1', '192.168.0.2', 10),
(4, 'tcp', 59330, 12346, '192.168.0.1', '192.168.0.2', 10))
Changed in neutron: | |
status: | New → Confirmed |
importance: | Undecided → High |
tags: | added: gate-failure |
Changed in neutron: | |
assignee: | nobody → Cuong Nguyen (cuongnv) |
Cuong Nguyen (cuongnv) wrote : | #1 |
Cao Xuan Hoang (hoangcx) wrote : | #2 |
I think the solution to isolate: We need to make sure there is no conntrack entry in a zone_id before executing the test case.
Fix proposed to branch: master
Review: https:/
Changed in neutron: | |
status: | Confirmed → In Progress |
tags: | added: functional-tests |
Changed in neutron: | |
milestone: | none → pike-rc1 |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit b19fb20336b5aa2
Author: Cuong Nguyen <email address hidden>
Date: Wed Aug 2 09:31:33 2017 +0700
Fixes input for netlink-lib functional tests
Netlink-lib creates some conntrack entries to verify list_entries()
method. Each entry contains a zone_id, which might be duplicated with
some already existed entries in OS.
This patch proposes a simple verification to make sure entries created by
netlink-lib do not contain existed zone_id.
Change-Id: I4bce5041f82782
Closes-Bug: #1708030
Changed in neutron: | |
status: | In Progress → Fix Released |
Fix proposed to branch: stable/pike
Review: https:/
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: stable/pike
commit 64351d1856e67e5
Author: Cuong Nguyen <email address hidden>
Date: Wed Aug 2 09:31:33 2017 +0700
Fixes input for netlink-lib functional tests
Netlink-lib creates some conntrack entries to verify list_entries()
method. Each entry contains a zone_id, which might be duplicated with
some already existed entries in OS.
This patch proposes a simple verification to make sure entries created by
netlink-lib do not contain existed zone_id.
Change-Id: I4bce5041f82782
Closes-Bug: #1708030
(cherry picked from commit b19fb20336b5aa2
tags: | added: in-stable-pike |
This issue was fixed in the openstack/neutron 11.0.0.0rc2 release candidate.
tags: | added: neutron-proactive-backport-potential |
This issue was fixed in the openstack/neutron 12.0.0.0b1 development milestone.
tags: | removed: neutron-proactive-backport-potential |
The possible problem here is zone_id created by netlink-lib during functional tests duplicated with zone_id created by something else. In this test case, netlink-lib lists all conntrack entries in OS with zone_id is '10', but some other entries with zone '10' already existed hence the test didn't work properly.
Problem is in the test case, netlink-lib doesn't create these two conntrack entries
(4, 'tcp', 40364, 12345, '192.168.0.1', '192.168.0.2', 10), (4, 'tcp', 59330, 12346, '192.168.0.1', '192.168.0.2', 10) so I'm in doubt that these conntrack entries somehow already existed before tests running.