Flaky config unit tests vnc_cfg_api_server.tests.test_virtual_network.TestVirtualNetwork.test_[create|update]_vn_with_configured_rt_in_system_range

Bug #1758960 reported by Édouard Thuleau
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
High
Édouard Thuleau

Bug Description

Traceback (most recent call last):
  File "vnc_cfg_api_server/tests/test_virtual_network.py", line 109, in test_create_vn_with_configured_rt_in_system_range
    self.assertRaises(BadRequest, self.api.virtual_network_create, vn)
  File "/home/zuul/contrail-5.0.0/build/debug/config/api-server/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 485, in assertRaises
    self.assertThat(our_callable, matcher)
  File "/home/zuul/contrail-5.0.0/build/debug/config/api-server/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 498, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: <functools.partial object at 0x7fe1e949a0a8> returned u'c3776b09-02e7-45f6-a69c-e060c8a028db'

Tags: config
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/41080
Submitter: Édouard Thuleau (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/41080
Committed: http://github.com/Juniper/contrail-controller/commit/031ee1c237dda557a7b071699d47c25b5081fbff
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 031ee1c237dda557a7b071699d47c25b5081fbff
Author: Édouard Thuleau <email address hidden>
Date: Mon Mar 26 17:56:18 2018 +0200

[config] Fix global ASN cache for unit tests

Since patch [0], APi server cache the global ASN in a
GlobalSystemConfigServer class variable which can leak between test
run.

Change-Id: I19718187f9ec8e74b5a467e57aece4d2254137fb
Closes-Bug: #1758960

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/41190
Submitter: Édouard Thuleau (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/41190
Committed: http://github.com/Juniper/contrail-controller/commit/3fbcb03e383b749cca8958055cc19be1fd4ed662
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 3fbcb03e383b749cca8958055cc19be1fd4ed662
Author: Édouard Thuleau <email address hidden>
Date: Thu Mar 29 19:04:33 2018 +0200

[config] Cache global ASN in API server instance

Do no store global value in class variable to prevent concurency access
in parallel unit test runs.

Change-Id: Ie6b3e546c9f81391a6fb8bcd94faae2e97ed9072
Closes-Bug: #1758960

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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