-
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
[Infra] Comparing Hosts within a Service #173813
Comments
@smith - Do you think this is something we should consider with the on-going work in the APM <> Hosts project or should we prioritise this for another project? (cc @neptunian) |
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
@smith I've moved this issue to the Hosts to GA phase 2 so we don't add to the scope of the APM <> Hosts project. |
@smith - was just discussing this with @neptunian (coincidentally) and she thought it made sense to do this as part of the APM <> Services project (i.e. whilst she's doing it now). Do you think you could have a chat and see if it's more efficient to tackle this now? I'd prefer to try and do it now if it's more efficient to do it now (rather than later) - I'm hoping this can be part of getting to Hosts to GA (so doing it in the most time efficient way is my preference) |
Since this is basically just adding the option to filter back to the hosts page on service.name, i'd rather at least get the final design for this going so i'm not building the UI as a list of pills and then having to change it immediately to a table or something. I think it makes sense to have it in the same project. |
@kkurstak - see @neptunian's comment above. Just to add on our brief discussion on this, I think it's important that the user can quickly filter to see all hosts the services running on without needing to click through to the service. For example, if you see the CPU is high on a host - one task would be to quickly check how this is running across all hosts without needing to go into the service...especially if you're a user who focuses on infrastructure:
I think there should be an option for a user to get to a list of the hosts the service is running on and compare metrics (i.e. the host list) without needing to go through other screens first. Does this make sense? |
Updatecc @neptunian @kkurstak @smith After speaking with @smith yesterday, we agreed to include this within the project scope of the Seamless Hosts <> APM as it saves time overall. Here are the main points:
@neptunian - Will that be OK for you? If @kkurstak needs to reach out to collaborate with you on it - do you think that would be OK if it's a light touch? |
UpdateHey @neptunian (cc @smith), @kkurstak and I have been looking at some possible designs on this and would love your help refining these. Karolina is away from Wednesday (and there is no overlap for a call) so she will share the design ideas here for collaboration. Could we work async on this Mon/Tue to see what makes the most sense? |
Hey Team, the designs are available to review under this link. They are not perfect, a quick hi-fi sketch of what we were thinking of. |
Hey @kkurstak (cc @neptunian), I walked through the designs with @smith and although I loved the designs, we agreed that the scope is a little to big to try and tackle...it would be more work than can be justified right now. Updated scopeBased on the engineering feedback, we'd like to reduce the scope down significantly to:
*this is because Sandra will actually be delivering the existing 'badge' design before we work on this issue so it's simplest just to add a simple filtering mechanism to it @kkurstak - Do you think you could propose a new design based on the badge design but with some kind of filtering icon? I'm wondering whether we should think about adopting the same kind of icons users are used to in Discover (given almost all users will be familiar with filtering in Discover): Either way, I think we probably want to follow the same design pattern between the metadata tab filtering and this new functionality in the service filtering we're proposing here so users know what to expect. The only problem here is that the Discover functionality doesn't align with how we're filtering in the metadata tab: Do you think we should continue the same filtering pattern we see in the metadata tab in this new functionality or do you think we should re-think whether we want to adopt the discover pattern both in this new service issue and updating the metadata filtering pattern too? My thinking here is that users should see familiar patterns around the whole UI where we can facilitate this. What do you think? |
Hey @smith - I moved this to hosts MVP phase 2 so that we can close the Hosts <> APM project to simplify things...hope that's OK: |
@roshan-elastic , I have some questions about this feature
What happens when we start using the flyout in other pages? Will we disable this feature or redirect the user to the hosts view? Applying filters to the Hosts View from the flyout can create an undesired dependency between them, that will make it harder to reuse the flyout elsewhere. Besides, it doesn't seem like too much effort for users to close the flyout and use the Service Name filter in the Hosts view.
Instead of showing just a count, why don't we show a table similar to what APM shows? This got me thinking whether we could reuse APM functionality in the flyout. Or even if we should just redirect the user to APM and improve the navigation flow. Showing other instances that are running a giving service sounds like APM's domain |
Hey @crespocarlos - good thinking. Maybe we should take a step back on this and wait to see how we show the relationships between services, containers and hosts? We'll want to update the host view to show the container and the services per container too...Karolina will be doing some design on this at some point so perhaps we should defer until then? We may find we close this ticket as a result. WDYT? |
Hey @roshan-elastic - I'm going to get to this now. This issue was missing the Design project badge and I was not seeing this on my board, apologies. |
Closing this as it is superseded by the following parent ticket: Helpful as a reference to the above issue though. |
Problem
When SREs debug problems with a Host which are running services, it is often important to check whether the issue is specific to the host or if it is common to other hosts running the same service. This helps them figure out if they need to dig further into the Host itself or whether they should start investigating the service:
Example flow
![image](https://private-user-images.githubusercontent.com/117740680/292159208-3bb81bd5-2d4f-4908-a967-36c25ce70095.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxMDA1NjMsIm5iZiI6MTczOTEwMDI2MywicGF0aCI6Ii8xMTc3NDA2ODAvMjkyMTU5MjA4LTNiYjgxYmQ1LTJkNGYtNDkwOC1hOTY3LTM2YzI1Y2U3MDA5NS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwOVQxMTI0MjNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03OTliMjA0YzhmOWQyMTQ5NzIwZmY5YjE3MTY5NjFkZjA4OTY0ODkzNjgyN2UyOWRhYTNhZjhkNTNjOTYyYmNiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.R0stytOvreIcKXNpaze76QQUqw30it1gMNZyWKf6R1w)
Currently, we allow users to filter by a service in the Hosts List an compare them:
Comparing all hosts running a service
![comparing hosts](https://private-user-images.githubusercontent.com/117740680/292160476-70b116ec-0b2d-4cc6-bfa2-4d060740c7ca.gif?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxMDA1NjMsIm5iZiI6MTczOTEwMDI2MywicGF0aCI6Ii8xMTc3NDA2ODAvMjkyMTYwNDc2LTcwYjExNmVjLTBiMmQtNGNjNi1iZmEyLTRkMDYwNzQwYzdjYS5naWY_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwOVQxMTI0MjNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03MGI0YzcwMDY3NzRlN2I1MDA5NmZjYjRlMTNjYTdjZjY0N2NjNWM1NDlkZjc3ZWI3OThlNDAyMTdmNmM3ZTU4JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.jXpt3N5r7eYWp24gS5F_aZpoFcoJTXBVuHBvmWsOUoc)
😢 Pain-point
Our pain-point is that if a user is viewing a single host via the fly-out or the full screen detail view, they either:
(a) may not find this functionality
(b) have to find spend a few clicks to do the comparison
For example:
Viewing fly-out-screen view of a host
Viewing full-screen view of a host
This is slightly challenging as the user isn't in the Host List at the moment, so we can't simply apply a filter to the host list running in the background.
Solution Proposal
Can we allow users to simply click on something in the host fly-out/detail views which will take them directly to the Host List filtered by the service they selected?
Users should be able to easily compare all the hosts that a service is running on
![image](https://private-user-images.githubusercontent.com/117740680/292163061-2774584f-de27-4035-ae6a-d3e0defd65a6.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxMDA1NjMsIm5iZiI6MTczOTEwMDI2MywicGF0aCI6Ii8xMTc3NDA2ODAvMjkyMTYzMDYxLTI3NzQ1ODRmLWRlMjctNDAzNS1hZTZhLWQzZTBkZWZkNjVhNi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwOVQxMTI0MjNaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT02MTVkMWE5OTc1Y2FhOTg1NmVjNzRlOTY3NzU1NmViODYyZTFjMmJkYzlmZDJhMzY5ZWZkYjQxMmFlMWM4OTdkJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.4vlaky1MSXG9---qTaZgCfJomk6IpHAR8EPZ00DInpo)
🎨 Latest Designs
Figma designs.
✔️ Acceptance criteria
TBC
The text was updated successfully, but these errors were encountered: