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.

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

[ML] Fixes capabilities polling in manage page #163399

Conversation

jgowdyelastic
Copy link
Member

@jgowdyelastic jgowdyelastic commented Aug 8, 2023

On ML's management page a new capabilities poll is created every time the user changes tabs.
getMlGlobalServices gets called every time the jobs list renders, which creates a new MlCapabilitiesService which creates a new timer to poll for ml's capabilities.

This change moves the mlServices so it isn't called on each render.
It also bumps up the interval from 1 min to 5 mins as think 1 min is a bit too frequent.

@jgowdyelastic jgowdyelastic added bug Fixes for quality problems that affect the customer experience release_note:fix :ml v8.10.0 labels Aug 8, 2023
@jgowdyelastic jgowdyelastic self-assigned this Aug 8, 2023
@jgowdyelastic jgowdyelastic marked this pull request as ready for review August 8, 2023 12:39
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner August 8, 2023 12:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

Copy link
Contributor

@darnautov darnautov left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@jgowdyelastic jgowdyelastic enabled auto-merge (squash) August 9, 2023 16:27
@jgowdyelastic jgowdyelastic merged commit 6673e9d into elastic:main Aug 9, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 3.5MB 3.5MB +48.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jgowdyelastic

@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 9, 2023
bryce-b pushed a commit to bryce-b/kibana that referenced this pull request Aug 9, 2023
On ML's management page a new capabilities poll is created every time
the user changes tabs.
`getMlGlobalServices` gets called every time the jobs list renders,
which creates a new `MlCapabilitiesService` which creates a new timer to
poll for ml's capabilities.

This change moves the `mlServices` so it isn't called on each render.
It also bumps up the interval from 1 min to 5 mins as think 1 min is a
bit too frequent.
@jgowdyelastic jgowdyelastic changed the title [ML] Fixing capabilites polling in manage page [ML] Fixing capabilities polling in manage page Aug 14, 2023
@szabosteve szabosteve changed the title [ML] Fixing capabilities polling in manage page [ML] Fixes capabilities polling in manage page Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting bug Fixes for quality problems that affect the customer experience :ml release_note:fix v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants