Skip to content

Ensures tests are run against the previously built image. #1054

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.

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

Merged
merged 1 commit into from
Jul 26, 2021

Conversation

rosbo
Copy link
Contributor

@rosbo rosbo commented Jul 23, 2021

In some rare cases, image is built on GPU worker 1 and tests are run on GPU worker 2. If GPU worker 2 has already an image with the ${branch_name}-pretest tag from a previous build, then, the tests are ran against the wrong image.

Solution: Always pull before running the tests. If the image on the worker is already the latest, this is a NoOp. If it is an image from a previous build, it will pull the latest image from GCR.

@rosbo rosbo requested a review from djherbis July 23, 2021 21:42
@rosbo rosbo merged commit 20bce43 into main Jul 26, 2021
@rosbo rosbo deleted the pull-before-run-test branch July 26, 2021 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants