Merge lp:~ltrager/maas-images/xenial_ppc_bootloader into lp:maas-images

Proposed by Lee Trager
Status: Merged
Merged at revision: 406
Proposed branch: lp:~ltrager/maas-images/xenial_ppc_bootloader
Merge into: lp:maas-images
Diff against target: 43 lines (+12/-10)
2 files modified
conf/bootloaders.yaml (+1/-1)
meph2/commands/mimport.py (+11/-9)
To merge this branch: bzr merge lp:~ltrager/maas-images/xenial_ppc_bootloader
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+352678@code.launchpad.net

Commit message

Revert Open Firmware PPC64EL bootloader to Xenial.

The Bionic bootloader for Open Firmware PPC64EL is requesting a GRUB
configuration file for the wrong MAC address(LP:1785859). Until this
bug is fixed revert to the Xenial bootloader.

To allow the bootloader to be reverted lp:maas-images will now only look to see if the bootloader discovered in the archive differs from the latest
bootloader in the stream. Older bootloaders in the stream are ignored. This allows the release for each bootloader to be reverted to an older version or upgraded to a newer version without changing anything other than bootloaders.yaml.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'conf/bootloaders.yaml'
--- conf/bootloaders.yaml 2018-07-19 03:21:42 +0000
+++ conf/bootloaders.yaml 2018-08-07 21:41:25 +0000
@@ -54,7 +54,7 @@
54 arch: ppc64el54 arch: ppc64el
55 arches: ppc64el,ppc6455 arches: ppc64el,ppc64
56 archive: http://ports.ubuntu.com56 archive: http://ports.ubuntu.com
57 release: bionic57 release: xenial
58 os: grub-ieee127558 os: grub-ieee1275
59 files:59 files:
60 - usr/lib/grub/powerpc-ieee1275/60 - usr/lib/grub/powerpc-ieee1275/
6161
=== modified file 'meph2/commands/mimport.py'
--- meph2/commands/mimport.py 2018-04-18 21:25:35 +0000
+++ meph2/commands/mimport.py 2018-08-07 21:41:25 +0000
@@ -112,15 +112,17 @@
112 # the packages in the archive112 # the packages in the archive
113 if product_id in product_tree['products']:113 if product_id in product_tree['products']:
114 versions = product_tree['products'][product_id]['versions']114 versions = product_tree['products'][product_id]['versions']
115 for data in versions.values():115 # Only check if the latest version in the stream matches the
116 for item in data['items'].values():116 # latest version from the archive. This allows bootloaders to
117 src_package = src_packages.get(item['src_package'])117 # be reverted to previous versions.
118 if (118 data = versions[max(versions.keys())]
119 src_package is not None and119 for item in data['items'].values():
120 src_package['src_version'] == item['src_version']120 src_package = src_packages.get(item['src_package'])
121 and121 if (
122 src_package['src_release'] == item['src_release']):122 src_package is not None and
123 src_packages[item['src_package']]['found'] = True123 src_package['src_version'] == item['src_version'] and
124 src_package['src_release'] == item['src_release']):
125 src_packages[item['src_package']]['found'] = True
124 bootloader_uptodate = True126 bootloader_uptodate = True
125 for src_package in src_packages.values():127 for src_package in src_packages.values():
126 if not src_package['found']:128 if not src_package['found']:

Subscribers

People subscribed via source and target branches