Browse Source

remove obsolete workaround

Signed-off-by: Blake Geno <blakegeno@gmail.com>
Blake Geno 10 years ago
parent
commit
9200fdd197
1 changed files with 4 additions and 6 deletions
  1. 4 6
      daemon/create.go

+ 4 - 6
daemon/create.go

@@ -53,12 +53,9 @@ func (daemon *Daemon) ContainerCreate(job *engine.Job) engine.Status {
 		job.Errorf("IPv4 forwarding is disabled.\n")
 		job.Errorf("IPv4 forwarding is disabled.\n")
 	}
 	}
 	container.LogEvent("create")
 	container.LogEvent("create")
-	// FIXME: this is necessary because daemon.Create might return a nil container
-	// with a non-nil error. This should not happen! Once it's fixed we
-	// can remove this workaround.
-	if container != nil {
-		job.Printf("%s\n", container.ID)
-	}
+
+	job.Printf("%s\n", container.ID)
+
 	for _, warning := range buildWarnings {
 	for _, warning := range buildWarnings {
 		job.Errorf("%s\n", warning)
 		job.Errorf("%s\n", warning)
 	}
 	}
@@ -108,6 +105,7 @@ func (daemon *Daemon) Create(config *runconfig.Config, hostConfig *runconfig.Hos
 	}
 	}
 	return container, warnings, nil
 	return container, warnings, nil
 }
 }
+
 func (daemon *Daemon) GenerateSecurityOpt(ipcMode runconfig.IpcMode) ([]string, error) {
 func (daemon *Daemon) GenerateSecurityOpt(ipcMode runconfig.IpcMode) ([]string, error) {
 	if ipcMode.IsHost() {
 	if ipcMode.IsHost() {
 		return label.DisableSecOpt(), nil
 		return label.DisableSecOpt(), nil