[pve-devel] [pve-manager v2 0/5] pvesh: implement 'ls' command

Dietmar Maurer dietmar at proxmox.com
Thu Jul 26 13:17:10 CEST 2018


> OLD:
> 
> # pvesh ls
> Dr--- access
> Dr--- cluster
> Dr--- nodes
> Dr-c- pools
> Dr-c- storage
> -r--- version
> 
> (output size is 863 bytes vs 79 bytes, just as a side note)
> 
> But besides disliking of the default formatting choice, the code changes 

OK, changed 'ls' back to old format. Thanks for the feedback.

> from this and common look OK  after a shallow review (did not tested the
> ACME CLI yaml print stuff though).
> It'd be nice to have manager / common in a shape that we can safely bump
> them again - rolling out other bug fixes, with being sure no
> regressions on CLI stuff are introduced.
> How do you want to proceed with this? Do you want to apply this
> yourself? Or has anyone else experienced (@Wolfgang, @Fabian, @Dominik ?)
> any comments or time to review this?

Just commited my changes to pve-common and pve-manager.




More information about the pve-devel mailing list