Call the install log 'install log' rather than 'curtin log'
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
High
|
Newell Jensen | ||
curtin |
Fix Released
|
High
|
Newell Jensen | ||
curtin (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Trusty |
Fix Released
|
Medium
|
Unassigned | ||
Utopic |
Fix Released
|
Medium
|
Unassigned | ||
Vivid |
Fix Released
|
Medium
|
Unassigned |
Bug Description
=== Begin SRU Information ===
[Impact]
The impact is for MAAS's UI, from a consistency perspective. In newer
versions of MAAS, certain files from the system during installation are
collected and made available in the UI. One of those was curtin's
log file 'curtin_
The impact is then that in debugging, the user would have to know if a
install used curtin or d-i in order to investigate the correct file name.
This is arguably of trivial importance, but the change to the code and impact
are also trivial.
[Test Case]
Install system with curtin in MAAS.
Check the install log on the node's page to see if the file is 'install.log'
or 'curtin_
[Other Information]
The version of curtin in 14.04 did not create any log by itself, so the
user of 14.04 version of curtin will not have any expectation on the file name.
The version of 14.10 in utopic would have created /var/log/
So, the exposed expectation is in users of 14.10 curtin who were using curtin
outside of MAAS (those using from maas aren't generally going looking for
log files on the ephemeral install system).
=== End SRU Information ===
sabdfl requested the curtin install log on a node's page not surface the name 'curtin' - it should just be called 'install log'.
Related branches
- Blake Rouse (community): Approve
-
Diff: 35 lines (+5/-2)2 files modifiedcurtin/commands/install.py (+1/-1)
curtin/reporter/__init__.py (+4/-1)
- Christian Reis (community): Approve
- Raphaël Badin (community): Approve
- Julian Edwards (community): Approve
-
Diff: 20 lines (+3/-0)1 file modifiedsrc/maasserver/views/nodes.py (+3/-0)
tags: | added: trivial ui |
Changed in maas: | |
assignee: | nobody → Newell Jensen (newell-jensen) |
Changed in curtin: | |
status: | New → Triaged |
importance: | Undecided → High |
assignee: | nobody → Newell Jensen (newell-jensen) |
Changed in curtin: | |
status: | Triaged → Fix Committed |
Changed in maas: | |
status: | Triaged → Fix Committed |
Changed in curtin: | |
status: | Fix Committed → Triaged |
Changed in maas: | |
status: | Fix Committed → Triaged |
Changed in maas: | |
status: | Triaged → In Progress |
Changed in curtin: | |
status: | Triaged → In Progress |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in curtin: | |
status: | In Progress → Fix Committed |
Changed in curtin (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in curtin (Ubuntu Trusty): | |
status: | New → Confirmed |
Changed in curtin (Ubuntu Utopic): | |
status: | New → Confirmed |
Changed in curtin (Ubuntu Trusty): | |
importance: | Undecided → Medium |
Changed in curtin (Ubuntu Utopic): | |
importance: | Undecided → Medium |
description: | updated |
Changed in maas: | |
status: | Fix Committed → Fix Released |
Changed in curtin: | |
status: | Fix Committed → Fix Released |
Let's see if we can get this in for 1.7.