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

Push fails with Daemon 1.10-rc1 #437

Closed
marcust opened this issue Jan 25, 2016 · 5 comments
Closed

Push fails with Daemon 1.10-rc1 #437

marcust opened this issue Jan 25, 2016 · 5 comments
Milestone
3.1.0-rc-1

Comments

@marcust
Copy link

marcust commented Jan 25, 2016

Apparently the Response to push was changed:

com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "aux" (class com.github.dockerjava.api.model.PushResponseItem), not marked as ignorable (9 known properties: "stream", "error", "from", "progress", "time", "status", "errorDetail", "id", "progressDetail"])
 at [Source: com.github.dockerjava.jaxrs.util.WrappedResponseInputStream@3f8bb62d; line: 134, column: 29] (through reference chain: com.github.dockerjava.api.model.PushResponseItem["aux"])
    at com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException.from(UnrecognizedPropertyException.java:51)
    at com.fasterxml.jackson.databind.DeserializationContext.reportUnknownProperty(DeserializationContext.java:817)
    at com.fasterxml.jackson.databind.deser.std.StdDeserializer.handleUnknownProperty(StdDeserializer.java:958)
    at com.fasterxml.jackson.databind.deser.BeanDeserializerBase.handleUnknownProperty(BeanDeserializerBase.java:1324)
    at com.fasterxml.jackson.databind.deser.BeanDeserializerBase.handleUnknownVanilla(BeanDeserializerBase.java:1302)
    at com.fasterxml.jackson.databind.deser.BeanDeserializer.vanillaDeserialize(BeanDeserializer.java:249)
    at com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:136)
    at com.fasterxml.jackson.databind.ObjectMapper._readValue(ObjectMapper.java:3534)
    at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:1870)
    at com.github.dockerjava.core.async.JsonStreamProcessor.processResponseStream(JsonStreamProcessor.java:44)
    at com.github.dockerjava.jaxrs.async.AbstractCallbackNotifier.call(AbstractCallbackNotifier.java:62)
    at com.github.dockerjava.jaxrs.async.AbstractCallbackNotifier.call(AbstractCallbackNotifier.java:22)
@KostyaSha
Copy link
Member

Could you refer to API version and documentation?

@marcust
Copy link
Author

marcust commented Jan 25, 2016

It's API v1.22, but apparently the progress response is not very well documented:
https://github.com/docker/docker/blob/master/docs/reference/api/docker_remote_api_v1.22.md

@KostyaSha
Copy link
Member

Would be cool to open upstream issue documentation request.

marcust pushed a commit to marcust/docker-java that referenced this issue Jan 27, 2016
Make API Version transitions easier
f2954e0 
IMHO there is no reason to be that strict about the JSON models, because it
leads to things like e.g. Issue docker-java#437 .

Be more lenient with changes in the data.
@marcust marcust mentioned this issue Jan 27, 2016
Merged
marcust pushed a commit to marcust/docker-java that referenced this issue Jan 28, 2016
Add aux to ResponseItem for API 1.22
a3d58c6 
As mentioned in Issue docker-java#437
@marcust marcust mentioned this issue Jan 28, 2016
Closed
marcust pushed a commit to marcust/docker-java that referenced this issue Jan 29, 2016
Add aux to ResponseItem for API 1.22
286731a 
As mentioned in Issue docker-java#437
@KostyaSha KostyaSha added this to the 3.1.0 milestone Jan 31, 2016
KostyaSha pushed a commit to KostyaSha/docker-java that referenced this issue Mar 10, 2016
@KostyaSha
Add aux to ResponseItem for API 1.22
8a27a63 
As mentioned in Issue docker-java#437

Signed-off-by: Kanstantsin Shautsou <kanstantsin.sha@gmail.com>
@KostyaSha KostyaSha mentioned this issue Mar 10, 2016
Merged
@llamahunter
Copy link

Just hit this 'aux' bug. I see it's merged, but the issue is not closed?

@Insomnium
Copy link

Same issue

KostyaSha pushed a commit to KostyaSha/docker-java that referenced this issue Mar 20, 2016
@KostyaSha
Add aux to ResponseItem for API 1.22
c1d67f4 
As mentioned in Issue docker-java#437

Signed-off-by: Kanstantsin Shautsou <kanstantsin.sha@gmail.com>
@Vad1mo Vad1mo mentioned this issue Apr 25, 2016
Closed
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.0-rc-1
Development

No branches or pull requests
5 participants
@marcust @KostyaSha @Insomnium @llamahunter @marcuslinke