chore(provisionerd): use correct log levels for template provisioner logs (#16232)

DanielleMaywood · web-flow · commit a6e054a115e0 · 2025-01-23T09:27:10.000Z
Relates to #14062 Previously a `logProvisionerJobLog` helper was added in #6508 to forward logs from the provisioner at the correct log level, but this was only used for logs produced in `buildWorkspace`. This PR uses this helper for forwarding logs produced in `runTemplateImportParse` and `runTemplateImportProvisionWithRichParameters` at the correct log level.
diff --git a/provisionerd/runner/runner.go b/provisionerd/runner/runner.go
@@ -613,7 +613,7 @@ func (r *Runner) runTemplateImportParse(ctx context.Context) (
 		}
 		switch msgType := msg.Type.(type) {
 		case *sdkproto.Response_Log:
-			r.logger.Debug(context.Background(), "parse job logged",
+			r.logProvisionerJobLog(context.Background(), msgType.Log.Level, "parse job logged",
 				slog.F("level", msgType.Log.Level),
 				slog.F("output", msgType.Log.Output),
 			)
@@ -711,7 +711,7 @@ func (r *Runner) runTemplateImportProvisionWithRichParameters(
 		}
 		switch msgType := msg.Type.(type) {
 		case *sdkproto.Response_Log:
-			r.logger.Debug(context.Background(), "template import provision job logged",
+			r.logProvisionerJobLog(context.Background(), msgType.Log.Level, "template import provision job logged",
 				slog.F("level", msgType.Log.Level),
 				slog.F("output", msgType.Log.Output),
 			)

Original file line number	Diff line number	Diff line change
`@@ -613,7 +613,7 @@ func (r *Runner) runTemplateImportParse(ctx context.Context) (`
`613`	`613`	`}`
`614`	`614`	`switch msgType := msg.Type.(type) {`
`615`	`615`	`case *sdkproto.Response_Log:`
`616`		`- r.logger.Debug(context.Background(), "parse job logged",`
	`616`	`+ r.logProvisionerJobLog(context.Background(), msgType.Log.Level, "parse job logged",`
`617`	`617`	`slog.F("level", msgType.Log.Level),`
`618`	`618`	`slog.F("output", msgType.Log.Output),`
`619`	`619`	`)`
`@@ -711,7 +711,7 @@ func (r *Runner) runTemplateImportProvisionWithRichParameters(`
`711`	`711`	`}`
`712`	`712`	`switch msgType := msg.Type.(type) {`
`713`	`713`	`case *sdkproto.Response_Log:`
`714`		`- r.logger.Debug(context.Background(), "template import provision job logged",`
	`714`	`+ r.logProvisionerJobLog(context.Background(), msgType.Log.Level, "template import provision job logged",`
`715`	`715`	`slog.F("level", msgType.Log.Level),`
`716`	`716`	`slog.F("output", msgType.Log.Output),`
`717`	`717`	`)`