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

[Uptime] Exploratory view: Mobile experience metrics names are inconsistent with APM #102700

Closed
formgeist opened this issue Jun 21, 2021 · 4 comments
Assignees
Labels
Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan-ok issue has passed test plan v7.14.0

Comments

@formgeist
Copy link
Contributor

Summary

We're using two different names for Latency and Throughput in the Exploratory view for the Mobile experience. Unless intentional, because the charts will be different, I think it makes sense to at least align on the names. The metrics are the same.

Screenshot 2021-06-21 at 10 12 49

Screenshot 2021-06-21 at 10 10 55

Additionally, it might be wise to sort them differently in the select field.

  • Latency
  • Throughput
  • Memory usage
  • CPU usage

Are we planning to add an option to break down Throughput by http.response.status_code so the user can get a similar view as from the APM transactions page?

cc @AlexanderWert @bryce-b @shahzad31 @sqren

@formgeist formgeist added Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.14.0 labels Jun 21, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@AlexanderWert
Copy link
Member

@formgeist I created #102711 to fix the names and order.

For including http.response.status_code into breakdown I think we would need to breakdown the KPI report into two different reports (one for transaction related metrics and one for System related), otherwise this label wouldn't make much sense for CPU and Memory usage.

Let's keep this enhancement plus others (for example adding selection of transaction name) for later, WDYT?

@formgeist
Copy link
Contributor Author

Sounds good - and thanks for opening the PR for fixing the names and the order in the selector 👍

@formgeist
Copy link
Contributor Author

Closed by #102711

@dominiqueclarke dominiqueclarke added the test-plan-ok issue has passed test plan label Jul 7, 2021
@dominiqueclarke dominiqueclarke self-assigned this Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan-ok issue has passed test plan v7.14.0
Projects
None yet
Development

No branches or pull requests

4 participants