saltstack/salt — pull request #9570 by bbinet (2014-01-04 10:47:32+00:00)
Title: Fix docker.start port_bindings

When starting docker container using the following state:
```
my_service:
docker.running:
- container: mysuperdocker
- port_bindings:
"5000/tcp":
- HostIp: ""
- HostPort: "5000"
```
I've realized that bound port on host was wrong (was not port 5000 as requested).
So this pull request intends to fix this. HostIp and HostPort are also now configured as dict keys (not list anymore) since it simplifies implementation.
So with this pull request, the following state works for me:
```
my_service:
docker.running:
- container: mysuperdocker
- port_bindings:
"5000/tcp":
HostPort: "5000"
```

@kiorky: it would be great if you could have a look since you're the one who did all the work on docker module and state?