Skip to content

fix: exclude deleted templates from metrics collection #17839

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
May 15, 2025
Merged

fix: exclude deleted templates from metrics collection #17839

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
fa348d0
test: added duplicate-template-names test
evgeniy-scherbina May 14, 2025
c062274
chore: exclude deleted templates from all queries
dannykopping May 14, 2025
d28fb15
test: improve duplicate-template-names test
evgeniy-scherbina May 14, 2025
9ba369d
fix: partially revert fix on db level
evgeniy-scherbina May 14, 2025
e42a083
fix: skip deleted templates on metrics collector level
evgeniy-scherbina May 14, 2025
d320cbc
fix: fix linter
evgeniy-scherbina May 14, 2025
4c1c469
fix: fix indentation
evgeniy-scherbina May 14, 2025
9e8bf7e
chore: restore deleted test cases
dannykopping May 15, 2025
b539ddc
chore: adding clarity around soft-deletion constraints
dannykopping May 15, 2025
9cc5049
chore: clear up metrics tests
dannykopping May 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore: exclude deleted templates from all queries 
deleted templates cannot have associated workspaces, so it is logically invalid to include them

Signed-off-by: Danny Kopping <dannykopping@gmail.com>
  • Loading branch information
@dannykopping @evgeniy-scherbina
dannykopping authored and evgeniy-scherbina committed May 14, 2025
commit c062274823170f091c425575d58b5271e78508ad
4 changes: 4 additions & 0 deletions coderd/database/queries.sql.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions coderd/database/queries/prebuilds.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ WHERE w.id IN (
AND b.template_version_id = t.active_version_id
AND p.current_preset_id = @preset_id::uuid
AND p.ready
AND NOT t.deleted
LIMIT 1 FOR UPDATE OF p SKIP LOCKED -- Ensure that a concurrent request will not select the same prebuild.
)
RETURNING w.id, w.name;
Expand All @@ -40,6 +41,7 @@ FROM templates t
INNER JOIN template_version_presets tvp ON tvp.template_version_id = tv.id
INNER JOIN organizations o ON o.id = t.organization_id
WHERE tvp.desired_instances IS NOT NULL -- Consider only presets that have a prebuild configuration.
AND NOT t.deleted
AND (t.id = sqlc.narg('template_id')::uuid OR sqlc.narg('template_id') IS NULL);

-- name: GetRunningPrebuiltWorkspaces :many
Expand Down Expand Up @@ -70,6 +72,7 @@ FROM workspace_latest_builds wlb
-- prebuilds that are still building.
INNER JOIN templates t ON t.active_version_id = wlb.template_version_id
WHERE wlb.job_status IN ('pending'::provisioner_job_status, 'running'::provisioner_job_status)
AND NOT t.deleted
GROUP BY t.id, wpb.template_version_id, wpb.transition, wlb.template_version_preset_id;

-- GetPresetsBackoff groups workspace builds by preset ID.
Expand Down Expand Up @@ -98,6 +101,7 @@ WITH filtered_builds AS (
WHERE tvp.desired_instances IS NOT NULL -- Consider only presets that have a prebuild configuration.
AND wlb.transition = 'start'::workspace_transition
AND w.owner_id = 'c42fdf75-3097-471c-8c33-fb52454d81c0'
AND NOT t.deleted
),
time_sorted_builds AS (
-- Group builds by preset, then sort each group by created_at.
Expand Down
Loading