Skip to content

Conversation

cpuguy83
Copy link
Member

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.

@cpuguy83
Copy link
Member Author

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

@cpuguy83 cpuguy83 force-pushed the revendor_engine-api branch from d431730 to c58d8f6 Compare August 15, 2016 17:07
@LK4D4
Copy link
Contributor

LK4D4 commented Aug 15, 2016

@cpuguy83 merged!

@cpuguy83 cpuguy83 force-pushed the revendor_engine-api branch from c58d8f6 to b443cf0 Compare August 15, 2016 20:38
@cpuguy83
Copy link
Member Author

And docker-py tests...
ping @shin-

@cpuguy83
Copy link
Member Author

This was removed from the API, but maybe needs to be added for older API versions?

EXEC_DRIVER = c.info()['ExecutionDriver']

@tonistiigi tonistiigi added the status/failing-ci Indicates that the PR in its current state fails the test suite label Aug 16, 2016
@cpuguy83
Copy link
Member Author

Opened docker/docker-py#1162 to address calls to ExecutionDriver from the docker info output in docker-py tests.
Also added a temporary wrapper around the info struct for older API versions.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
@cpuguy83 cpuguy83 force-pushed the revendor_engine-api branch from b443cf0 to 6d98e34 Compare August 16, 2016 18:16
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
@cpuguy83
Copy link
Member Author

Ok, this is updated with the new docker-py bump, looks like all issues are resolved.

@cpuguy83 cpuguy83 removed the status/failing-ci Indicates that the PR in its current state fails the test suite label Aug 16, 2016
@LK4D4
Copy link
Contributor

LK4D4 commented Aug 16, 2016

LGTM

@vdemeester
Copy link
Member

LGTM 👼

@cpuguy83 cpuguy83 merged commit 37302bb into moby:master Aug 16, 2016
@cpuguy83 cpuguy83 deleted the revendor_engine-api branch August 16, 2016 21:18
shin- pushed a commit to docker/docker-py that referenced this pull request Aug 17, 2016
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants