diff options
author | Felix Edel <felix.edel@bmw.de> | 2021-11-26 08:38:51 +0100 |
---|---|---|
committer | James E. Blair <jim@acmegating.com> | 2021-11-30 08:58:27 -0800 |
commit | c924435f1e6763f08bcf8cdc8586ac13afec0ac0 (patch) | |
tree | 464373e8417a7b7d1da976f885430ce2ad57b149 /tests/unit/test_gerrit_crd.py | |
parent | f34a0f26c09d951417db6af6da093908351b6141 (diff) | |
download | zuul-c924435f1e6763f08bcf8cdc8586ac13afec0ac0.tar.gz |
Skip tests asserting on tenant reconfig results on multi scheduler
Those tests are failing with multiple schedulers depending on which
scheduler completed the tenant reconfiguration first. As all assertions
are done on the objects from scheduler-0, they will fail if scheduler-1
completed the tenant reconfiguration first.
To make those tests work with multiple schedulers, we might want to wait
until all schedulers completed their reconfigs before doing the
assertions.
There are also a few test cases that don't rely directly on the results
of a tenant reconfiguration, but still use local tenant objects for
assertions. I assume that those tests are failing for the same reason -
because the tenant object used for the assertion is not up-to-date.
Change-Id: I4df816ec98f5fbab25cd412a5146f0f85d6d0138
Diffstat (limited to 'tests/unit/test_gerrit_crd.py')
-rw-r--r-- | tests/unit/test_gerrit_crd.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/unit/test_gerrit_crd.py b/tests/unit/test_gerrit_crd.py index e42bafef5..480d96e60 100644 --- a/tests/unit/test_gerrit_crd.py +++ b/tests/unit/test_gerrit_crd.py @@ -16,6 +16,7 @@ from tests.base import ( ZuulTestCase, simple_layout, + skipIfMultiScheduler, ) URL_FORMATS = [ @@ -542,9 +543,11 @@ class TestGerritCRD(ZuulTestCase): self.assertEqual(self.history[0].changes, '2,1 1,1') self.assertEqual(len(tenant.layout.pipelines['check'].queues), 0) + @skipIfMultiScheduler() def test_crd_check_reconfiguration(self): self._test_crd_check_reconfiguration('org/project1', 'org/project2') + @skipIfMultiScheduler() def test_crd_undefined_project(self): """Test that undefined projects in dependencies are handled for independent pipelines""" |