Frontend: Node: Add pod usage tile for node view#5039
Frontend: Node: Add pod usage tile for node view#5039Vikaspal8923 wants to merge 1 commit intokubernetes-sigs:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Vikaspal8923 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
sniok
left a comment
There was a problem hiding this comment.
Ci is failing on the i18n check, please make sure you've updated the translations by running npm run i18n in the frontend folder
|
Done , i have updated translations . Let me know if any other changes needed |
|
could you please squash it into a single commit, translations should be in the same commit that changes them |
635bf7a to
383ea09
Compare
|
Done 👍 |
There was a problem hiding this comment.
Pull request overview
Adds a “Pod Usage” capacity tile to the Node details header, showing scheduled pod count vs the node’s allocatable/capacity pod limit.
Changes:
- Fetch node-scoped pods in the node details view using
fieldSelector=spec.nodeName=<node>. - Add a new
PodCapacityCircularCharttile and wire it into the node header charts section. - Introduce new i18n keys for “Pod Usage” and the
{{ used }} / {{ total }} Podslegend across locales.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/components/node/Details.tsx | Fetches pods for the node and renders the new Pod Usage tile in the header charts grid. |
| frontend/src/components/cluster/Charts/ResourceCharts.tsx | Implements PodCapacityCircularChart using TileChart. |
| frontend/src/components/cluster/Charts/index.ts | Re-exports the new pod capacity chart. |
| frontend/src/i18n/locales/en/translation.json | Adds English strings for the new Pod Usage tile. |
| frontend/src/i18n/locales/de/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/es/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/fr/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/hi/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/it/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/ja/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/ko/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/pt/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/ta/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/zh/translation.json | Adds new keys (currently empty values) for Pod Usage. |
| frontend/src/i18n/locales/zh-tw/translation.json | Adds new keys (currently empty values) for Pod Usage. |
383ea09 to
4647244
Compare
illume
left a comment
There was a problem hiding this comment.
Can you please have a look at that one break point review comment?
4647244 to
cad9a7b
Compare
|
can you take a look now ? |
Summary
This PR adds pod capacity usage to the node details view so users can see how many pods are scheduled on a node compared with that node's allocatable pod limit.
Related Issue
Fixes #5034
Changes
spec.nodeName=<node>field selector in the node details viewSteps to Test
Screenshots (if applicable)
Notes for the Reviewer
fieldSelector=spec.nodeName=<node-name>status.allocatable.podsand falls back tostatus.capacity.pods