Ubuntu
ubuntu-advantage-tools package

'ubuntu-advantage status': output formatting?

Bug #1719034 reported by Steve Langasek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-advantage-script
Fix Released
Unknown
ubuntu-advantage-tools (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

I like that there is now a 'status' subcommand. The current output looks like:

$ ubuntu-advantage status
livepatch: disabled (not available)

esm: disabled (not available)

fips: disabled (not available)
$

I think the output format could be improved in order to make it easier to scan.

 - vertical whitespace: either omit all blank lines, or include an additional blank line before the first line of output

 - horizontal whitespace: align on the start of the word 'disabled'?

 - ordering: why are these services listed in the order they are? they're not alphabetical, they're not in the order in which they were added to the tool, and they don't match the ordering in the --help output.

Steve Langasek (vorlon)
Changed in ubuntu-advantage-tools (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

When a service is enabled, the output contains more information. Where should we indent that?

Example:

ubuntu@87-69:~$ ubuntu-advantage status
livepatch: enabled
  kernel: 4.4.0-96.119-generic
  fully-patched: true
  version: ""

esm: disabled (not available)

fips: disabled

Changed in ubuntu-advantage-tools (Ubuntu):
status: New → Triaged
Bug Watch Updater (bug-watch-updater)
Changed in ubuntu-advantage-script:
status: Unknown → New
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Livepatch is the only status output that is multi-line for now, I think we can leave the indentation on those extra lines as is. It would look like this after aligning the enabled/disabled bits:

ubuntu@87-69:~$ ubuntu-advantage status
livepatch: enabled
  kernel: 4.4.0-1002.2-fips
  fully-patched: true
  version: ""
esm: disabled (not available)
fips: enabled

I'll work on the rest of your suggestions and see what it looks like.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This was addressed in master, and should be available in our daily PPA shortly: https://launchpad.net/~ahasenack/+archive/ubuntu/ubuntu-advantage-tools-devel

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-advantage-tools - 14

---------------
ubuntu-advantage-tools (14) bionic; urgency=medium

  * New upstream release:
    - repositories are only added after credentials are verified
      (LP: #1730361)
    - Livepatch MOTD script (LP: #1710976)
    - better "status" command output formatting (LP: #1719034)
    - sources.list.d files no longer contain credentials. The "auth.conf"
      facility is used instead. (LP: #1700611)
    - enabled Livepatch support for Bionic 18.04 LTS

 -- Andreas Hasenack <email address hidden> Tue, 06 Feb 2018 09:58:03 -0200

Changed in ubuntu-advantage-tools (Ubuntu):
status: Triaged → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in ubuntu-advantage-script:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.