Dec 2, 2024 · Nov 29, 2024 · Nov 29, 2024 · Dec 1, 2024 · Dec 1, 2024 · Dec 1, 2024
diff --git a/coderd/workspacebuilds.go b/coderd/workspacebuilds.go
 })
 return
 }
 err = provisionerjobs.PostJob(api.Pubsub, *provisionerJob)
 if err != nil {
 // Client probably doesn't care about this error, so just log it.
 api.Logger.Error(ctx, "failed to post provisioner job to pubsub", slog.Error(err))

 if provisionerJob != nil {
 if err := provisionerjobs.PostJob(api.Pubsub, *provisionerJob); err != nil {
 // Client probably doesn't care about this error, so just log it.
 api.Logger.Error(ctx, "failed to post provisioner job to pubsub", slog.Error(err))
 }
 }

 var matchedProvisioners codersdk.MatchedProvisioners
Original file line number	Diff line number	Diff line change
Expand Up		@@ -387,10 +387,12 @@ func (api API) postWorkspaceBuilds(rw http.ResponseWriter, r http.Request) {
		})
		return
		}
		err = provisionerjobs.PostJob(api.Pubsub, *provisionerJob)
		if err != nil {
		// Client probably doesn't care about this error, so just log it.
		api.Logger.Error(ctx, "failed to post provisioner job to pubsub", slog.Error(err))

		if provisionerJob != nil {
		if err := provisionerjobs.PostJob(api.Pubsub, *provisionerJob); err != nil {
		// Client probably doesn't care about this error, so just log it.
		api.Logger.Error(ctx, "failed to post provisioner job to pubsub", slog.Error(err))
		}
Comment on lines -387 to +395 Copy link MemberAuthor johnstcnDec 1, 2024• edited Loading Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. self-review: linter complained about a possible nil ptr reference in pre-existing code.
		}

		var matchedProvisioners codersdk.MatchedProvisioners
Expand Down