-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Data/Search Sessions] Management UI #81707
[Data/Search Sessions] Management UI #81707
Conversation
0730d47
to
f9b0d2b
Compare
cb91db0
to
33a2c6d
Compare
5ce124c
to
aa20b63
Compare
e05ae8a
to
4a826e5
Compare
…oard Renamed search session test service
@elasticmachine merge upstream |
x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/actions/types.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/table/table.tsx
Outdated
Show resolved
Hide resolved
x-pack/test/send_search_to_background_integration/services/send_to_background.ts
Outdated
Show resolved
Hide resolved
I can't approve this since it is my own Pull Request, but all the changes made since I last worked on this look GREAT! I only have a couple things, which maybe are nits:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest changes LGTM!
* logging and error handling in session client routes * [Data] Background Search Session Management UI * functional tests * fix ci * new functional tests * fix fn tests * cleanups * cleanup * restore test * configurable refresh and fetch timeout * more tests * feedback items * take expiresSoon field out of the interface * move helper to common * remove bg sessions w/find and delete * add storybook * fix tests * storybook actions * refactor expiration status calculation * isViewable as calculated field * refreshInterval 10s default * list newest first * "Type" => "App" * remove inline view action * in-progress status tooltip shows expire date * move date_string to public * fix tests * Adds management to tsconfig refs * removes preemptive script fix * view action was removed * rename the feature to Search Sessions * Update x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/status.tsx Co-authored-by: Liza Katz <[email protected]> * Update x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/status.tsx Co-authored-by: Liza Katz <[email protected]> * add TODO * use RedirectAppLinks * code review and react improvements * config * fix test * Fix merge * Fix management test * @Dosant code review * code review * Deleteed story * some more code review stuffs * fix ts * Code review and cleanup * Added functional tests for restoring, reloading and canceling a dashboard Renamed search session test service * Don't show expiration time for canceled, expired or errored sessions * fix jest * Moved UISession to public * @tsullivan code review * Fixed import Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Christiane Heiligers <[email protected]> Co-authored-by: Liza Katz <[email protected]> Co-authored-by: Liza K <[email protected]>
* logging and error handling in session client routes * [Data] Background Search Session Management UI * functional tests * fix ci * new functional tests * fix fn tests * cleanups * cleanup * restore test * configurable refresh and fetch timeout * more tests * feedback items * take expiresSoon field out of the interface * move helper to common * remove bg sessions w/find and delete * add storybook * fix tests * storybook actions * refactor expiration status calculation * isViewable as calculated field * refreshInterval 10s default * list newest first * "Type" => "App" * remove inline view action * in-progress status tooltip shows expire date * move date_string to public * fix tests * Adds management to tsconfig refs * removes preemptive script fix * view action was removed * rename the feature to Search Sessions * Update x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/status.tsx Co-authored-by: Liza Katz <[email protected]> * Update x-pack/plugins/data_enhanced/public/search/sessions_mgmt/components/status.tsx Co-authored-by: Liza Katz <[email protected]> * add TODO * use RedirectAppLinks * code review and react improvements * config * fix test * Fix merge * Fix management test * @Dosant code review * code review * Deleteed story * some more code review stuffs * fix ts * Code review and cleanup * Added functional tests for restoring, reloading and canceling a dashboard Renamed search session test service * Don't show expiration time for canceled, expired or errored sessions * fix jest * Moved UISession to public * @tsullivan code review * Fixed import Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Christiane Heiligers <[email protected]> Co-authored-by: Liza Katz <[email protected]> Co-authored-by: Liza K <[email protected]> Co-authored-by: Tim Sullivan <[email protected]> Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Christiane Heiligers <[email protected]>
💔 Build Failed
Failed CI Steps
Test FailuresChrome X-Pack UI Functional Tests.x-pack/test/functional/apps/ml/data_frame_analytics/cloning·ts.machine learning data frame analytics jobs cloning supported by UI form classification job supported by the form opens the existing job in the data frame analytics job wizardStandard Out
Stack Trace
X-Pack Endpoint Functional Tests.x-pack/test/security_solution_endpoint/apps/endpoint/endpoint_list·ts.endpoint endpoint list when initially navigating to page finds data after load and pollingStandard Out
Stack Trace
X-Pack API Integration Tests.x-pack/test/api_integration/apis/ml/modules/setup_module·ts.apis Machine Learning modules module setup sets up module data for logs_ui_categories with prefix, startDatafeed true and estimateModelMemory trueStandard Out
Stack Trace
and 3 more failures, only showing the first 3. Metrics [docs]Module Count
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
Release note
Adds Search Sessions to the Kibana section of Stack Management to allow users to Inspect, Extend or Delete a saved Search Session
Summary
Part of #61738
Closes #61741
This PR implements a Management UI for Background Sessions.
TODO:
[ ] Use server-side table paging instead of in-memory table[ ] More actions support: cancel, extendlTODO in a later PR)Release Note
For Background Sessions, there is a new management section featured under Stack Management, to manage background sessions. From there, a user can navigate back to the app and restore the search session to view the async search result data.
Depends on: