Sep 1, 2025 · Aug 29, 2025 · Sep 1, 2025
diff --git a/coderd/database/dbauthz/dbauthz.go b/coderd/database/dbauthz/dbauthz.go
 }

 // Only used by metrics cache.
 func (q*querier)GetTemplateAverageBuildTime(ctx context.Context,argdatabase.GetTemplateAverageBuildTimeParams) (database.GetTemplateAverageBuildTimeRow,error) {
 func (q*querier)GetTemplateAverageBuildTime(ctx context.Context,arguuid.NullUUID) (database.GetTemplateAverageBuildTimeRow,error) {
 iferr:=q.authorizeContext(ctx,policy.ActionRead,rbac.ResourceSystem);err!=nil {
 return database.GetTemplateAverageBuildTimeRow{},err
 }
diff --git a/coderd/database/dbauthz/dbauthz_test.go b/coderd/database/dbauthz/dbauthz_test.go
 check.Args(arg).Asserts(rbac.ResourceSystem,policy.ActionRead).Returns(p)
 }))
 s.Run("GetTemplateAverageBuildTime",s.Mocked(func(dbm*dbmock.MockStore,_*gofakeit.Faker,check*expects) {
 arg:=database.GetTemplateAverageBuildTimeParams{}
 arg:=uuid.NullUUID{}
 dbm.EXPECT().GetTemplateAverageBuildTime(gomock.Any(),arg).Return(database.GetTemplateAverageBuildTimeRow{},nil).AnyTimes()
 check.Args(arg).Asserts(rbac.ResourceSystem,policy.ActionRead)
 }))
diff --git a/coderd/database/dbmetrics/querymetrics.go b/coderd/database/dbmetrics/querymetrics.go
diff --git a/coderd/database/dbmock/dbmock.go b/coderd/database/dbmock/dbmock.go
diff --git a/coderd/database/querier.go b/coderd/database/querier.go
diff --git a/coderd/database/queries.sql.go b/coderd/database/queries.sql.go
diff --git a/coderd/database/queries/templates.sql b/coderd/database/queries/templates.sql
 WHERE
 template_versions.template_id = @template_id AND
 (pj.completed_at IS NOT NULL) AND (pj.started_at IS NOT NULL) AND
 (pj.started_at > @start_time) AND
 (pj.canceled_at IS NULL) AND
 ((pj.error IS NULL) OR (pj.error = ''))
 ORDER BY
 workspace_builds.created_at DESC
 LIMIT 100
 )
 SELECT
 -- Postgres offers no clear way to DRY this short of a function or other
diff --git a/coderd/metricscache/metricscache.go b/coderd/metricscache/metricscache.go
 for _, template := range templates {
 ids = append(ids, template.ID)

 templateAvgBuildTime, err := c.database.GetTemplateAverageBuildTime(ctx, database.GetTemplateAverageBuildTimeParams{
 TemplateID:uuid.NullUUID{
 templateAvgBuildTime, err := c.database.GetTemplateAverageBuildTime(ctx,
 uuid.NullUUID{
 UUID:  template.ID,
 Valid: true,
 },
 StartTime: sql.NullTime{
 Time:  dbtime.Time(c.clock.Now().AddDate(0, 0, -30)),
 Valid: true,
 },
 })
 )
 if err != nil {
 return err
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -2792,7 +2792,7 @@ func (q *querier) GetTemplateAppInsightsByTemplate(ctx context.Context, arg data
		}

		// Only used by metrics cache.
		func (q*querier)GetTemplateAverageBuildTime(ctx context.Context,argdatabase.GetTemplateAverageBuildTimeParams) (database.GetTemplateAverageBuildTimeRow,error) {
		func (q*querier)GetTemplateAverageBuildTime(ctx context.Context,arguuid.NullUUID) (database.GetTemplateAverageBuildTimeRow,error) {
		iferr:=q.authorizeContext(ctx,policy.ActionRead,rbac.ResourceSystem);err!=nil {
		return database.GetTemplateAverageBuildTimeRow{},err
		}
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -3160,7 +3160,7 @@ func (s *MethodTestSuite) TestSystemFunctions() {
		check.Args(arg).Asserts(rbac.ResourceSystem,policy.ActionRead).Returns(p)
		}))
		s.Run("GetTemplateAverageBuildTime",s.Mocked(func(dbmdbmock.MockStore,_gofakeit.Faker,check*expects) {
		arg:=database.GetTemplateAverageBuildTimeParams{}
		arg:=uuid.NullUUID{}
		dbm.EXPECT().GetTemplateAverageBuildTime(gomock.Any(),arg).Return(database.GetTemplateAverageBuildTimeRow{},nil).AnyTimes()
		check.Args(arg).Asserts(rbac.ResourceSystem,policy.ActionRead)
		}))
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -203,11 +203,11 @@ JOIN provisioner_jobs pj ON
		WHERE
		template_versions.template_id = @template_id AND
		(pj.completed_at IS NOT NULL) AND (pj.started_at IS NOT NULL) AND
		(pj.started_at > @start_time) AND
		(pj.canceled_at IS NULL) AND
		((pj.error IS NULL) OR (pj.error = ''))
		ORDER BY
		workspace_builds.created_at DESC
		LIMIT 100
		)
		SELECT
		-- Postgres offers no clear way to DRY this short of a function or other
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -101,16 +101,12 @@ func (c *Cache) refreshTemplateBuildTimes(ctx context.Context) error {
		for _, template := range templates {
		ids = append(ids, template.ID)

		templateAvgBuildTime, err := c.database.GetTemplateAverageBuildTime(ctx, database.GetTemplateAverageBuildTimeParams{
		TemplateID:uuid.NullUUID{
		templateAvgBuildTime, err := c.database.GetTemplateAverageBuildTime(ctx,
		uuid.NullUUID{
		UUID: template.ID,
		Valid: true,
		},
		StartTime: sql.NullTime{
		Time: dbtime.Time(c.clock.Now().AddDate(0, 0, -30)),
		Valid: true,
		},
		})
		)
		if err != nil {
		return err
		}
Expand Down