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

Testcontainers fails on free disk space of more than 2TB #379

Closed
chibenwa opened this issue Jun 27, 2017 · 5 comments
Closed

Testcontainers fails on free disk space of more than 2TB #379

chibenwa opened this issue Jun 27, 2017 · 5 comments
Labels
Milestone

Comments

@chibenwa
Copy link
Contributor

chibenwa commented Jun 27, 2017

[Building Java 8] log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.
[Building Java 8]         ℹ︎ Checking the system... 
[Building Java 8]         ✔ Docker version is newer than 1.6.0 
[Building Java 8] java.lang.NumberFormatException: For input string: "2982480572"
[Building Java 8] 	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
[Building Java 8] 	at java.lang.Integer.parseInt(Integer.java:583)
[Building Java 8] 	at java.lang.Integer.valueOf(Integer.java:766)
[Building Java 8] 	at org.testcontainers.DockerClientFactory.parseAvailableDiskSpace(DockerClientFactory.java:250)
[Building Java 8] 	at org.testcontainers.DockerClientFactory.checkDiskSpace(DockerClientFactory.java:165)
[Building Java 8] 	at org.testcontainers.DockerClientFactory.lambda$client$1(DockerClientFactory.java:126)
[Building Java 8] 	at org.testcontainers.DockerClientFactory.runInsideDocker(DockerClientFactory.java:229)
[Building Java 8] 	at org.testcontainers.DockerClientFactory.client(DockerClientFactory.java:117)
[Building Java 8] 	at org.testcontainers.containers.GenericContainer.<init>(GenericContainer.java:110)
[ ............. ]
[Building Java 8] 	at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:121)

IMO disk space should be represented as a long.

@bsideup bsideup added this to the 1.4.0 milestone Jun 27, 2017
@bsideup
Copy link
Member

bsideup commented Jun 27, 2017

Hi @chibenwa,

Wow, that's awkward :D

I'll fix it now.
Meanwhile, you can put checks.disable=true to $HOME/.testcontainers.properties file.

Thanks for reporting!

@chibenwa
Copy link
Contributor Author

chibenwa commented Jun 27, 2017

Ah ok, thanks for giving me a temporary fix. It seems better than creating a ~ 800GB file, what I was going to do...

However I run testcontainer from a docker container (we have a CI that launches Docker containers for multiple reasons), is there a way to read these kinds of properties from the classpath?

That would avoid me changing the docker compilation image...

@bsideup
Copy link
Member

bsideup commented Jun 27, 2017

Sure, it will merge home folder's config with classpath's one called testcontainers.properties

@chibenwa
Copy link
Contributor Author

Nice ;-) Thanks for your helpfull support!

@bsideup
Copy link
Member

bsideup commented Jun 27, 2017

@chibenwa PR is ready, you can try it with JitPack:
https://jitpack.io/#testcontainers/testcontainers-java/fix_available_space_detection-SNAPSHOT

@rnorth rnorth closed this as completed in 1b89280 Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants