Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Remove StopJob test workaround #216

Open
ysimonson opened this issue Aug 12, 2020 · 0 comments
Open

Remove StopJob test workaround #216

ysimonson opened this issue Aug 12, 2020 · 0 comments

Comments

@ysimonson
Copy link
Contributor

We currently have a workaround in the tests due to an ergonomics issue in StopJob. We should address this in tests once the StopJob issue is fixed: pachyderm/pachyderm#3856

# This is necessary because `StopJob` does not wait for the job to be
# killed before returning a result.
# TODO: remove once this is fixed:
# https://github.com/pachyderm/pachyderm/issues/3856
time.sleep(1)
job = sandbox.client.inspect_job(job_id)
assert job.state == python_pachyderm.JobState.JOB_KILLED.value

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant