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

circleci: cache the docker-spy image #3025

Closed
tbg opened this issue Nov 5, 2015 · 5 comments
Closed

circleci: cache the docker-spy image #3025

tbg opened this issue Nov 5, 2015 · 5 comments
Assignees
@maddyblue
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) E-easy Easy issue to tackle, requires little or no CockroachDB experience help wanted Help is requested / needed by the one who filed the issue to fix it.
Milestone
Beta

Comments

@tbg
Copy link
Member

tbg commented Nov 5, 2015

The following test appears to have failed:

#8991:

=== RUN   TestBuildInfo
I1105 23:07:19.511778 15932 acceptance/localcluster/localcluster.go:196  pulling cockroachdb/docker-spy
panic: test timed out after 5m0s

goroutine 29 [running]:
testing.startAlarm.func1()
    /usr/local/go/src/testing/testing.go:703 +0x132
created by time.goFunc
    /usr/local/go/src/time/sleep.go:129 +0x3a

goroutine 1 [chan receive]:
testing.RunTests(0xb5e5b0, 0xf2d1c0, 0x8, 0x8, 0xa59301)
    /usr/local/go/src/testing/testing.go:562 +0x8ad

Please assign, take a look and update the issue accordingly.
@tbg tbg added the C-test-failure Broken test (automatically or manually discovered). label Nov 5, 2015
@tamird tamird changed the title Test failure in CI build 8991 CI: cache the docker-spy image Nov 5, 2015
@tamird
Copy link
Contributor

tamird commented Nov 5, 2015

We're not currently caching the docker-spy image, so this happens every now and again. Bummer.

This was referenced Nov 13, 2015
Merged
Closed
@tamird tamird added help wanted Help is requested / needed by the one who filed the issue to fix it. E-easy Easy issue to tackle, requires little or no CockroachDB experience labels Nov 13, 2015
@tbg tbg removed the C-test-failure Broken test (automatically or manually discovered). label Jan 6, 2016
@petermattis
Copy link
Collaborator

@mjibson Should be straightforward as part of the other CircleCI docker image caching you were doing.

@petermattis petermattis mentioned this issue Feb 12, 2016
Closed
@petermattis petermattis added the C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) label Feb 14, 2016
@petermattis petermattis modified the milestone: Beta Feb 14, 2016
@petermattis petermattis changed the title CI: cache the docker-spy image circleci: cache the docker-spy image Feb 16, 2016
@petermattis
Copy link
Collaborator

Fixed by #4414.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maddyblue maddyblue

Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) E-easy Easy issue to tackle, requires little or no CockroachDB experience help wanted Help is requested / needed by the one who filed the issue to fix it.
Projects
None yet
Milestone
Beta
Development

No branches or pull requests
5 participants
@maddyblue @tamird @tbg @petermattis and others