Apply dannys comment

BrunoQuaresma · BrunoQuaresma · commit 299cd7ffcfc9 · 2024-07-18T16:01:21.000Z
diff --git a/coderd/autobuild/lifecycle_executor.go b/coderd/autobuild/lifecycle_executor.go
@@ -332,7 +332,7 @@ func (e *Executor) runOnce(t time.Time) Stats {
 						*dormantNotification,
 					)
 					if err != nil {
-						e.log.Warn(e.ctx, "failed to notify of workspace marked as dormant", slog.Error(err), slog.F("workspace", dormantNotification.Workspace.Id))
+						e.log.Warn(e.ctx, "failed to notify of workspace marked as dormant", slog.Error(err), slog.F("workspace_id", dormantNotification.Workspace.ID))
 					}
 				}
 				return nil
diff --git a/coderd/workspaces.go b/coderd/workspaces.go
@@ -955,7 +955,13 @@ func (api *API) putWorkspaceDormant(rw http.ResponseWriter, r *http.Request) {
 	if req.Dormant && apiKey.UserID != workspace.OwnerID {
 		initiator, err := api.Database.GetUserByID(ctx, apiKey.UserID)
 		if err != nil {
-			api.Logger.Warn(ctx, "failed to fetch the user that marked the workspace "+workspace.Name+" as dormant", slog.Error(err))
+			api.Logger.Warn(
+				ctx,
+				"failed to fetch the user that marked the workspace",
+				slog.Error(err),
+				slog.F("workspace_id", workspace.ID),
+				slog.F("user_id", apiKey.UserID),
+			)
 		} else {
 			_, err = dormancy.NotifyWorkspaceDormant(
 				ctx,

Original file line number	Diff line number	Diff line change
`@@ -332,7 +332,7 @@ func (e *Executor) runOnce(t time.Time) Stats {`
`332`	`332`	`*dormantNotification,`
`333`	`333`	`)`
`334`	`334`	`if err != nil {`
`335`		`- e.log.Warn(e.ctx, "failed to notify of workspace marked as dormant", slog.Error(err), slog.F("workspace", dormantNotification.Workspace.Id))`
	`335`	`+ e.log.Warn(e.ctx, "failed to notify of workspace marked as dormant", slog.Error(err), slog.F("workspace_id", dormantNotification.Workspace.ID))`
`336`	`336`	`}`
`337`	`337`	`}`
`338`	`338`	`return nil`