test/e2e: Preserve the existing environment when using exec.Command (#2876)

timflannagan · web-flow · commit 91e5d90b3ea3 · 2022-10-21T10:22:25.000-04:00
Signed-off-by: timflannagan &lt;timflannagan@gmail.com&gt;

Signed-off-by: timflannagan &lt;timflannagan@gmail.com&gt;
diff --git a/test/e2e/ctx/ctx.go b/test/e2e/ctx/ctx.go
@@ -130,7 +130,7 @@ func (ctx TestContext) DumpNamespaceArtifacts(namespace string) error {
 	}
 
 	cmd := exec.Command(ctx.artifactsScriptPath)
-	cmd.Env = append(cmd.Env, envvars...)
+	cmd.Env = append(os.Environ(), envvars...)
 	cmd.Stdout = os.Stdout
 	cmd.Stderr = os.Stderr
 	if err := cmd.Run(); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ func (ctx TestContext) DumpNamespaceArtifacts(namespace string) error {`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`cmd := exec.Command(ctx.artifactsScriptPath)`
`133`		`- cmd.Env = append(cmd.Env, envvars...)`
	`133`	`+ cmd.Env = append(os.Environ(), envvars...)`
`134`	`134`	`cmd.Stdout = os.Stdout`
`135`	`135`	`cmd.Stderr = os.Stderr`
`136`	`136`	`if err := cmd.Run(); err != nil {`