zuul
/
zuul
Code Issues Proposed changes

Limit scheduler_count to 1 for Scale out Scheduler tests 

The tests in test_sos.py are used to test specific interactions between
multiple schedulers. Therefore, those tests create new schedulers when
necessary and start or stop them individually to test certain
interactions.

Using the scheduler_count variable in addition to create even more
schedulers doesn't make sense for those tests.

Thus, we set the scheduler_count to 1 to for those tests.

Change-Id: Id5502574c136c1eeef38ff4a3304746c3f73918e
This commit is contained in:
Felix Edel 2021-11-22 15:35:09 +01:00 committed by James E. Blair
parent 6d2975b6fa
commit 56d646a530
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/unit/test_sos.py
View File

@ -18,6 +18,12 @@ from tests.base import iterate_timeout, ZuulTestCase
class TestScaleOutScheduler(ZuulTestCase):
tenant_config_file = "config/single-tenant/main.yaml"
# Those tests are testing specific interactions between multiple
# schedulers. They create additional schedulers as necessary and
# start or stop them individually to test specific interactions.
# Using the scheduler_count in addition to create even more
# schedulers doesn't make sense for those tests.
scheduler_count = 1
def test_multi_scheduler(self):
# A smoke test that we can enqueue a change with one scheduler