-
Notifications
You must be signed in to change notification settings - Fork 123
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
Add docker_image_load module #90
Add docker_image_load module #90
Conversation
recheck |
This PR contains parts of #87, will rebase once that one is merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately I couldn't find anything else but these :)
Co-authored-by: Amin Vakil <[email protected]>
38ec2b0
to
fb52e4c
Compare
Restarting tests. |
b0ae6ae
to
1f6c5a8
Compare
3d0e338
to
335eeb1
Compare
First run succeeded, restart tests. |
Second successful run, restart tests... |
Third successful run, restarting... |
Fourth successful run, restarting... |
With five successful CI runs in a row, I'm pretty sure that all spurious problems are now gone. (Or seldom enough that they don't matter too much :) ) ready_for_review |
Restarting tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixfontein thanks for the module!
Co-authored-by: Andrew Klychkov <[email protected]>
Co-authored-by: Andrew Klychkov <[email protected]>
Hmm, something's wrong with AZP. Restarting tests... |
@aminvakil @Andersson007 thanks a lot for reviewing this! |
@felixfontein @Andersson007 Thank you! |
SUMMARY
First basic, untested version. Fixes #86.
ISSUE TYPE
COMPONENT NAME
docker_image_load