-
Notifications
You must be signed in to change notification settings - Fork 18.8k
revendor engine-api #25721
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
revendor engine-api #25721
Conversation
6bf3c5b
to
d431730
Compare
Found more issues with engine-api swallowing errors after a recent change, breaking some tests here: fixed in docker-archive-public/docker.engine-api#366 |
d431730
to
c58d8f6
Compare
@cpuguy83 merged! |
c58d8f6
to
b443cf0
Compare
And docker-py tests... |
This was removed from the API, but maybe needs to be added for older API versions?
|
Opened docker/docker-py#1162 to address calls to |
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
b443cf0
to
6d98e34
Compare
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Ok, this is updated with the new docker-py bump, looks like all issues are resolved. |
LGTM |
LGTM 👼 |
Docker no longer has an `ExecutionDriver` as of Docker 1.11. The field in the `docker info` API will not be present in 1.13. Found this while working on moby/moby#25721 Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Revendoring engine-api separate of any PR because changes required just to make docker compile are too numerous for other PR's to have to implement.