Skip to content
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

Jump to bottom

[Fleet] Integration details performance - Package info loading #122560

Closed
nchaulet opened this issue Jan 10, 2022 · 3 comments · Fixed by #123509
Closed

[Fleet] Integration details performance - Package info loading #122560

nchaulet opened this issue Jan 10, 2022 · 3 comments · Fixed by #123509
Assignees
@nchaulet
Labels
performance Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@nchaulet
Copy link
Member

nchaulet commented Jan 10, 2022 •
edited
Loading

Description

Related to #118751

In progress

The first time we call the details API for a package could be extremely slow (~1s for the APM package, ~5s for the AWS one), I am currently trying to see what we are doing here that is that slow.

This example trace for AWS show the issue

Screen Shot 2022-01-11 at 9 24 30 AM

Here you see a 6s call to the EPR we are downloading the full package we probably do not need to do so, and can get only the package manifest.
@nchaulet nchaulet added the Team:Fleet Team label for Observability Data Collection Fleet team label Jan 10, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@nchaulet nchaulet mentioned this issue Jan 11, 2022
Closed
@nchaulet nchaulet self-assigned this Jan 20, 2022
@joshdover joshdover mentioned this issue Feb 7, 2022
Closed
@joshdover
Copy link
Contributor

@nchaulet Curious if you've been able to make any progress on this issue. We need to decide whether or not to include this in our 8.2 planning and it would be good to know the remaining effort to close this one out.

@nchaulet nchaulet mentioned this issue Feb 8, 2022
Merged
@nchaulet
Copy link
Member Author

nchaulet commented Feb 8, 2022

@joshdover Yes I have a draft PR for that #123509 I need to fix some tests and potentially add new ones but hopefully I can finish that this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nchaulet nchaulet

Labels
performance Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fleet] Get package info should not store the whole package nchaulet/kibana
3 participants
@nchaulet @joshdover @elasticmachine