yahoo-eng-team team mailing list archive

[OpenStack Infra <1614964@xxxxxxxxxxxxxxxxxx>]

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 <ihrachys@xxxxxxxxxx>
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

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1614964

Title:
  NetworksSearchCriteriaTest. test_list_pagination_with_href_links fails
  due to external networks not filtered out

Status in neutron:
  Fix Released

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.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1614964/+subscriptions