From 0069831e8ddc73973c0531c0900b71bdd759302d Mon Sep 17 00:00:00 2001 From: ElliotG Date: Wed, 5 Apr 2023 12:30:53 -0600 Subject: [PATCH] fix: use error log when failing provisioner job (#6812) Co-authored-by: Colin Adler --- provisionerd/provisionerd_test.go | 4 ++-- provisionerd/runner/runner.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/provisionerd/provisionerd_test.go b/provisionerd/provisionerd_test.go index e15a79cf28..fc6fb4282b 100644 --- a/provisionerd/provisionerd_test.go +++ b/provisionerd/provisionerd_test.go @@ -1011,7 +1011,7 @@ func TestProvisionerd(t *testing.T) { t.Cleanup(func() { close(done) }) - logger := slogtest.Make(t, nil) + logger := slogtest.Make(t, &slogtest.Options{IgnoreErrors: true}) m := sync.Mutex{} var ops []string completeChan := make(chan struct{}) @@ -1120,7 +1120,7 @@ func createTar(t *testing.T, files map[string]string) []byte { // Creates a provisionerd implementation with the provided dialer and provisioners. func createProvisionerd(t *testing.T, dialer provisionerd.Dialer, provisioners provisionerd.Provisioners) *provisionerd.Server { server := provisionerd.New(dialer, &provisionerd.Options{ - Logger: slogtest.Make(t, nil).Named("provisionerd").Leveled(slog.LevelDebug), + Logger: slogtest.Make(t, &slogtest.Options{IgnoreErrors: true}).Named("provisionerd").Leveled(slog.LevelDebug), JobPollInterval: 50 * time.Millisecond, UpdateInterval: 50 * time.Millisecond, Provisioners: provisioners, diff --git a/provisionerd/runner/runner.go b/provisionerd/runner/runner.go index cbe443625d..6dfe2f9efb 100644 --- a/provisionerd/runner/runner.go +++ b/provisionerd/runner/runner.go @@ -900,7 +900,7 @@ func (r *Runner) buildWorkspace(ctx context.Context, stage string, req *sdkproto }) case *sdkproto.Provision_Response_Complete: if msgType.Complete.Error != "" { - r.logger.Warn(context.Background(), "provision failed; updating state", + r.logger.Error(context.Background(), "provision failed; updating state", slog.F("state_length", len(msgType.Complete.State)), slog.F("error", msgType.Complete.Error), )