Bug-fix: mapping "networks" in statistics#360
Bug-fix: mapping "networks" in statistics#360bonigarcia wants to merge 0 commit intodocker-java:masterfrom
Conversation
|
Could you provide info on what docker version have you checked, could you try to create a test case? Could you check that it wasn't changed between different docker API versions? |
|
I am using Docker version 1.9.0, build 76d6bc9. |
|
I tested with docker-java 2.1.1, 2.1.2 and 3.0.0-SNAPSHOT. Both of three have the same bug. |
|
Sorry, i mean docker host API |
|
I am using Docker 1.9.0, and according to Docker documentation, API version should be 1.21. |
|
Seems stats JSON has been changed with latest docker remote API: |
|
@marcuslinke seems it because it now support multiple interfaces. The right way to fix will be add new field, mark as |
|
@KostyaSha Right. I agree. @bonigarcia Could you do so? |
|
@bonigarcia next time please better force push to this PR and change title according. Will allow easier find discussion for changes. |
There is a bug in the mapping of the network statistics. Currently, calling to com.github.dockerjava.api.model.Statistics.getNetworkStats() returns null.