Skip to content

Conversation

@skoeva
Copy link
Contributor

@skoeva skoeva commented Apr 16, 2025

This change adds a new icon displaying the status of each container in the pods list. Hovering over the icon gives important info about the container at a glance: name, status, reason, exit code, and start/finish times.

Fixes: #2689, follow-up from #3008

Testing

  • Navigate to the Pods page in Headlamp
  • Note the container status icon, and hover over to see additional info about the container

image

image

@skoeva skoeva added the frontend Issues related to the frontend label Apr 16, 2025
@skoeva skoeva self-assigned this Apr 16, 2025
@k8s-ci-robot k8s-ci-robot added do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 16, 2025
@k8s-ci-robot k8s-ci-robot requested review from illume and yolossn April 16, 2025 15:11
@skoeva skoeva changed the title frontend: Allow direct editing of resource fields frontend: pod: Add quick status for containers Apr 16, 2025
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 16, 2025
This change adds a new icon displaying the status of each container in
the pods list. Hovering over the icon gives important info about the
container at a glance: name, status, reason, exit code, and start/finish
times.

Signed-off-by: Evangelos Skopelitis <[email protected]>
@skoeva skoeva force-pushed the container-status-2 branch from ed04f44 to 78416ca Compare April 16, 2025 15:12
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Apr 16, 2025
@skoeva skoeva added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 16, 2025
Copy link
Contributor

@illume illume left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Very useful, thanks!

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: illume, skoeva

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 17, 2025
@illume illume merged commit ac1d06d into kubernetes-sigs:main Apr 17, 2025
11 of 12 checks passed
@illume illume added this to the v0.31.0 milestone Apr 24, 2025
@joaquimrocha joaquimrocha moved this from Queued to Done in Headlamp Release Plan / Roadmap Jun 24, 2025
@skoeva skoeva deleted the container-status-2 branch July 11, 2025 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. frontend Issues related to the frontend kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

Development

Successfully merging this pull request may close these issues.

Add quick status for each container (design TBD)

3 participants