Skip to content

Comments

feat(show): unify volume status reporting with structured enum and description#85

Merged
imlk0 merged 1 commit intomasterfrom
unified-status
Feb 3, 2026
Merged

feat(show): unify volume status reporting with structured enum and description#85
imlk0 merged 1 commit intomasterfrom
unified-status

Conversation

@imlk0
Copy link
Collaborator

@imlk0 imlk0 commented Feb 2, 2026

  • Replace scattered boolean fields (device_exists, needs_initialize, initialized, opened) in JSON and table output with a single, machine-readable status field using the new VolumeStatusKind enum.
  • Introduce human-readable description field to explain current volume state.
  • Update table output to show only the unified Status column with color coding.
  • Restructure JSON output to wrap volumes in a top-level volumes array.
  • Add determine_status() method on VolumeConfig to encapsulate status logic.
  • Align init command to use the same status determination for validation.

…scription

- Replace scattered boolean fields (`device_exists`, `needs_initialize`,
  `initialized`, `opened`) in JSON and table output with a single,
  machine-readable `status` field using the new `VolumeStatusKind` enum.
- Introduce human-readable `description` field to explain current volume state.
- Update table output to show only the unified `Status` column with color coding.
- Restructure JSON output to wrap volumes in a top-level `volumes` array.
- Add `determine_status()` method on `VolumeConfig` to encapsulate status logic.
- Align `init` command to use the same status determination for validation.

Signed-off-by: Kun Lai <laikun@linux.alibaba.com>
@imlk0 imlk0 merged commit 3c62c78 into master Feb 3, 2026
25 of 26 checks passed
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