Page MenuHomeVyOS Platform
Create Task
Maniphest T6244

Improve formatting in "show system uptime"
Closed, ResolvedPublicFEATURE REQUEST
Actions

Description

The output from

show system uptime

is very hard to parse quickly when reading as it doesn't space between weeks/days/hours/minutes/seconds:

tim@ferrari:~$ show system uptime
Uptime: 2w4d22h54m9s

It would be much nicer to have a space between all the units (I assume month and year are also part of it)

Example Output with "fix":

tim@ferrari:~$ show system uptime
Uptime: 2w 4d 22h 54m 9s

Details

Version
1.4.0-rc1
Is it a breaking change?
Perfectly compatible
Issue type
Cosmetic issue (typos etc.)

Related Objects

Event Timeline

tjh created this task.Apr 17 2024, 9:00 AM
tjh assigned this task to c-po.
Viacheslav triaged this task as Wishlist priority.Apr 17 2024, 9:02 AM
tjh updated the task description. (Show Details)Apr 17 2024, 9:11 AM
pasik subscribed.Apr 17 2024, 9:19 AM
c-po added a project: VyOS 1.5 Circinus.Apr 19 2024, 2:45 PM
c-po edited a custom field.
c-po changed Version from - to 1.4.0-rc1.
c-po added a comment.Apr 20 2024, 8:06 AM

https://github.com/vyos/vyos-1x/pull/3336

c-po changed the task status from Open to In progress.Apr 20 2024, 8:06 AM
c-po edited projects, added VyOS 1.4 Sagitta (1.4.0-epa3); removed VyOS 1.4 Sagitta.
c-po moved this task from Open to Finished on the VyOS 1.5 Circinus board.
Restricted Repository Identity mentioned this in rVYOSONEX5ebcbddb0842: Merge pull request #3336 from c-po/t6244-seconds_to_human.Apr 22 2024, 4:14 PM
c-po mentioned this in rVYOSONEX31b21d26751b: op-mode: T6244: add whitespace after time unit in "show system uptime".Apr 22 2024, 4:14 PM
c-po mentioned this in rVYOSONEX8d8f3137d174: vyos.utils: T6244: add support for year timebase in seconds_to_human().
c-po mentioned this in rVYOSONEX6e9cd8821ca0: vyos.utils: T6244: use list to build up result string.
Restricted Repository Identity mentioned this in rVYOSONEX2ccc0e105223: vyos.utils: T6244: use list to build up result string.Apr 22 2024, 4:15 PM
Restricted Repository Identity mentioned this in rVYOSONEX94780c9b5a68: vyos.utils: T6244: add support for year timebase in seconds_to_human().
Restricted Repository Identity mentioned this in rVYOSONEX8747c300e521: op-mode: T6244: add whitespace after time unit in "show system uptime".
Restricted Repository Identity mentioned this in rVYOSONEX626321ad1c3f: Merge pull request #3343 from vyos/mergify/bp/sagitta/pr-3336.Apr 22 2024, 5:01 PM
c-po closed this task as Resolved.Apr 24 2024, 5:47 AM
c-po moved this task from Need Triage to Finished on the VyOS 1.4 Sagitta (1.4.0-epa3) board.
dmbaturin renamed this task from Spacing of "Show System Uptime" hard to parse to Improve formatting in "show system uptime".May 11 2024, 6:58 PM
dmbaturin removed a project: VyOS 1.5 Circinus.