Skip to content

HealthCheck.java integer not big enough #1143

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
Closed
Grisu118 opened this issue Dec 17, 2018 · 1 comment
Closed

HealthCheck.java integer not big enough #1143

Grisu118 opened this issue Dec 17, 2018 · 1 comment
Milestone
3.1.1

Comments

@Grisu118
Copy link
Contributor

Grisu118 commented Dec 17, 2018 •
edited
Loading

The startPeriod field is currently an Integer, docker uses nano seconds to represent this in the docker inspect command. So the max start period currently supported is 2.15 seconds. We cannot use the latest version because we use a start period of 60 seconds.

Version used of docker-java: 3.1.0-rc-7

javax.ws.rs.client.ResponseProcessingException: com.fasterxml.jackson.databind.JsonMappingException: Numeric value (60000000000) out of range of int at [Source: (org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream); line: 1, column: 5520] (through reference chain: com.github.dockerjava.api.command.InspectContainerResponse["Config"]->com.github.dockerjava.api.model.ContainerConfig["Healthcheck"]->com.github.dockerjava.api.model.HealthCheck["StartPeriod"])
@KostyaSha
Copy link
Member

Could you PR?

@Grisu118 Grisu118 mentioned this issue Jan 21, 2019
Merged
@KostyaSha KostyaSha added this to the 3.1.1 milestone Jan 28, 2019
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
3.1.1
Development

No branches or pull requests
2 participants
@KostyaSha @Grisu118