citadel/citadel — pull request #79 by tombee (2014-11-16 14:41:59+00:00)
Title: Fixes for upstream changes to samalba/dockerclient api

Citadel isn't building at the moment due to changes in samalba/dockerclient

- `PullImage` no longer supports tag parameter, I think this is pretty important for us, so I've submitted a PR to get that back: https://github.com/samalba/dockerclient/pull/39
- `ListContainers` now supports a parameter to request the size of a container as per @ehazlett's change, I figured that we would probably always want this information so set this to true all of the time. Thought it'd be extra clutter and a breaking change to the citadel API if we made this optional.
- `ContainerLogs` supports extra parameters via the new LogOptions parameter, not sure any of the additional options are required in citadel as it stands, so I have maintained the current API and simply constructed what looks to be the equivalent request with the new API.

@crosbymichael @ehazlett: Let me know what you think, I'm not that familiar with the code, but thought I'd try to help try to investigate/resolve the issues where I can.