feat(cli): render the describe view - #326
Draft
rogirun wants to merge 1 commit into
Draft
Conversation
rogirun
force-pushed
the
feat/cli-describe-renderer
branch
from
September 6, 2026 12:59
8bc3d59 to
061c0fc
Compare
rogirun
force-pushed
the
feat/cli-describe-view
branch
from
September 6, 2026 12:59
c300c49 to
7bb0144
Compare
rogirun
force-pushed
the
feat/cli-describe-renderer
branch
from
September 7, 2026 10:43
061c0fc to
4203a4c
Compare
rogirun
force-pushed
the
feat/cli-describe-view
branch
from
September 7, 2026 10:43
7bb0144 to
b7ee02d
Compare
Render the DescribeView as the four sections the command promises: a header naming the workload and the definition that resolved it, the component tree with its pod rows, the normalized phase, and a resource breakdown per component with the workload total last. The tree and the resource table go through one tab writer each, so component rows and pod rows share a column grid without hand-computed widths. --pod-limit defaults to showing every pod, the way kubectl-tree renders every descendant: a hidden pod is the one a reader most needs. When a limit is set, unhealthy pods sort first so truncation can never hide a failing pod, and the note says how many were hidden and how many unhealthy ones survived. The machine formats emit the view itself rather than the items/count envelope the list commands carry: an envelope says nothing about a single workload and costs every consumer an items[0] hop. RenderOne carries that in the generic renderer, so json and yaml stay one code path. Refs #206 Signed-off-by: Roee Gil <roee.gil@run.ai>
rogirun
force-pushed
the
feat/cli-describe-renderer
branch
from
September 7, 2026 12:20
4203a4c to
bb6fd11
Compare
rogirun
force-pushed
the
feat/cli-describe-view
branch
from
September 7, 2026 12:20
b7ee02d to
5dcd00c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Renders the view as header, tree, phase and per-component resources. --pod-limit defaults to showing every pod; when set, unhealthy pods sort first so truncation never hides a failing pod. The machine formats emit the view itself rather than the items/count envelope the list commands carry.
Part of a seven-PR stack implementing
kli describe(#206). Targetsfeat/cli-describe-view, which must merge first.Related issue(s)
Refs #206
Checklist
git commit -s)make check)🤖 Generated with Claude Code