kernel boot command line for baremetal assumes block device UUID is correct root

Bug #1177655 reported by Robert Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Invalid
Medium
Lucas Alvares Gomes
OpenStack Compute (nova)
Won't Fix
Undecided
Unassigned

Bug Description

It does this by checking blkid on the device it wrote the image too, but if the image contains lvm or any other translation layer, this won't be the actual UUID to use.

We may be better off with a LABEL that we can document every image to have (e.g. cloudimg-rootfs)

this can be worked around for now by editing the template

root=${ROOT} -> root=LABEL=cloudimg-rootfs

Tags: baremetal
Revision history for this message
Joe Gordon (jogo) wrote :

Since we are in the process of deprecating nova maremetal, we should focus work on ironic instead.

Changed in nova:
status: Triaged → Opinion
Revision history for this message
Dmitry Tantsur (divius) wrote :

Note for implementer: this will likely require a written spec

Changed in ironic:
status: New → Triaged
importance: Undecided → Medium
aeva black (tenbrae)
Changed in nova:
status: Opinion → Won't Fix
importance: High → Undecided
aeva black (tenbrae)
Changed in ironic:
milestone: none → kilo-rc1
Revision history for this message
John Stafford (john-stafford) wrote :

Devananda - I noticed that Dmitry indicated on this bug that it will likely require a spec.. Are we good with keeping this as a bug or should we push it to L?

Changed in ironic:
milestone: kilo-rc1 → liberty-1
Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

I've put up a spec (which is now merged) that allows you to PXE boot an image using a specific kernel command line for that image. See: https://review.openstack.org/#/c/177859/

With that you can boot from a different root filesystem.

The code is here (waiting for reviews): https://review.openstack.org/#/c/178229/

Changed in ironic:
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
status: Triaged → In Progress
Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

The bug is being closed because the fix is now being proposed as a spec

Changed in ironic:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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