vmtest: s390x requires zipl command
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
curtin |
Fix Released
|
Medium
|
Unassigned | ||
curtin (Ubuntu) |
Fix Released
|
Low
|
Unassigned | ||
Xenial |
Fix Released
|
Low
|
Scott Moser | ||
Yakkety |
Fix Released
|
Low
|
Unassigned |
Bug Description
[Impact]
Installations of s390x systems will fail. s390x requires zipl to boot
and curtin assumed it was in the target image. The change is simply
to install it from the archive if it is not present.
[Test Case]
Run curtin vmtest on s390 as:
/tools/
[Regression Potential]
No regression potential on s390x as it simply did not work.
Errant code could accidently attempt installation of zipl on other
arch, but successful installation of other arch show that is not the
case.
[Other Info]
Before any s390x vmtests can be run on any regular basis the zipl command needs to be added as a dependency during install. Currently a basic test fails [1] with the output:
[ 114.303504] cloud-init[1482]: Running command ['chroot', '/tmp/tmpua0mww
[ 114.304453] cloud-init[1482]: chroot: failed to run command 'zipl': No such file or directory
[1] https:/
Related branches
- Joshua Powers (community): Approve
- Server Team CI bot: Approve (continuous-integration)
- curtin developers: Pending requested
-
Diff: 19 lines (+9/-0)1 file modifiedcurtin/commands/curthooks.py (+9/-0)
Changed in curtin: | |
status: | New → Confirmed |
Changed in curtin: | |
importance: | Undecided → Medium |
Changed in curtin (Ubuntu): | |
status: | New → Fix Released |
Changed in curtin (Ubuntu Xenial): | |
assignee: | nobody → Scott Moser (smoser) |
importance: | Undecided → Low |
status: | New → Confirmed |
description: | updated |
Changed in curtin: | |
status: | Confirmed → Fix Committed |
Changed in curtin (Ubuntu Yakkety): | |
status: | New → Confirmed |
importance: | Undecided → Low |
Changed in curtin (Ubuntu): | |
importance: | Undecided → Low |
tags: | added: verification-done-xenial verification-done-yakkety |
Hello Joshua, or anyone else affected,
Accepted curtin into yakkety-proposed. The package will build now and be available at https:/ /launchpad. net/ubuntu/ +source/ curtin/ 0.1.0~bzr460- 0ubuntu1~ 16.10.1 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification- failed. In either case, details of your testing will help us make a better decision.
Further information regarding the verification process can be found at https:/ /wiki.ubuntu. com/QATeam/ PerformingSRUVe rification . Thank you in advance!