Merge lp:~rvb/maas/fix-bootloader-path into lp:~maas-committers/maas/trunk

Proposed by Raphaël Badin
Status: Merged
Approved by: Raphaël Badin
Approved revision: no longer in the source branch.
Merged at revision: 2445
Proposed branch: lp:~rvb/maas/fix-bootloader-path
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 14 lines (+2/-2)
1 file modified
src/provisioningserver/boot/pxe.py (+2/-2)
To merge this branch: bzr merge lp:~rvb/maas/fix-bootloader-path
Reviewer Review Type Date Requested Status
Graham Binns (community) Approve
Review via email: mp+223920@code.launchpad.net

Commit message

Fix bootloader locations.

Description of the change

Since the path to the bootloaders differs between releases, we just introduced a way to iterate over a list of possible locations (see https://code.launchpad.net/~julian-edwards/maas/utopic-syslinux-location/+merge/223851). Unfortunately, the path for Utopic was wrong: http://paste.ubuntu.com/7674610/. (For reference, this is for Trusty: http://paste.ubuntu.com/7674618/.) This branch fixes it.

To post a comment you must log in.
Revision history for this message
Graham Binns (gmb) :
review: Approve
Revision history for this message
Julian Edwards (julian-edwards) wrote :

On 20/06/14 23:36, Raphaël Badin wrote:
> -BOOTLOADER_DIRS = ['/usr/lib/syslinux', '/usr/lib/syslinux/bios']
> +# Ubuntu releases.)
> +BOOTLOADER_DIRS = ['/usr/lib/syslinux', '/usr/lib/syslinux/modules/bios']
>

Wow, this must have changed recently, because I ran up a utopic
Canonistack instance and verified where the file was!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/provisioningserver/boot/pxe.py'
2--- src/provisioningserver/boot/pxe.py 2014-06-20 03:07:03 +0000
3+++ src/provisioningserver/boot/pxe.py 2014-06-20 13:32:10 +0000
4@@ -32,8 +32,8 @@
5
6 # Possible locations in which to find the files. Search these in this
7 # order for each file. (This exists because locations differ across
8-# Ubuntu releases)
9-BOOTLOADER_DIRS = ['/usr/lib/syslinux', '/usr/lib/syslinux/bios']
10+# Ubuntu releases.)
11+BOOTLOADER_DIRS = ['/usr/lib/syslinux', '/usr/lib/syslinux/modules/bios']
12
13
14 class ARP_HTYPE: