feat(cli): add namespace enumeration capabilites#6132
Open
geovannewashington wants to merge 2 commits intomasterfrom
Open
feat(cli): add namespace enumeration capabilites#6132geovannewashington wants to merge 2 commits intomasterfrom
geovannewashington wants to merge 2 commits intomasterfrom
Conversation
|
Claude finished @geovannewashington's task in 5m 38s —— View job Code Review CompleteReviewed 7 files across code quality, security, testing, Go patterns, and architecture — no issues found. The code looks good as-is. Highlights:
If you push additional changes and want a new review, tag |
This command lists all namespaces in the system and supports an optional quiet mode (-q). In quiet mode, it outputs a single field, making it convenient for shell operations such as command substitution. The command allows selecting either the 'name' or 'tenant-id' field. When quiet mode is not used, it formats the output neatly in a table.
geovannewashington
force-pushed
the
feature/namespace-enumeration
branch
3 times, most recently
from
148da01 to
971b03f
Compare
This change introduces a new 'namespace inspect' command to the CLI, allowing users to retrieve detailed information about a specific namespace in a structured and human-readable format. This addition complements the 'namespace list' functionality by enabling deeper inspection of individual entries, improving overall usability of the CLI when managing namespaces.
geovannewashington
force-pushed
the
feature/namespace-enumeration
branch
from
971b03f to
2b2d8f3
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 changed
Adds namespace enumeration capabilities to the CLI, introducing the namespace list command with support for quiet mode and field selection. Also introduces the namespace inspect command, designed to integrate naturally with the listing workflow by enabling deeper inspection of individual entries.
The inspect functionality is inspired by tools like the Docker CLI, following the pattern of separating resource listing from detailed inspection.
Why
Managing namespaces requires both quick visibility and detailed introspection. The list command provides fast enumeration, while inspect enables deeper analysis of a specific namespace when needed. This improves usability and aligns the CLI with familiar patterns used in other tools.
How to test
make start./bin/cli namespace listor./bin/cli namespace lsto verify table output./bin/cli namespace ls -qor--quietto verify quiet mode./bin/cli namespace ls -q tenant-idto verify field selection./bin/cli namespace inspect <name>to verify detailed output./bin/cli namespace inspect $(./bin/cli namespace ls -q | sed -n '1p')./bin/cli namespace inspect --tenant-id $(./bin/cli namespace ls -q tenant-id | sed -n '1p')