-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Docker image volume fix and Sharness tests #2064
Conversation
Circle CI is failing with:
I am not sure it supports docker or maybe some options are needed to make it run docker. |
About CircleCI I found an option in the doc for docker support, so I added a commit to set it. |
Travis-CI supports Docker on Linux only for now. |
travis-ci/push has this failure in go tests:
|
b3d6d5f
to
781506c
Compare
I rebased to remove debug output. |
We get "Unsupported: Exec is not supported by the lxc driver" with CircleCi. It looks like "docker exec" doesn't work on CircleCi: |
I think I will have to do something like this:
|
There is this error on travis-ci/push:
|
On travis-ci/pr there is this error:
|
ok
wow, bummer. maybe put that in a shell scrpt and call it
the hashes seem different. content somehow different??
ugh yeah that sometimes happens. there's some test that hangs travis-- we should look into them. unrelated i think |
the closed network connection stuff is odd. probably some issue with the docker networking |
781506c
to
ae53f1b
Compare
Yeah, with the docker_exec() that has a special case for CircleCI it works on CircleCI! |
@chriscool i think the go test failure here is not a usual one-- i do not think i have seen it on travis without docker. |
The got test errors are:
Yeah, they are perhaps related to the Travis CI environment with docker. @whyrusleeping do you know what in the Travis CI environment with docker could cause those failures? |
648fe54
to
ec1a886
Compare
@chriscool i've got the ipv6 stuff fixed now, but it appears that travis and t0042 don't get along well... |
@whyrusleeping I think the Travis tests passed except the usual |
And one of those failures is on Linux not OSX! |
It's the one from continuous-integration/travis-ci/pr. |
LGTM |
License: MIT Signed-off-by: Christian Kniep <christian@qnib.org>
License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
We have to do something special for CircleCI in docker_exec() because "docker exec" doesn't work on CircleCi: https://circleci.com/docs/docker#docker-exec We indeed get "Unsupported: Exec is not supported by the lxc driver" with CircleCi, when using "docker exec". License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
ec1a886
to
1fbbd1d
Compare
Rebased on current master. Let's have a look at the tests. |
License: MIT Signed-off-by: Jeromy <jeromyj@gmail.com>
Docker image volume fix and Sharness tests
This PR contains @ChristianKniep's fix from PR #1806 (change Dockerfile order to create VOLUME under user ipfs) and add a Sharness test script (t0300) with some docker tests.
@ChristianKniep's fix is fixing issue #1804 (Boot2Docker: VOLUME prevents chown to have effect).