Patched options() to let container options propagate to job containers (#80)

nicorac · wolfogre · commit b4edc952d98b · 2024-03-25T15:43:14.000+08:00
This PR let "general" container config to be propagated to each job container. See: - https://gitea.com/gitea/act_runner/issues/265#issuecomment-744382 - https://gitea.com/gitea/act_runner/issues/79 - https://gitea.com/gitea/act_runner/issues/378 Reviewed-on: https://gitea.com/gitea/act/pulls/80 Reviewed-by: Jason Song <i@wolfogre.com> Co-authored-by: Claudio Nicora <claudio.nicora@gmail.com> Co-committed-by: Claudio Nicora <claudio.nicora@gmail.com>
diff --git a/pkg/runner/run_context.go b/pkg/runner/run_context.go
@@ -649,7 +649,7 @@ func (rc *RunContext) options(ctx context.Context) string {
 	job := rc.Run.Job()
 	c := job.Container()
 	if c != nil {
-		return rc.ExprEval.Interpolate(ctx, c.Options)
+		return rc.Config.ContainerOptions + " " + rc.ExprEval.Interpolate(ctx, c.Options)
 	}
 
 	return rc.Config.ContainerOptions

Original file line number	Diff line number	Diff line change
`@@ -649,7 +649,7 @@ func (rc *RunContext) options(ctx context.Context) string {`
`649`	`649`	`job := rc.Run.Job()`
`650`	`650`	`c := job.Container()`
`651`	`651`	`if c != nil {`
`652`		`- return rc.ExprEval.Interpolate(ctx, c.Options)`
	`652`	`+ return rc.Config.ContainerOptions + " " + rc.ExprEval.Interpolate(ctx, c.Options)`
`653`	`653`	`}`
`654`	`654`
`655`	`655`	`return rc.Config.ContainerOptions`