-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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-py integration tests aginst the docker daemon #9741
Conversation
This clones and run the integration tests for docker-py master as part of the integration tests created on master. docker-py hits the api directly and should be a good way to identify regressions in the api. Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
See, another case where having Drone run |
I also propose we wait to really consider this until tomorrow when 0.7.0 is expected to drop so we can switch over to using that tag instead of this temporary 0.6.0-integration branch that @shin- is nice enough to have up for us with some fixes that've happened to that test since 0.6.0 was tagged. 👍 |
- move docker/docker-py clone to the Dockerfile - put "integration test daemon startup" code in a separate file for both scripts to source - add new test-docker-py Makefile target - include "python-websocket" package in Dockerfile for running the tests Signed-off-by: Andrew "Tianon" Page <admwiggin@gmail.com>
d1a8c30
to
100267d
Compare
Updated with 0.7.0! 👍 |
wooohoooo |
Haha, panic on drone! |
👍 |
LGTM |
I create the new drone images as soon as this is merged |
LGTM |
Add docker-py integration tests aginst the docker daemon
Closes #9676
Changes from @crosbymichael's code: