Skip to content

docs(cli): document the describe command in the design - #329

Draft
rogirun wants to merge 1 commit into
feat/cli-describe-file-modefrom
docs/cli-describe
Draft

docs(cli): document the describe command in the design#329
rogirun wants to merge 1 commit into
feat/cli-describe-file-modefrom
docs/cli-describe

Conversation

@rogirun

@rogirun rogirun commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Replaces the workload tree, status and resources sketches with describe, and brings the design document in line with what shipped: verb-first grammar, two-stage pod matching, and the WorkloadView struct describe emits.

Part of a seven-PR stack implementing kli describe (#206). Targets feat/cli-describe-file-mode, which must merge first.

Related issue(s)

Refs #206

Checklist

  • All commits are signed off with DCO (git commit -s)
  • New/modified files have SPDX license and copyright headers
  • Documentation updated (if applicable)
  • Tests pass (make check)
  • No proprietary or internal information included

🤖 Generated with Claude Code

@rogirun
rogirun force-pushed the feat/cli-describe-file-mode branch 2 times, most recently from c17c138 to a7e281c Compare September 7, 2026 10:43
Replace the workload tree, status and resources sketches with describe, which
supersedes all three: they answered three parts of one question, and a reader
who ran one usually wanted the others.

Bring the rest of the document in line with what shipped. The command grammar
is verb-first, not two top-level nouns. Pod matching is two stages, not one:
selectors say which role a pod plays and ownership says which workload it
belongs to, and the PodMatcher interface sketched here was never the shape that
worked. WorkloadView is the struct describe emits, with a note on why the
"3/4" ReadyCount it used to declare was rejected.

Refs #206

Signed-off-by: Roee Gil <roee.gil@run.ai>
@rogirun
rogirun force-pushed the feat/cli-describe-file-mode branch from a7e281c to e93d041 Compare September 7, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant