-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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.
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
[Fleet] Add caching on epr package queries #160130
Comments
Pinging @elastic/fleet (Team:Fleet) |
We have changed the archive cache recently, so I'm not sure if this is still an issue. |
@juliaElastic I am wondering if can implement some browser caching to avoid those requests use something like I was thinking we could a
What do you think? |
Good idea, we could try first with Etag, as it seems Last-Modified is less accurate. |
I tried to play with the etag, but it seems a little harder than expected as the It seems we reduced a lot the number of EPR queries needed on the details package for example for the nginx package @juliaElastic I am wondering if we should just closed that issue |
Sounds good, I think we can close it. |
Related to #125794
Caching on
GET /api/fleet/epm/packages/<name>/<version>
API is not implemented, because it fetches info from kibana saved objects, as well as epr.When hitting the Settings page of an Integration, we are making the query to epr many times (seeing 20+ for one Settings page load). e.g.
https://epr.elastic.co/search?package=system&prerelease=true&kibana.version=8.8.2
This is very inefficient and causes unnecessary load on epr.
We could use a library like node-fetch-cache to cache for a
ttl
of10m
(similarly to other/epm
routes cache control headers).The text was updated successfully, but these errors were encountered: