ClusterHQ/flocker — pull request #871 by dwgebler (2014-10-06 10:48:46+00:00)
Title: Dockerclient list images ports 207

Design review only, there are a few other bits of code throughout flocker that need to swap out "unknown" or other null / fake information for real image and ports info coming from `DockerClient.list()`

This locally passes unit and functional tests for the two tests added (test image name from `list()` and test `PortMap`s from `list()`)

There is a theoretical possibility of a `ValueError` when `list()` attempts to cast the `unicode` port values retrieved from Docker to `int`s. I don't know whether to catch those in this code, or leave it to the caller of `list()`'s returned `Deferred` to add an `errback` - comments on this would be appreciated.