Skip to content

Fix: change frontend cache time#1889

Merged
MarceloRobert merged 1 commit intokernelci:mainfrom
MarceloRobert:fix/frontend-cache-time
May 6, 2026
Merged

Fix: change frontend cache time#1889
MarceloRobert merged 1 commit intokernelci:mainfrom
MarceloRobert:fix/frontend-cache-time

Conversation

@MarceloRobert
Copy link
Copy Markdown
Collaborator

@MarceloRobert MarceloRobert commented May 5, 2026

The frontend query cache can stale some requests, which in turn can show outdated numbers between treeListing, treeDetails and treeCommitHistory. Lowering the cache time will technically increase the number of requests, but this can be done without much concern since we have improved endpoint performance. Besides, without this risk for outdated numbers there is less chance for confusion in users when they see trees that are receiving data in real time.

Changes

  • Changed frontend request cache from 5 minutes to 30 seconds

Also removes eslint rule ignore

How to test

Go to any frontend page, check that if you make the same request within 30s it will get the data from cache (it won't perform the request again) and after 30s it will redo the request.

Closes #1890

@MarceloRobert MarceloRobert self-assigned this May 5, 2026
@MarceloRobert MarceloRobert added the Frontend Most or all of the changes for this issue will be in the frontend code. label May 5, 2026
Copy link
Copy Markdown
Contributor

@alanpeixinho alanpeixinho left a comment

Choose a reason for hiding this comment

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

LGTM

Also removes eslint rule ignore

Closes kernelci#1890
@MarceloRobert MarceloRobert force-pushed the fix/frontend-cache-time branch from 9acc6ca to 5ba02dc Compare May 6, 2026 11:34
@MarceloRobert MarceloRobert added this pull request to the merge queue May 6, 2026
Merged via the queue into kernelci:main with commit 9e88a21 May 6, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend Most or all of the changes for this issue will be in the frontend code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lower frontend request cache

3 participants