@@ -26,7 +26,12 @@ type WorkspaceAppHealthReporter func(ctx context.Context)
2626
2727// NewWorkspaceAppHealthReporter creates a WorkspaceAppHealthReporter that reports app health to coderd.
2828func NewWorkspaceAppHealthReporter (logger slog.Logger ,apps []codersdk.WorkspaceApp ,postWorkspaceAgentAppHealth PostWorkspaceAgentAppHealth )WorkspaceAppHealthReporter {
29+ logger = logger .Named ("apphealth" )
30+
2931runHealthcheckLoop := func (ctx context.Context )error {
32+ ctx ,cancel := context .WithCancel (ctx )
33+ defer cancel ()
34+
3035// no need to run this loop if no apps for this workspace.
3136if len (apps )== 0 {
3237return nil
@@ -87,6 +92,7 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace
8792return nil
8893}()
8994if err != nil {
95+ nowUnhealthy := false
9096mu .Lock ()
9197if failures [app .ID ]< int (app .Healthcheck .Threshold ) {
9298// increment the failure count and keep status the same.
@@ -96,14 +102,21 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace
96102// set to unhealthy if we hit the failure threshold.
97103// we stop incrementing at the threshold to prevent the failure value from increasing forever.
98104health [app .ID ]= codersdk .WorkspaceAppHealthUnhealthy
105+ nowUnhealthy = true
99106}
100107mu .Unlock ()
108+ logger .Debug (ctx ,"error checking app health" ,
109+ slog .F ("id" ,app .ID .String ()),
110+ slog .F ("slug" ,app .Slug ),
111+ slog .F ("now_unhealthy" ,nowUnhealthy ),slog .Error (err ),
112+ )
101113}else {
102114mu .Lock ()
103115// we only need one successful health check to be considered healthy.
104116health [app .ID ]= codersdk .WorkspaceAppHealthHealthy
105117failures [app .ID ]= 0
106118mu .Unlock ()
119+ logger .Debug (ctx ,"workspace app healthy" ,slog .F ("id" ,app .ID .String ()),slog .F ("slug" ,app .Slug ))
107120}
108121
109122t .Reset (time .Duration (app .Healthcheck .Interval )* time .Second )
@@ -137,7 +150,9 @@ func NewWorkspaceAppHealthReporter(logger slog.Logger, apps []codersdk.Workspace
137150Healths :lastHealth ,
138151})
139152if err != nil {
140- logger .Error (ctx ,"failed to report workspace app stat" ,slog .Error (err ))
153+ logger .Error (ctx ,"failed to report workspace app health" ,slog .Error (err ))
154+ }else {
155+ logger .Debug (ctx ,"sent workspace app health" ,slog .F ("health" ,lastHealth ))
141156}
142157}
143158}