coder
diff --git a/‎coderd/database/queries.sql.go
Lines changed: 26 additions & 39 deletions b/‎coderd/database/queries.sql.go
Lines changed: 26 additions & 39 deletions
diff --git a/‎coderd/database/queries/insights.sql
Lines changed: 22 additions & 31 deletions b/‎coderd/database/queries/insights.sql
Lines changed: 22 additions & 31 deletions
@@ -143,40 +143,31 @@ FROM
 	insights;
 
 -- name: GetTemplateInsightsByTemplate :many
-WITH
-	-- The purpose of this CTE is to flatten the data so that a user is
-	-- only counted once per interval/template.
-	insights AS (
-		SELECT
-			template_id,
-			user_id,
-			-- See motivation in GetTemplateInsights for LEAST(SUM(n), 30).
-			LEAST(SUM(usage_mins), 30) AS usage_mins,
-			LEAST(SUM(ssh_mins), 30) AS ssh_mins,
-			LEAST(SUM(sftp_mins), 30) AS sftp_mins,
-			LEAST(SUM(reconnecting_pty_mins), 30) AS reconnecting_pty_mins,
-			LEAST(SUM(vscode_mins), 30) AS vscode_mins,
-			LEAST(SUM(jetbrains_mins), 30) AS jetbrains_mins
-		FROM
-			template_usage_stats
-		WHERE
-			start_time >= @start_time::timestamptz
-			AND end_time <= @end_time::timestamptz
-		GROUP BY
-			start_time, template_id, user_id
-	)
+WITH agent_stats_by_interval_and_user AS (
+	SELECT
+		date_trunc('minute', was.created_at) AS created_at_trunc,
+		was.template_id,
+		was.user_id,
+		CASE WHEN SUM(was.session_count_vscode) > 0 THEN 60 ELSE 0 END AS usage_vscode_seconds,
+		CASE WHEN SUM(was.session_count_jetbrains) > 0 THEN 60 ELSE 0 END AS usage_jetbrains_seconds,
+		CASE WHEN SUM(was.session_count_reconnecting_pty) > 0 THEN 60 ELSE 0 END AS usage_reconnecting_pty_seconds,
+		CASE WHEN SUM(was.session_count_ssh) > 0 THEN 60 ELSE 0 END AS usage_ssh_seconds
+	FROM workspace_agent_stats was
+	WHERE
+		was.created_at >= @start_time::timestamptz
+		AND was.created_at < @end_time::timestamptz
+		AND was.connection_count > 0
+	GROUP BY created_at_trunc, was.template_id, was.user_id
+)
 
 SELECT
 	template_id,
-	COUNT(DISTINCT user_id) AS active_users,
-	(SUM(usage_mins) * 60)::bigint AS usage_total_seconds, -- Includes app usage.
-	(SUM(ssh_mins) * 60)::bigint AS usage_ssh_seconds,
-	(SUM(sftp_mins) * 60)::bigint AS usage_sftp_seconds,
-	(SUM(reconnecting_pty_mins) * 60)::bigint AS usage_reconnecting_pty_seconds,
-	(SUM(vscode_mins) * 60)::bigint AS usage_vscode_seconds,
-	(SUM(jetbrains_mins) * 60)::bigint AS usage_jetbrains_seconds
-FROM
-	insights
+	COALESCE(COUNT(DISTINCT user_id))::bigint AS active_users,
+	COALESCE(SUM(usage_vscode_seconds), 0)::bigint AS usage_vscode_seconds,
+	COALESCE(SUM(usage_jetbrains_seconds), 0)::bigint AS usage_jetbrains_seconds,
+	COALESCE(SUM(usage_reconnecting_pty_seconds), 0)::bigint AS usage_reconnecting_pty_seconds,
+	COALESCE(SUM(usage_ssh_seconds), 0)::bigint AS usage_ssh_seconds
+FROM agent_stats_by_interval_and_user
 GROUP BY template_id;
 
 -- name: GetTemplateAppInsights :many