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

[Infra] Add Profiling Tab to the container views #184910

Open
jennypavlova opened this issue Jun 6, 2024 · 4 comments
Open

[Infra] Add Profiling Tab to the container views #184910

jennypavlova opened this issue Jun 6, 2024 · 4 comments
Labels
enhancement New value added to drive a business result stale Used to mark issues that were closed for being stale Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@jennypavlova
Copy link
Member

We want to add the Universal Profiling tab to the container details view (similar to the host view). See this comment for some context

Image

AC:

  • Add the profiling tab to the asset details flyout for containers
  • Add the profiling tab to the asset details page for containers
@jennypavlova jennypavlova added enhancement New value added to drive a business result Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jun 6, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@jennypavlova
Copy link
Member Author

jennypavlova commented Jun 13, 2024

@roshan-elastic I remember that we discussed that we should descope the profiling tab and not include it in the Container view (At least in the scope of this Epic). Should we move this issue to the backlog and remove it from the epic?

Some notes based on what I checked so far:
I checked the host view implementation and what we need to decide what we want to show for containers. Also, all the filters and links are currently set to use the host fields - they need to be changed to accept container id as well. We also need to check how to test profiling with container data and how to set it up locally/using obs cluster. So I think we will need some time to figure that out so I don't think we can be ready with that as part of this epic.

cc: @smith

@smith
Copy link
Contributor

smith commented Jun 13, 2024

@jennypavlova I've removed this from the epic for now.

@botelastic
Copy link

botelastic bot commented Dec 10, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@botelastic botelastic bot added the stale Used to mark issues that were closed for being stale label Dec 10, 2024
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 stale Used to mark issues that were closed for being stale Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Development

No branches or pull requests

3 participants