Ver código fonte

testutil/daemon: print all arguments when failing to start daemon

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sebastiaan van Stijn 5 anos atrás
pai
commit
f6842327b0
1 arquivos alterados com 12 adições e 11 exclusões
  1. 12 11
      testutil/daemon/daemon.go

+ 12 - 11
testutil/daemon/daemon.go

@@ -73,6 +73,7 @@ type Daemon struct {
 	dockerdBinary              string
 	dockerdBinary              string
 	log                        logT
 	log                        logT
 	pidFile                    string
 	pidFile                    string
+	args                       []string
 
 
 	// swarm related field
 	// swarm related field
 	swarmListenAddr string
 	swarmListenAddr string
@@ -224,7 +225,7 @@ func (d *Daemon) Cleanup(t testing.TB) {
 func (d *Daemon) Start(t testing.TB, args ...string) {
 func (d *Daemon) Start(t testing.TB, args ...string) {
 	t.Helper()
 	t.Helper()
 	if err := d.StartWithError(args...); err != nil {
 	if err := d.StartWithError(args...); err != nil {
-		t.Fatalf("[%s] failed to start daemon with arguments %v : %v", d.id, args, err)
+		t.Fatalf("[%s] failed to start daemon with arguments %v : %v", d.id, d.args, err)
 	}
 	}
 }
 }
 
 
@@ -251,7 +252,7 @@ func (d *Daemon) StartWithLogFile(out *os.File, providedArgs ...string) error {
 		d.pidFile = filepath.Join(d.Folder, "docker.pid")
 		d.pidFile = filepath.Join(d.Folder, "docker.pid")
 	}
 	}
 
 
-	args := append(d.GlobalFlags,
+	d.args = append(d.GlobalFlags,
 		"--containerd", containerdSocket,
 		"--containerd", containerdSocket,
 		"--data-root", d.Root,
 		"--data-root", d.Root,
 		"--exec-root", d.execRoot,
 		"--exec-root", d.execRoot,
@@ -261,19 +262,19 @@ func (d *Daemon) StartWithLogFile(out *os.File, providedArgs ...string) error {
 		"--containerd-plugins-namespace", d.id+"p",
 		"--containerd-plugins-namespace", d.id+"p",
 	)
 	)
 	if d.defaultCgroupNamespaceMode != "" {
 	if d.defaultCgroupNamespaceMode != "" {
-		args = append(args, []string{"--default-cgroupns-mode", d.defaultCgroupNamespaceMode}...)
+		d.args = append(d.args, []string{"--default-cgroupns-mode", d.defaultCgroupNamespaceMode}...)
 	}
 	}
 	if d.experimental {
 	if d.experimental {
-		args = append(args, "--experimental")
+		d.args = append(d.args, "--experimental")
 	}
 	}
 	if d.init {
 	if d.init {
-		args = append(args, "--init")
+		d.args = append(d.args, "--init")
 	}
 	}
 	if !(d.UseDefaultHost || d.UseDefaultTLSHost) {
 	if !(d.UseDefaultHost || d.UseDefaultTLSHost) {
-		args = append(args, []string{"--host", d.Sock()}...)
+		d.args = append(d.args, []string{"--host", d.Sock()}...)
 	}
 	}
 	if root := os.Getenv("DOCKER_REMAP_ROOT"); root != "" {
 	if root := os.Getenv("DOCKER_REMAP_ROOT"); root != "" {
-		args = append(args, []string{"--userns-remap", root}...)
+		d.args = append(d.args, []string{"--userns-remap", root}...)
 	}
 	}
 
 
 	// If we don't explicitly set the log-level or debug flag(-D) then
 	// If we don't explicitly set the log-level or debug flag(-D) then
@@ -289,14 +290,14 @@ func (d *Daemon) StartWithLogFile(out *os.File, providedArgs ...string) error {
 		}
 		}
 	}
 	}
 	if !foundLog {
 	if !foundLog {
-		args = append(args, "--debug")
+		d.args = append(d.args, "--debug")
 	}
 	}
 	if d.storageDriver != "" && !foundSd {
 	if d.storageDriver != "" && !foundSd {
-		args = append(args, "--storage-driver", d.storageDriver)
+		d.args = append(d.args, "--storage-driver", d.storageDriver)
 	}
 	}
 
 
-	args = append(args, providedArgs...)
-	d.cmd = exec.Command(dockerdBinary, args...)
+	d.args = append(d.args, providedArgs...)
+	d.cmd = exec.Command(dockerdBinary, d.args...)
 	d.cmd.Env = append(os.Environ(), "DOCKER_SERVICE_PREFER_OFFLINE_IMAGE=1")
 	d.cmd.Env = append(os.Environ(), "DOCKER_SERVICE_PREFER_OFFLINE_IMAGE=1")
 	d.cmd.Stdout = out
 	d.cmd.Stdout = out
 	d.cmd.Stderr = out
 	d.cmd.Stderr = out