Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CI is blocked on minio download. #1483

Closed
bwplotka opened this issue Sep 2, 2019 · 1 comment · Fixed by #1484
Closed

CI is blocked on minio download. #1483

bwplotka opened this issue Sep 2, 2019 · 1 comment · Fixed by #1484

Comments

@bwplotka
Copy link
Member

bwplotka commented Sep 2, 2019 

>> produced /go/bin/alertmanager-v0.15.2
>> fetching github.com/minio/minio@RELEASE.2018-10-06T00-15-16Z revision/version
# cd .; git clone https://git.apache.org/thrift.git /tmp/thanos-go/src/git.apache.org/thrift.git
Cloning into '/tmp/thanos-go/src/git.apache.org/thrift.git'...
fatal: unable to access 'https://git.apache.org/thrift.git/': Failed to connect to git.apache.org port 443: Connection timed out
package github.com/minio/minio
	imports git.apache.org/thrift.git/lib/go/thrift: exit status 128
Makefile:209: recipe for target 'test-deps' failed
make[2]: *** [test-deps] Error 1
make[2]: Leaving directory '/go/src/github.com/thanos-io/thanos'
Makefile:193: recipe for target 'test-only-gcs' failed
make[1]: *** [test-only-gcs] Error 2
make[1]: Leaving directory '/go/src/github.com/thanos-io/thanos'
Makefile:202: recipe for target 'test-local' failed
make: *** [test-local] Error 2
Exited with code 2

Trying to unblock now by caching the images in dockerfile.
@bwplotka bwplotka mentioned this issue Sep 2, 2019
Merged
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
80f9ab7 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
3196b5f 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
@bwplotka bwplotka mentioned this issue Sep 2, 2019
Merged
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
c790881 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
@bwplotka
Copy link
Member Author

bwplotka commented Sep 2, 2019

Or.. at least flaky.

bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
1b17f0e 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
111bc51 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
@kakkoyun kakkoyun mentioned this issue Sep 2, 2019
Merged
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
f41a1c3 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI.
a01c9e9 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
bwplotka added a commit that referenced this issue Sep 2, 2019
@bwplotka
Baked required deps in docker image for CircleCI. (#1484)
c4fff05 
This should fix #1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
wbh1 pushed a commit to wbh1/thanos that referenced this issue Sep 17, 2019
@bwplotka @wbh1
Baked required deps in docker image for CircleCI. (thanos-io#1484)
6655470 
This should fix thanos-io#1483

Also it should massively improve our CI time.

Signed-off-by: Bartek Plotka <bwplotka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Baked required deps in docker image for CircleCI. thanos-io/thanos
1 participant
@bwplotka