Merge lp:~sil2100/ubuntu-cdimage/uc-rebuild-requests into lp:ubuntu-cdimage

Proposed by Łukasz Zemczak
Status: Merged
Merged at revision: 2059
Proposed branch: lp:~sil2100/ubuntu-cdimage/uc-rebuild-requests
Merge into: lp:ubuntu-cdimage
Diff against target: 25 lines (+11/-2)
1 file modified
bin/rebuild-requests (+11/-2)
To merge this branch: bzr merge lp:~sil2100/ubuntu-cdimage/uc-rebuild-requests
Reviewer Review Type Date Requested Status
Steve Langasek Needs Fixing
Review via email: mp+424207@code.launchpad.net

Commit message

rebuild-requests: Add support for Ubuntu Core image rebuilds

Description of the change

rebuild-requests: Add support for Ubuntu Core image rebuilds

Tested this in a hacky way and it seems to be doing what it should. All because ubuntu-core is a very special snowflake.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

Kinda ugly but makes sense.

review: Needs Fixing
2060. By Łukasz Zemczak

Use entry[2] as proposed by vorlon

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Thanks for the review! I changed the env['CHANNEL'] entry and will now merge it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/rebuild-requests'
--- bin/rebuild-requests 2022-05-04 12:23:22 +0000
+++ bin/rebuild-requests 2022-06-09 08:31:20 +0000
@@ -188,10 +188,19 @@
188 env['ARCHES'] = arches188 env['ARCHES'] = arches
189 env['DIST'] = entries[0][0].series_title.lower()189 env['DIST'] = entries[0][0].series_title.lower()
190190
191 # The ubuntu-base image is actually hacked up in overall, so even
192 # though it's defined as "daily", it's a "daily-live" type
193 image_type = entry[2]191 image_type = entry[2]
194 if project == "ubuntu-base":192 if project == "ubuntu-base":
193 # The ubuntu-base image is actually hacked up in overall, so
194 # even though it's defined as "daily", it's a "daily-live" type
195 image_type = "daily-live"
196 elif project in ("ubuntu-core", "ubuntu-appliance"):
197 # The ubuntu-core images however are also a bit special, as
198 # they have a different publish structure, CHANNEL arguments
199 # and more
200 env['EXTRA_PPAS'] = "snappy-dev/image"
201 # Due to the different structure, IMAGE_TYPE is the channel
202 # here
203 env['CHANNEL'] = entry[2]
195 image_type = "daily-live"204 image_type = "daily-live"
196205
197 cmd = ["for-project", project, "cron.%s" % image_type]206 cmd = ["for-project", project, "cron.%s" % image_type]

Subscribers

People subscribed via source and target branches