[Metrics UI] Refactor Process List API to fetch only top processes#84716
[Metrics UI] Refactor Process List API to fetch only top processes#84716Zacqary merged 10 commits intoelastic:masterfrom
Conversation
|
Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui) |
phillipb
left a comment
There was a problem hiding this comment.
Looks pretty good nice work. Question about the timerange of the process list query.
| { | ||
| range: { | ||
| [timefield]: { | ||
| gte: to - 60 * 1000, // 1 minute |
There was a problem hiding this comment.
Why only one minues? I'm seeing a difference between the total count of processes between this PR and the previous implementation. Wondering if this could be the reason.
There was a problem hiding this comment.
Primarily to optimize performance, but I think it makes sense given that the Inventory view presents a snapshot of the past 1 minute of data. Are you able to tell if the lower Total Processes count is because some of the processes had ended in the past 15 minutes, or if it's actually failing to pick up a bunch of still active processes?
There was a problem hiding this comment.
Should be fixed in the latest commit
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Test FailuresChrome X-Pack UI Functional Tests.x-pack/test/functional/apps/uptime/locations·ts.Uptime app with generated data Observer location displays less monitor availabilityStandard OutStack TraceMetrics [docs]Module Count
Async chunks
Distributable file count
History
To update your PR or re-run it, just comment with: |
…lastic#84716) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
…or-disable-searchable-snapshot-fields * 'master' of github.com:elastic/kibana: (29 commits) HTTP CRUD+ API for Index Patterns (elastic#83576) Don't list packages in devDependencies already listed in dependencies (elastic#85120) Remove unused devDependencies (elastic#85121) [ILM] Reposition form toggles (elastic#85143) [APM] Make sure jest script can be run from anywhere (elastic#85111) Add EuiButtonEmptyTo components (elastic#85213) skip flaky suite (elastic#85216) skip flaky suite (elastic#83775) skip flaky suite (elastic#83774) skip flaky suite (elastic#83773) skip flaky suite (elastic#83793) skip flaky suite (elastic#85215) skip flaky suite (elastic#85217) [Usage Collection] Kibana Overview Page UI Metrics (elastic#81937) [Alerting UI] Replaced AlertsContextProvider with KibanaContextProvider and exposed components in API (elastic#84604) skip flaky suite (elastic#85208) [Security Solutions][Detection Engine] Fixes cypress errors by using latest signals mapping (elastic#84600) Small fixes to Kibana search bar (elastic#85079) Change link text to say Fleet (elastic#85083) [Metrics UI] Refactor Process List API to fetch only top processes (elastic#84716) ... # Conflicts: # x-pack/plugins/index_lifecycle_management/public/application/sections/edit_policy/components/phases/hot_phase/hot_phase.tsx # x-pack/plugins/index_lifecycle_management/public/application/sections/edit_policy/components/phases/shared_fields/searchable_snapshot_field/searchable_snapshot_field.tsx


Summary
Closes #83968
Refactors the Process List API to:
Checklist
Delete any items that are not applicable to this PR.