You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@stgraber
I have a question from the beginning: please clarify several things about the new keys:
refresh - Should this be an int flag or time.Duration?
columns - I assume this is a list of columns to be shown from the possible ones. Am I right? BTW, do you want to support some ordering for the columns?
format - I can only guess what this is (format for output values in the columns?). Can you please provide me with an example, or if it is some standard behavior, where it is used, so I can use it as a reference?
flagRefresh should be an int with a default value of 10. flagColumns should be similar to what we do in commands like incus list though at the beginning we'd just stick with the set of columns we have right now but make it possible show/hide them and indeed to re-order them. flagFormat should behave like our other list commands with the exception that for incus top, we'd likely just allow table and compact as the other formats (csv, json and yaml) don't make as much sense for an interactive tool.
So basically we should be able to do: incus top --refresh=30 --all-projects --columns=enDum --format=compact and get a live updated table that looks like:
Splitting that one off of #1026
Currently
incus top
can only be configured through shortcuts.It'd be good to have it support our standard
--format
and --columnsoptions as well as a
--refresh` to configure the refresh delay.The text was updated successfully, but these errors were encountered: