Removal command output can be improved
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical Juju |
Fix Released
|
Medium
|
Anastasia | ||
| 2.6 |
Fix Released
|
Medium
|
Anastasia | ||
Bug Description
The output to destroy-model and kill-controller and destroy-controller is often exaggerated, and frankly, unsightly. I propose that any repeated line be replaced by a single dot at the end of the line. That is, a dot gets appended for every line that would otherwise get printed.
For example, this `destroy-model` output:
Destroying model
Waiting on model to be removed, 1 application(s), 1 volume(s), 1 filesystems(s)...
Waiting on model to be removed, 1 application(s), 1 volume(s), 1 filesystems(s)...
Waiting on model to be removed, 1 application(s), 1 volume(s), 1 filesystems(s)...
Waiting on model to be removed, 1 application(s)...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Waiting on model to be removed...
Model destroyed.
would become:
Destroying model
Waiting on model to be removed, 1 application(s), 1 volume(s), 1 filesystems(s)..
Waiting on model to be removed, 1 application(s)
Waiting on model to be removed.
Model destroyed.
Alternative suggestion: implement a CLI spinner for each unique line.
n.b. The phrase "Waiting on model to be removed" should be "Waiting for model to be removed".
| Anastasia (anastasia-macmood) wrote | #1 |
| Changed in juju: | |
| status: | New → In Progress |
| importance: | Undecided → Medium |
| assignee: | nobody → Anastasia (anastasia-macmood) |
| tags: | added: usability |
| Anastasia (anastasia-macmood) wrote | #2 |
| Tim McNamara (tim-clicks) wrote | #3 |
| Anastasia (anastasia-macmood) wrote | #4 |
| Anastasia (anastasia-macmood) wrote | #5 |
| Changed in juju: | |
| milestone: | none → 2.7-beta1 |
| Changed in juju: | |
| status: | In Progress → Fix Committed |
| Changed in juju: | |
| status: | Fix Committed → Fix Released |
Thank you, @Peter Matulis,
I am in the area and will address as a drive-by :)