Columnise find to reduce output width #12409
Merged
Conversation
This file contains 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
|
|
The output find is rather large, now that we have more and more populated fields, we're potentially showing to much information. Instead of just removing the fields that look unimportant, we should tailor it depending on the users needs. This PR introduces columns for the tabular field. Using --columns flag with the column alias seems like a decent approach. Other canonical tools also do this, so it isn't something new (LXD). With decent defaults it actually looks rather good as well.
SimonRichardson
force-pushed
the
charmhub-find-writer
branch
from
890288f
to
9c3cf51
Compare
|
|
Merged
jujubot
added a commit
that referenced
this pull request
Dec 9, 2020
#12427 Merge 2.9 with these PRs #12397 AddPendingResources charhub #12405 move resources api #12404 Update the upgrade step for 2.9 for charm origin #12401 Adds services to service watcher. #12407 Use model-configured default space for NetworkInfo bindings #12411 Save charm origin platform to state #12409 Columnise find to reduce output width #12400 Make UpdateLatestRevisions controller facade work with charmhub #12415 Add Juju-Metadata HTTP headers for charmhub Refresh requests #12410 Charmhub download progress #12413 Charmhub info optional flag #12419 Ensure NetworksForRelation never returns a CAAS container address #12417 fix intermittent failure in upgrade series test #12420 Remove state from common client #12422 Add path and description to ResourceRevision #12423 charmrevisionupdater: remove "handlers" indirection for saving resources #12425 Ensure process limits are correctly set with using juju-db snap No conflicts ## QA steps see PRs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The output find is rather large, now that we have more and more
populated fields, we're potentially showing to much information. Instead
of just removing the fields that look unimportant, we should tailor it
depending on the users needs.
This PR introduces columns for the tabular field. Using --columns flag
with the column alias seems like a decent approach. Other canonical
tools also do this, so it isn't something new (LXD).
With decent defaults it actually looks rather good as well.
QA steps
Default column output
$ juju bootstrap lxd test $ juju find kubernetesTailoured
$ juju bootstrap lxd test $ juju find --columns=nbvaS