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.

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

test(integration): properly waited for the image pulls to complete #2842

Merged
merged 1 commit into from Jun 26, 2023

Conversation

travi
Copy link
Member

@travi travi commented Jun 20, 2023

this uses the technique for waiting for the pull to complete from the dockerode readme and mostly gets rid of our dependence on get-stream.

the remaining get-stream usage is the .array() that is removed in v7. i suggest we deal with that usage separately from this PR

@travi travi marked this pull request as draft June 20, 2023 04:51
@travi
Copy link
Member Author

travi commented Jun 20, 2023

it appears that the pulls are happening before each test in the integration file. would be better to do the pulls before the overall suite run so that they only happen once

@travi
Copy link
Member Author

travi commented Jun 20, 2023

it appears that the pulls are happening before each test in the integration file. would be better to do the pulls before the overall suite run so that they only happen once

i was wrong. it is using test.before, not test.beforeEach, so this should already be correct. since we are actually fully waiting for the pulls, we may need to extend the timeout for the integration test, though

@travi travi force-pushed the docker-pull branch 3 times, most recently from f7feb95 to c9e1b99 Compare June 24, 2023 04:49
@travi travi marked this pull request as ready for review June 24, 2023 04:53
@travi
Copy link
Member Author

travi commented Jun 24, 2023

it looks like this change does not fix the flakiness, but i think it is still a worthwhile change to get merged

@gr2m gr2m merged commit e39ae90 into master Jun 26, 2023
3 of 4 checks passed
@gr2m gr2m deleted the docker-pull branch June 26, 2023 00:51
@github-actions
Copy link

🎉 This PR is included in version 21.0.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants