fix(harness): use TF_CLI_ARGS to correctly pass tf args (#445)

floreks · web-flow · commit e18248dd733a · 2025-05-23T09:21:28.000+02:00
diff --git a/pkg/harness/stackrun/v1/types.go b/pkg/harness/stackrun/v1/types.go
@@ -87,16 +87,16 @@ func (in *StackRun) Env() []string {
 		env = append(env, fmt.Sprintf("PLURAL_CONSOLE_URL=%s", lo.FromPtr(in.Creds.URL)))
 	}
 
-	var tfArgs []string
+	tfArgs := "TF_CLI_ARGS="
 	if in.Parallelism != nil {
-		tfArgs = append(tfArgs, fmt.Sprintf("-parallelism=%d", *in.Parallelism))
+		tfArgs += fmt.Sprintf("-parallelism=%d ", *in.Parallelism)
 	}
 
 	if in.Refresh != nil {
-		tfArgs = append(tfArgs, fmt.Sprintf("-refresh=%t", *in.Refresh))
+		tfArgs += fmt.Sprintf("-refresh=%t", *in.Refresh)
 	}
 
-	return append(env, tfArgs...)
+	return append(env, tfArgs)
 }
 
 // Vars parses the StackRun.Variables map as a valid JSON.

Original file line number	Diff line number	Diff line change
`@@ -87,16 +87,16 @@ func (in *StackRun) Env() []string {`
`87`	`87`	`env = append(env, fmt.Sprintf("PLURAL_CONSOLE_URL=%s", lo.FromPtr(in.Creds.URL)))`
`88`	`88`	`}`
`89`	`89`
`90`		`- var tfArgs []string`
	`90`	`+ tfArgs := "TF_CLI_ARGS="`
`91`	`91`	`if in.Parallelism != nil {`
`92`		`- tfArgs = append(tfArgs, fmt.Sprintf("-parallelism=%d", *in.Parallelism))`
	`92`	`+ tfArgs += fmt.Sprintf("-parallelism=%d ", *in.Parallelism)`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`if in.Refresh != nil {`
`96`		`- tfArgs = append(tfArgs, fmt.Sprintf("-refresh=%t", *in.Refresh))`
	`96`	`+ tfArgs += fmt.Sprintf("-refresh=%t", *in.Refresh)`
`97`	`97`	`}`
`98`	`98`
`99`		`- return append(env, tfArgs...)`
	`99`	`+ return append(env, tfArgs)`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`// Vars parses the StackRun.Variables map as a valid JSON.`