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

[Infra] Rename 'Docker Containers' to 'Containers' in Inventory and Alerts #180806

Closed
jennypavlova opened this issue Apr 15, 2024 · 5 comments
Closed

[Infra] Rename 'Docker Containers' to 'Containers' in Inventory and Alerts #180806

jennypavlova opened this issue Apr 15, 2024 · 5 comments
Assignees
@jennypavlova
Labels
enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability needs-refinement A reason and acceptance criteria need to be defined for this issue Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@jennypavlova
Copy link
Member

The current infra and alerts view should show 'Containers' instead of 'Docker Containers' in the UI

Note: The Container flyout will be replaced with the asset details

Image

Image

AC

  • 'Docker Containers' should be reworded as 'containers' in Alerting and Inventory
@jennypavlova jennypavlova added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Apr 15, 2024
@jennypavlova jennypavlova self-assigned this Apr 15, 2024
@elasticmachine
Copy link
Contributor

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

@jennypavlova jennypavlova added needs-refinement A reason and acceptance criteria need to be defined for this issue enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability labels Apr 15, 2024
@MiriamAparicio MiriamAparicio mentioned this issue May 1, 2024
Merged
MiriamAparicio added a commit that referenced this issue May 16, 2024
@MiriamAparicio
[ObsUx] [Infra] Change container details view with asset details view (…
a4579a7 
…#180436)

Part of #179844

### In this PR
- From Inventory, open asset details page view for Containers
- Show overview tab with CPU and Memory KPIs and metric charts
- Metadata tab with old fields, more metadata fields will be shown in
follow-up PR
- Added links to container metrics documentation, currently there are no
docs for K8s metrics just for docker containers

#### How to test
- The feature is under a FF, on inventory page go to settings and enable
`Container view`
- In containers inventory, select a container and click on 'Docker
container metrics' link (there's an
[issue](#180806) to reword this
links as K8s containers are also shown)
- Container details page should be shown with overview and metadata tabs
- On overview tab KPIs for CPU and Memory and Metrics section with CPU
and Memory charts should be displayed
<img width="937" alt="image"
src="https://github.com/elastic/kibana/assets/31922082/d2f25f2a-ea4f-4516-b216-38464682fd14">
@jennypavlova
Copy link
Member Author

I am closing this issue for now based on the comment here

@jennypavlova jennypavlova closed this as not planned Won't fix, can't repro, duplicate, stale May 17, 2024
@smith
Copy link
Contributor

smith commented May 18, 2024

We have an existing issue for issues with non-docker containers: #100229 cc @roshan-elastic.

@smith
Copy link
Contributor

smith commented May 18, 2024

Another related issue: #134331

@smith
Copy link
Contributor

smith commented May 18, 2024

And another: #133125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jennypavlova jennypavlova

Labels
enhancement New value added to drive a business result Feature:ObsHosts Hosts feature within Observability needs-refinement A reason and acceptance criteria need to be defined for this issue Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@smith @jennypavlova @elasticmachine