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

[Infrastructure UI] Show processes on a single hosts #150907

Closed
jennypavlova opened this issue Feb 10, 2023 · 4 comments · Fixed by #153866
Closed

[Infrastructure UI] Show processes on a single hosts #150907

jennypavlova opened this issue Feb 10, 2023 · 4 comments · Fixed by #153866
Assignees
@jennypavlova
Labels
beta Required for a feature to move to beta Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.8.0

Comments

@jennypavlova
Copy link
Member

jennypavlova commented Feb 10, 2023 •
edited
Loading

📓 Summary

During their investigation, SREs want to see processes running on their hosts. We want to achieve a similar experience to the inventory view when a single host is selected and the Processes tab is opened.

image

🎨 Design

Figma

👨‍💻 Implementation hints

  • In order to get the processes running on a single host we can use the process list API: usePreocesList

✔️ Acceptance criteria

  • Processes running on a single host should be visible after a host selection
  • The overview of the processes running on the host should be displayed
  • The processes should be sortable
@botelastic botelastic bot added the needs-team Issues missing a team label label Feb 10, 2023
@jennypavlova jennypavlova added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Feb 10, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Feb 10, 2023
@mohamedhamed-ahmed
Copy link
Contributor

@jennypavlova are you planning to show only the processes tab or integrate the whole flyout?

@jennypavlova
Copy link
Member Author

@mohamedhamed-ahmed The story is still in process of refining and the design work is still ongoing. @kkurstak is designing it now so we are not sure if it will be a flyout and how it will look yet.
The information we plan to include for sure is the metadata, processes (scope of this issue), and link to APM.

@jennypavlova jennypavlova added Feature:Snapshot and Restore Elasticsearch snapshots and repositories UI needs-refinement A reason and acceptance criteria need to be defined for this issue Feature:ObsHosts Hosts feature within Observability and removed Feature:Snapshot and Restore Elasticsearch snapshots and repositories UI labels Feb 10, 2023
@mohamedhamed-ahmed
Copy link
Contributor

@jennypavlova Thank you 😊 will keep an eye on how this goes as I will be working on integrating the logs tab to the view and wanted to have an idea what the flyout will contain. Thank you :)

@jennypavlova jennypavlova removed needs design needs-refinement A reason and acceptance criteria need to be defined for this issue labels Feb 28, 2023
@jennypavlova jennypavlova mentioned this issue Mar 9, 2023
Merged
@roshan-elastic roshan-elastic added beta Required for a feature to move to beta v8.8.0 labels Mar 13, 2023
jennypavlova added a commit that referenced this issue Mar 23, 2023
@jennypavlova @kibanamachine @tonyghiani
[Infrastructure UI] Show metadata for a single host on the host UI (#…
6591aa9 
…152956)

Closes [#150893](#150893)
## Summary
This PR adds a flyout with single host metadata with an option to
open/close it with a click on an expand/minimize icon in the table.

⚠️ This PR doesn't include metadata filtering/actions, or processes tab
inside flyout (they will be added in follow-up issues). For now, the
metadata will be displayed and no actions will be available.
This PR will unblock
[#151010](#151010),
[#150907](#150907) and
[#150985](#150985)

## Testing
- Open the hosts view and click on the expand icon for a single host in
the table
  
<img width="1464" alt="image"
src="https://user-images.githubusercontent.com/14139027/224077010-71aece78-40d1-4a3a-90a6-8e699001b37a.png">

- The flyout should be visible with a preselected metadata tab
containing
  - Host name as the flyout title
  - Metadata in a table view with field and value columns
  
<img width="1807" alt="image"
src="https://user-images.githubusercontent.com/14139027/224048634-cd49aa0f-f1a5-4442-9fd0-f16cd4cb84da.png">
- The flyout can be closed using the close icon and the minimize icon or
can show a different host when another host is expanded
<img width="1727" alt="image"
src="https://user-images.githubusercontent.com/14139027/224084969-daa525c5-4ec4-4504-b072-4711db63fe18.png">

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Marco Antonio Ghiani <[email protected]>
@jennypavlova jennypavlova self-assigned this Mar 27, 2023
@jennypavlova jennypavlova mentioned this issue Mar 28, 2023
Merged
jennypavlova added a commit that referenced this issue Apr 3, 2023
@jennypavlova @crespocarlos @kibanamachine
[Infrastructure UI] Show processes on a single hosts (#153866)
ef4afdd 
Closes #150907

## Summary
This PR adds the Processes tab to the single host flyout. The component
is already implemented in Inventory so we can reuse it here

## Testing
- Go to hosts view
- Open the flyout for any host to see the single host details
- Click on the processes tab 
⚠️ If you want to see the processes summary on top (where the total
processes are displayed) you need inside your metricbeat modules yml
configuration to include the `process_summary` so your config should
include:
```
metricbeat.modules:
  - module: system
    metricsets:
      ..... other data  .......
      - process # Per process metrics
      - process_summary # Process summary
    ..... other data  .......
```
<img width="1913" alt="image"
src="https://user-images.githubusercontent.com/14139027/228534978-c38437e4-4279-4ad4-9fc8-5222cbd15c2e.png">

---------

Co-authored-by: Carlos Crespo <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jennypavlova jennypavlova

Labels
beta Required for a feature to move to beta Feature:Metrics UI Metrics UI feature Feature:ObsHosts Hosts feature within Observability Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v8.8.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Infrastructure UI] Show processes on a single hosts jennypavlova/kibana
5 participants
@mohamedhamed-ahmed @jennypavlova @elasticmachine @roshan-elastic @kkurstak