fix: stop activity bump if no tracked sessions

f0ssel · f0ssel · commit 9e07bb40ee66 · 2024-10-25T19:42:42.000Z
diff --git a/coderd/workspacestats/reporter.go b/coderd/workspacestats/reporter.go
@@ -136,8 +136,8 @@ func (r *Reporter) ReportAgentStats(ctx context.Context, now time.Time, workspac
 		}, stats.Metrics)
 	}
 
-	// if no active connections we do not bump activity
-	if stats.ConnectionCount == 0 {
+	// if no active sessions we do not bump activity
+	if stats.SessionCountJetbrains == 0 && stats.SessionCountVscode == 0 && stats.SessionCountReconnectingPty == 0 && stats.SessionCountSsh == 0 {
 		return nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -136,8 +136,8 @@ func (r *Reporter) ReportAgentStats(ctx context.Context, now time.Time, workspac`
`136`	`136`	`}, stats.Metrics)`
`137`	`137`	`}`
`138`	`138`
`139`		`- // if no active connections we do not bump activity`
`140`		`- if stats.ConnectionCount == 0 {`
	`139`	`+ // if no active sessions we do not bump activity`
	`140`	`+ if stats.SessionCountJetbrains == 0 && stats.SessionCountVscode == 0 && stats.SessionCountReconnectingPty == 0 && stats.SessionCountSsh == 0 {`
`141`	`141`	`return nil`
`142`	`142`	`}`
`143`	`143`