Update to syslinux 6 breaks TFTPboot

Bug #1351010 reported by John Stafford
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
High
Lucas Alvares Gomes
tripleo
Fix Released
High
jan grant

Bug Description

A major change was introduced to syslinux from version 4 to version 6. It introduced extra files as a requirement for PXE booting. See:

http://www.syslinux.org/wiki/index.php/Library_modules

Changed in tripleo:
status: New → Confirmed
assignee: nobody → Jon-Paul Sullivan (jonpaul-sullivan)
Revision history for this message
aeva black (tenbrae) wrote :

Debian jessie and sid are both using syslinux6 now. Ubuntu trusty is still using syslinux4.

Changed in ironic:
status: New → Confirmed
Dmitry Tantsur (divius)
Changed in ironic:
importance: Undecided → High
Revision history for this message
Ben Nemec (bnemec) wrote :

Does this require changes in tripleo, or does it just need to be fixed in Ironic?

Changed in tripleo:
status: Confirmed → Incomplete
Revision history for this message
Jon-Paul Sullivan (jonpaul-sullivan) wrote :

This needs changes in tripleo.

The changes are to ensure that the correct packages are installed, and that the tftpboot area is correctly setup before being used by ironic.

I am not sure that there are any ironic changes needed, as once the tftpboot area is correctly setup, things just start working again.

I have a working patchset to upload to show this.

Changed in tripleo:
status: Incomplete → Confirmed
Ben Nemec (bnemec)
Changed in tripleo:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-image-elements (master)

Fix proposed to branch: master
Review: https://review.openstack.org/117798

Changed in tripleo:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-image-elements (master)

Change abandoned by Jon-Paul Sullivan (jp_at_hp) (<email address hidden>) on branch: master
Review: https://review.openstack.org/117798
Reason: Should have been a new patchset to https://review.openstack.org/#/c/110259

Revision history for this message
Lucas Alvares Gomes (lucasagomes) wrote :

Perhpas we don't need change in the Ironic code but would be great to update the Ironic documentation to expose that difference between the syslinux versions

Changed in ironic:
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/118194

Changed in ironic:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/118194
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=1d54000e8690813516a0c2de336608356b5cb350
Submitter: Jenkins
Branch: master

commit 1d54000e8690813516a0c2de336608356b5cb350
Author: Lucas Alvares Gomes <email address hidden>
Date: Mon Sep 1 17:58:58 2014 +0100

    Update installation guide for syslinux 6

    Fedora 21 and the next Ubuntu will have syslinux 6 in them. In syslinux
    6 modules are not "stand alone" as it was in versions prior to 4, see:
    http://www.syslinux.org/wiki/index.php/Library_modules

    Partial-Bug: #1351010
    Change-Id: Ie6bb031c10a9f8cf978e2c5dbd643b4a19cc93d0

Revision history for this message
Dmitry Tantsur (divius) wrote :

I guess fixing the docs is the only thing we can do on Ironic side

Changed in ironic:
status: In Progress → Fix Released
Changed in tripleo:
assignee: Jon-Paul Sullivan (jonpaul-sullivan) → jan grant (jan-grant)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-image-elements (master)

Change abandoned by Jon-Paul Sullivan (jp_at_hp) (<email address hidden>) on branch: master
Review: https://review.openstack.org/110259

Ben Nemec (bnemec)
Changed in tripleo:
status: In Progress → Fix Released
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.