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.

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

Integration tests in Docker can fill up disk #512

Closed
jbarrick-mesosphere opened this issue Jul 5, 2019 · 0 comments · Fixed by #522
Closed

Integration tests in Docker can fill up disk #512

jbarrick-mesosphere opened this issue Jul 5, 2019 · 0 comments · Fixed by #522
Labels

Comments

@jbarrick-mesosphere
Copy link
Member

What happened:

My integration tests running in Docker ran out of disk space:

coverage: 76.0% of statements
ok  	github.com/kudobuilder/kudo/pkg/kudoctl/bundle	0.044s	coverage: 76.0% of statements
?   	github.com/kudobuilder/kudo/pkg/kudoctl/bundle/finder	[no test files]
# go.uber.org/zap/internal/exit
vet: failed to write analysis facts: open $WORK/b593/vet.out: no space left on device
# k8s.io/client-go/plugin/pkg/client/auth/gcp
vet: failed to write analysis facts: open $WORK/b576/vet.out: no space left on device
# github.com/kudobuilder/kudo/pkg/kudoctl/util/kudo.test
/usr/local/go/pkg/tool/linux_amd64/link: flushing $WORK/b563/kudo.test: write $WORK/b563/kudo.test: no space left on device
# github.com/kudobuilder/kudo/pkg/test/utils.test
/usr/local/go/pkg/tool/linux_amd64/link: flushing $WORK/b570/utils.test: write $WORK/b570/utils.test: no space left on device

I was running them in a loop and we do not run the Docker container with --rm so every container stays around.

jbarrick-mesosphere added a commit to jbarrick-mesosphere/kudo that referenced this issue Jul 8, 2019
… set --rm to remove containers after tests (kudobuilder#512), -it to support cancelling tests.
kensipe pushed a commit that referenced this issue Jul 9, 2019
…o remove containers after tests (#512), -it to support cancelling tests. (#522)

* Set Docker memory limit for tests to prevent hangs (#511), set --rm to remove containers after tests (#512), -it to support cancelling tests.

* Reduce memory limit to 2 gigs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant