8000 Switch CircleCI build VM to 14.04 by brb · Pull Request #2491 · weaveworks/weave · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

Switch CircleCI build VM to 14.04 #2491

Merged
merged 3 commits into from
Sep 5, 2016
Merged

Switch CircleCI build VM to 14.04 #2491

merged 3 commits into from
Sep 5, 2016

Conversation

brb
Copy link
Contributor
@brb brb commented Aug 26, 2016

The 14.04 VM includes go1.6 which uses vendor/.

@brb brb added this to the 1.6.2 milestone Aug 26, 2016
@rade
Copy link
Member
rade commented Aug 26, 2016

This is not the first time that joffrey/docker-py tests fail. In the past we fixed things w/o pinning. Given that the purpose of the tests is to check that the proxy is faithfully handling all docker traffic - regardless of version - I rather think pinning is a bad idea.

@brb brb force-pushed the switch-ci-14.04 branch 4 times, most recently from 4701131 to bea258f Compare August 31, 2016 17:18
@brb
Copy link
Contributor Author
brb commented Sep 5, 2016

The pinning to 1.8.1 is required because the latest version of joffrey/docker-py tests requires 1.24 Docker API (=> 1.12.x Docker). Unfortunately, 1.12.x Docker is very unreliable and hangs from time to time during a run of the smoke tests.

I've created a separate issue for switching back to joffrey/docker-py:latest and 1.12.1 Docker: #2499

@rade
Copy link
Member
rade commented Sep 5, 2016

the latest version of joffrey/docker-py tests requires 1.24 Docker

@brb That is a bug in docker-py. The docker-py tests are annotated with version information, so only tests for the right version should be run.

We should not be pinning the joffrey/docker-py image.

@@ -52,7 +52,7 @@ following command on any peer:
weave status ipam

This command displays the peer names and nicknames, absolute quantity/percentage of allocation
range managed by peer and also identifies the names of unreachable peers. If you are satisifed
range managed by peer and also identifies the names of unreachable peers. If you are satisfied

This comment was marked as abuse.

@rade rade assigned brb Sep 5, 2016
@rade
Copy link
Member
rade commented Sep 5, 2016

We should not be pinning the joffrey/docker-py image.

Hmm. The docker-py tests really ought to work out the server version by themselves. Alas I cannot find any code in docker-py that actually does that. I was convinced we had run into this sort of problem before, but have not been able to find an issue/pr in weave or docker-py for it, so I must have been imagining it.

I am perplexed what the purpose of the version annotations in the tests is when there is no way to actually run the tests with a version other than DEFAULT_DOCKER_API_VERSION, which is set to 1.24 in the latest docker-py.

@brb
Copy link
Contributor Author
brb commented Sep 5, 2016

docker/docker-py#1183

@brb brb changed the title Switch CircleCI build VM to 14.04 and make the CI tests pass Switch CircleCI build VM to 14.04 Sep 5, 2016
@brb brb force-pushed the switch-ci-14.04 branch 3 times, most recently from 0f3cadc to aee35b3 Compare September 5, 2016 11:30
brb added 3 commits September 5, 2016 14:30
As it is already installed on CircleCI 14.04 build VMs
Otherwise "go get -tags netgo" fails with the permission denied error.
The latest image requires 1.12.x dockerd.
@rade rade merged commit 3d12e80 into 1.6 Sep 5, 2016
@brb brb deleted the switch-ci-14.04 branch September 5, 2016 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0