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

[Metrics UI] Enhanced host details - Processes #80307

Closed
sorantis opened this issue Oct 13, 2020 · 11 comments · Fixed by #83477
Closed

[Metrics UI] Enhanced host details - Processes #80307

sorantis opened this issue Oct 13, 2020 · 11 comments · Fixed by #83477
Assignees
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@sorantis
Copy link

sorantis commented Oct 13, 2020

Add Properties tab to Enhanced Host Details

Screen Shot 2020-10-19 at 17 35 14

Process summary

  • system.process.summary.dead
  • system.process.summary.idle
  • system.process.summary.runnin
  • system.process.summary.sleeping
  • system.process.summary.stopped
  • system.process.summary.total
  • system.process.summary.zombie
  • system.process.summary.unknown

Process table

State Command Time CPU % Memory
system.process.state system.process.cmdline system.process.start_time system.process.cpu.total.pct system.process.memory.rss.pct

Expanded process row

  • system.process.cmdline (Full name)
  • process.pid
  • process.ppid
  • process.args
  • user.name
  • system.process.fd.open
  • CPU graph (last 15min)
  • Memory graph (last 15min)
  • For APM process: View in APM link

APM correlation

host.name, process.pid, @timestamp can be used to correlate a process with an APM service, that can place an APM agent language icon beside the process name on the table.

@sorantis sorantis added enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Oct 13, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@sorantis
Copy link
Author

@fearful-symmetry could verify that everything's correct?

@alex-fedotyev
Copy link

@formgeist, @sqren - I think that in 7.11 it would make sense to link this page to the new service overview with a kuery bar filter to match host name and PID.
What do you think?

@sorenlouv
Copy link
Member

sorenlouv commented Oct 27, 2020

I think that in 7.11 it would make sense to link this page to the new service overview with a kuery bar filter to match host name and PID.

I think that makes sense 👍
(I assume we are talking about the service inventory page - the one that shows a list of all services)

@sorantis
Copy link
Author

Related to querying APM: #80311 (comment)

@alex-fedotyev
Copy link

@sqren - I think that since this integration already has a particular PID on particular host, would it make more sense to land in the individual service overview landing page for that service?
This probably means a separate look up needed to be implemented to confirm that APM is collecting that for that PID ahead of the linking.

The user flow would look like this:

  • SRE is troubleshooting a high CPU alert on a particular host.
  • SRE identified that a particular Java process is consuming most of the CPU.
  • SRE switches to APM to troubleshooting that particular Java application.

@sorenlouv
Copy link
Member

sorenlouv commented Oct 28, 2020

This probably means a separate look up needed to be implemented to confirm that APM is collecting that for that PID ahead of the linking.

So given a particular PID, link to a service? We can do that if PIDs are unique across all services. But I'm not sure they are, in which case we could have several services matching the same PID. Perhaps if we combine host + PID it has a higher chance of finding a uniquely matching service?

We could make a link like this, that would redirect to the correct service:

/app/apm/link-to/service?host=my-host&pid=my-pid 

# Will result in a redirect to: /app/apm/my-java-service

@Zacqary Zacqary self-assigned this Oct 29, 2020
@alex-fedotyev
Copy link

Agreed.
This would have to be HOST + PID type of filter as well as time information.
In this use case, user is starting on the host and looking at the list of processes looking for highest consumption... so processes are expected to be unique.

@Zacqary
Copy link
Contributor

Zacqary commented Nov 11, 2020

Need some guidance on how to format the running time column. A lot of processes I'm running into during testing are active for multiple hours, so I had to chop off the milliseconds display for them. But listing them along with entries that contain milliseconds looks confusing:

Screen Shot 2020-11-11 at 11 32 12 AM

Should we just drop milliseconds and stick with HH:MM:SS for consistency? Or something else?

@Zacqary
Copy link
Contributor

Zacqary commented Nov 11, 2020

Screen Shot 2020-11-11 at 11 38 13 AM

Are those little lines supposed to be dynamically generated from the data, or are they just graphics?

@sorantis
Copy link
Author

  1. we can drop the milliseconds
  2. these are real data points

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants