Fix issue #6511 - CLI listing formatting ignored when passing --search #6531
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.
This pull request addresses issue #6511, which concerns the CLI listing formatting being ignored when the
--search
option is used with theonevm list
command.The problem was pinpointed to a section of the
onevm
command located insrc/cli
. The issue occurred in the command definition block for:list
, where the conditional logic did not account for--json
and--yaml
formatting options when the--search
flag was used. Specifically, the block that needed modification was as follows:I introduced conditional branches to handle the
--json
and--yaml
options:With these additions, the onevm list command now respects the --json and --yaml flags in conjunction with the --search flag, providing output in the expected format.