]> git.apps.os.sepia.ceph.com Git - teuthology.git/commit
Finish removing teuthology-worker
authorZack Cerza <zack@redhat.com>
Mon, 24 Jun 2024 22:05:25 +0000 (16:05 -0600)
committerZack Cerza <zack@redhat.com>
Tue, 25 Jun 2024 21:41:52 +0000 (15:41 -0600)
commit5216561d886fa56eb8261ac37d5a0f9d8b1df712
tree3f0bd7c300685ef28e5b82488f486bda5c60c29a
parentbc919dbd59aafb86996e91254fe9a142fa2c8ccb
Finish removing teuthology-worker

The dispatcher and supervisor were added in #1546, but code was copied and
pasted into the new modules, leaving the worker untouched. Also untouched were
the unit tests, meaning that the dispatcher and supervisor were never unit
tested. As the copied code changed, the dispatcher and supervisor were not being
tested for regressions, while the worker - which wasn't being anymore - had
passing unit tests, giving some false sense of security.

This commit removes the old worker code, and adapts the old worker tests to
apply to the dispatcher and supervisor. It also splits out teuthology-supervisor
into its own command.

Signed-off-by: Zack Cerza <zack@redhat.com>
13 files changed:
scripts/dispatcher.py
scripts/supervisor.py [new file with mode: 0644]
scripts/test/test_dispatcher.py [new file with mode: 0644]
scripts/test/test_supervisor.py [new file with mode: 0644]
scripts/test/test_worker.py [deleted file]
scripts/worker.py [deleted file]
setup.cfg
teuthology/dispatcher/__init__.py
teuthology/dispatcher/supervisor.py
teuthology/dispatcher/test/test_dispatcher_.py [new file with mode: 0644]
teuthology/dispatcher/test/test_supervisor_.py [new file with mode: 0644]
teuthology/test/test_worker.py [deleted file]
teuthology/worker.py [deleted file]