NetworksSearchCriteriaTest. test_list_pagination_with_href_links fails due to external networks not filtered out

Bug #1614964 reported by Ihar Hrachyshka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Critical
Ihar Hrachyshka

Bug Description

http://logs.openstack.org/13/350613/6/check/gate-neutron-dsvm-api/0dcba21/testr_results.html.gz

Traceback (most recent call last):
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/test_networks.py", line 124, in test_list_pagination_with_href_links
    self._test_list_pagination_with_href_links()
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/base.py", line 486, in inner
    return f(self, *args, **kwargs)
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/base.py", line 477, in inner
    return f(self, *args, **kwargs)
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/base.py", line 664, in _test_list_pagination_with_href_links
    self._test_list_pagination_iteratively(self._list_all_with_hrefs)
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/base.py", line 593, in _test_list_pagination_iteratively
    len(expected_resources), sort_args
  File "/opt/stack/new/neutron/neutron/tests/tempest/api/base.py", line 644, in _list_all_with_hrefs
    self.assertNotIn('next', prev_links)
  File "/usr/local/lib/python2.7/dist-packages/testtools/testcase.py", line 455, in assertNotIn
    self.assertThat(haystack, matcher, message)
  File "/usr/local/lib/python2.7/dist-packages/testtools/testcase.py", line 498, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: {u'previous': u'http://127.0.0.1:9696/v2.0/networks?limit=1&shared=False&sort_dir=asc&sort_key=name&marker=1a533251-9f37-4f03-bc48-6e8356baffd6&page_reverse=True', u'next': u'http://127.0.0.1:9696/v2.0/networks?limit=1&shared=False&sort_dir=asc&sort_key=name&marker=1a533251-9f37-4f03-bc48-6e8356baffd6'} matches Contains('next')

In initial list result, we see a network that does not belong to the tenant. This is because it's external.

Tags: gate-failure
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/357877

Changed in neutron:
assignee: nobody → Ihar Hrachyshka (ihar-hrachyshka)
status: New → In Progress
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :
Changed in neutron:
importance: Undecided → Critical
tags: added: gate-failure
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

It doesn't look like, but let's try to use a single bug to track these types of issues, otherwise we go nuts chasing/marking them all down.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/357877
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=c3c9cdd6f29334476479d0c0e3760ac7b90a530c
Submitter: Jenkins
Branch: master

commit c3c9cdd6f29334476479d0c0e3760ac7b90a530c
Author: Ihar Hrachyshka <email address hidden>
Date: Fri Aug 19 14:32:12 2016 +0100

    Filter out external networks in NetworksSearchCriteriaTest

    Otherwise those networks that may even belong to another tenant still
    show up in the results that we then compare iterated networks to.

    Change-Id: I24b117401a1886dce0b78900b522ac9bace533bf
    Closes-Bug: #1614964

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 9.0.0.0b3

This issue was fixed in the openstack/neutron 9.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.