Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Finish removing teuthology-worker #1960

Merged
merged 2 commits into from
Jul 25, 2024
Merged

Finish removing teuthology-worker #1960

merged 2 commits into from
Jul 25, 2024

Commits on Jul 25, 2024

  1. 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 <[email protected]>
    @zmc
    zmc committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    8d2b939 View commit details
    Browse the repository at this point in the history

  2. dispatcher: Temporarily pass through to supervisor 

    The old dispatcher expects to be able to invoke the supervisor via
`teuthology-dispatcher --supervisor`, so add this compatibility shim for the
time being.

Signed-off-by: Zack Cerza <[email protected]>
    @zmc
    zmc committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    0e66771 View commit details
    Browse the repository at this point in the history