Skip to content

feat: increase metrics cache time#1934

Merged
gustavobtflores merged 1 commit into
kernelci:mainfrom
gustavobtflores:feat/increase-metrics-cache-time
Jun 10, 2026
Merged

feat: increase metrics cache time#1934
gustavobtflores merged 1 commit into
kernelci:mainfrom
gustavobtflores:feat/increase-metrics-cache-time

Conversation

@gustavobtflores

Copy link
Copy Markdown
Contributor

Increase the metrics endpoint cache time. Right now, metrics queries are expensive, and since they are aggregated over multiple days, we can keep them cached for longer.

@gustavobtflores gustavobtflores self-assigned this Jun 10, 2026
@gustavobtflores gustavobtflores added the Backend Most or all of the changes for this issue will be in the backend code. label Jun 10, 2026

@alanpeixinho alanpeixinho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

TopIssue,
)

METRICS_CACHE_TIMEOUT = 60 * 60 # 1 hour

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can go even higher here. Considering this page is still a prototype for design iteration.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, for sure, increased this to 6h

@gustavobtflores gustavobtflores force-pushed the feat/increase-metrics-cache-time branch from a56871b to 44261e1 Compare June 10, 2026 20:19
@gustavobtflores gustavobtflores added this pull request to the merge queue Jun 10, 2026
Merged via the queue into kernelci:main with commit a5e93fc Jun 10, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Most or all of the changes for this issue will be in the backend code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants