luxolus/open-nomad
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

Make sure createHostConfig is called only once

Browse source 
The second call discards any mods to port binding or
network type

Fixes #185 and #196
This commit is contained in:
Abhishek Chanda 2015-10-02 10:43:37 -07:00
parent 191ea91cdd
commit 3751b46673
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
client/driver/docker.go
View file

@ -267,7 +267,7 @@ func (d *DockerDriver) Start(ctx *ExecContext, task *structs.Task) (DriverHandle
d.logger.Printf("[INFO] driver.docker: created container %s", container.ID) d.logger.Printf("[INFO] driver.docker: created container %s", container.ID)
// Start the container // Start the container
err = client.StartContainer(container.ID, createHostConfig(task)) err = client.StartContainer(container.ID, container.HostConfig)
if err != nil { if err != nil {
d.logger.Printf("[ERR] driver.docker: starting container %s", container.ID) d.logger.Printf("[ERR] driver.docker: starting container %s", container.ID)
return nil, fmt.Errorf("Failed to start container %s", container.ID) return nil, fmt.Errorf("Failed to start container %s", container.ID)