Skip to content

fix: xpk workload list should sum podSet counts#1106

Merged
jamOne- merged 1 commit intoAI-Hypercomputer:mainfrom
jamOne-:b/482104524-wildcard
Mar 11, 2026
Merged

fix: xpk workload list should sum podSet counts#1106
jamOne- merged 1 commit intoAI-Hypercomputer:mainfrom
jamOne-:b/482104524-wildcard

Conversation

@jamOne-
Copy link
Collaborator

@jamOne- jamOne- commented Mar 10, 2026

Description

Change fixes an issue where xpk workload list underreported TPU VM counts for complex workloads containing multiple podSets (such as Pathways jobs that separate head and worker roles into distinct ReplicatedJobs). Previously, the logic only extracted resource metrics from the very first podSet (e.g., the 1-VM head node), ignoring the rest of the deployment.

Issue

  • No real bug, just an improvement that Gemini found.

Testing

  • UT
  • manual

@jamOne- jamOne- marked this pull request as ready for review March 10, 2026 14:16
@jamOne- jamOne- force-pushed the b/482104524-wildcard branch from 8528e11 to 9571eee Compare March 10, 2026 15:06
@jamOne- jamOne- marked this pull request as draft March 10, 2026 15:07
@jamOne- jamOne- force-pushed the b/482104524-wildcard branch from 9571eee to 1617da4 Compare March 10, 2026 15:19
@jamOne- jamOne- marked this pull request as ready for review March 10, 2026 17:08
@jamOne- jamOne- added this pull request to the merge queue Mar 11, 2026
Merged via the queue into AI-Hypercomputer:main with commit a72a752 Mar 11, 2026
10 checks passed
@jamOne- jamOne- deleted the b/482104524-wildcard branch March 11, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants