Information provided by new curtin reporter, is not user friendly
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| curtin |
In Progress
|
Medium
|
Unassigned | ||
Bug Description
Information provided by the new curtin reporter is not user friendly at all. This information needs to be user friendly. The current inform being shown:
Node installation - curtin: executing late commands Tue, 01 Sep. 2015 19:58:24
Node installation - curtin: running 'maas' Tue, 01 Sep. 2015 19:58:24
Installation complete - Node disabled netboot Tue, 01 Sep. 2015 19:58:24
Node installation - curtin: running 'maas' Tue, 01 Sep. 2015 19:58:24
Node installation - curtin: running 'copy_etc' Tue, 01 Sep. 2015 19:58:24
Node installation - curtin: curtin command in-target Tue, 01 Sep. 2015 19:58:24
Node installation - curtin: curtin command in-target Tue, 01 Sep. 2015 19:58:23
Node installation - curtin: running 'copy_etc' Tue, 01 Sep. 2015 19:58:23
Node installation - curtin: executing late commands Tue, 01 Sep. 2015 19:58:23
Node installation - curtin: finalizing installation Tue, 01 Sep. 2015 19:58:22
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:58:22
Node installation - curtin: curtin command hook Tue, 01 Sep. 2015 19:58:22
Node installation - curtin: curtin command hook Tue, 01 Sep. 2015 19:58:22
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:58:21
Node installation - curtin: finalizing installation Tue, 01 Sep. 2015 19:58:21
Node installation - curtin: configuring installed system Tue, 01 Sep. 2015 19:58:21
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:58:21
Node installation - curtin: curtin command curthooks Tue, 01 Sep. 2015 19:58:21
Node installation - curtin: curtin command curthooks Tue, 01 Sep. 2015 19:57:00
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:59
Node installation - curtin: configuring installed system Tue, 01 Sep. 2015 19:56:59
Node installation - curtin: writing install sources to disk Tue, 01 Sep. 2015 19:56:59
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:59
Node installation - curtin: curtin command extract Tue, 01 Sep. 2015 19:56:59
Node installation - curtin: curtin command extract Tue, 01 Sep. 2015 19:56:27
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: writing install sources to disk Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: configuring network Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: curtin command net-meta Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: curtin command net-meta Tue, 01 Sep. 2015 19:56:26
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:25
Node installation - curtin: configuring network Tue, 01 Sep. 2015 19:56:25
Node installation - curtin: configuring storage Tue, 01 Sep. 2015 19:56:25
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:56:25
Node installation - curtin: curtin command block-meta Tue, 01 Sep. 2015 19:56:25
Node installation - curtin: curtin command block-meta Tue, 01 Sep. 2015 19:55:20
Node installation - curtin: running 'builtin' Tue, 01 Sep. 2015 19:55:20
Node installation - curtin: configuring storage Tue, 01 Sep. 2015 19:55:19
Node installation - curtin: preparing for installation Tue, 01 Sep. 2015 19:55:19
Node installation - curtin: running 'block_poweroff' Tue, 01 Sep. 2015 19:55:19
Node installation - curtin: running 'block_poweroff' Tue, 01 Sep. 2015 19:55:19
Node installation - curtin: preparing for installation Tue, 01 Sep. 2015 19:55:19
Node installation - curtin: curtin command install Tue, 01 Sep. 2015 19:55:19
In reality, the type of events that curtin should be showing should actually provide more iunformation as to what's going on, i.e.
curint installing packages package1, package2, package3.
curtin partitioning disk sda
curtin downloading the image from http://
curtin installing image [image-name] into /dev/sda1
curtin running early_command block_poweroff
curtin running early_command 12345
curtin running late_command maas
etc.
Related branches
- Server Team CI bot: Needs Fixing (continuous-integration)
- curtin developers: Pending requested
-
Diff: 569 lines (+247/-60)10 files modifiedcurtin/commands/block_meta.py (+14/-6)
curtin/commands/curthooks.py (+29/-12)
curtin/commands/extract.py (+25/-21)
curtin/commands/install.py (+2/-2)
curtin/commands/main.py (+2/-2)
curtin/reporter/events.py (+31/-11)
curtin/reporter/handlers.py (+15/-1)
curtin/util.py (+2/-1)
doc/topics/reporting.rst (+6/-2)
tests/unittests/test_reporter.py (+121/-2)
- Scott Moser (community): Approve
- Server Team CI bot: Approve (continuous-integration)
-
Diff: 978 lines (+543/-141)15 files modifiedcurtin/commands/block_meta.py (+2/-1)
curtin/commands/curthooks.py (+21/-8)
curtin/commands/install.py (+18/-0)
curtin/reporter/events.py (+4/-9)
curtin/util.py (+12/-6)
tests/unittests/test_pack.py (+1/-0)
tests/unittests/test_reporter.py (+4/-8)
tests/vmtests/__init__.py (+74/-6)
tests/vmtests/helpers.py (+4/-1)
tests/vmtests/report_webhook_logger.py (+187/-0)
tools/curtin-log-print (+197/-0)
tools/report-webhook-logger (+0/-100)
tools/report_webhook_logger (+6/-0)
tools/run-pep8 (+6/-1)
tools/run-pyflakes (+7/-1)
| description: | updated |
| description: | updated |
| description: | updated |
| Changed in curtin: | |
| importance: | Undecided → Medium |
| status: | New → Confirmed |
| Scott Moser (smoser) wrote | #1 |
| Wesley Wiedenmeier (wesley-wiedenmeier) wrote | #2 |
| Changed in curtin: | |
| status: | Confirmed → In Progress |
The discussion we had in Barcelona was that we should
a.) Add a 'Level' attribute to the reporting dictionary
b.) Generally clean up the messages to be more user friendly, and set Level appropriately.