Merge lp:~heber013/ubuntu-server-iso-testing/fixing-url-format into lp:ubuntu-server-iso-testing

Proposed by Heber Parrucci
Status: Merged
Approved by: Joshua Powers
Approved revision: 273
Merged at revision: 273
Proposed branch: lp:~heber013/ubuntu-server-iso-testing/fixing-url-format
Merge into: lp:ubuntu-server-iso-testing
Diff against target: 18 lines (+3/-3)
1 file modified
download-latest-test-iso.py (+3/-3)
To merge this branch: bzr merge lp:~heber013/ubuntu-server-iso-testing/fixing-url-format
Reviewer Review Type Date Requested Status
Joshua Powers (community) Approve
Review via email: mp+358251@code.launchpad.net

Commit message

Fixing the URL format from {base_url}/{release}/{flavor} to {base_url}/{flavor}/{release}

Description of the change

When you specify a flavor other than ubuntu, for example kubuntu, ubuntu-mate, etc. The URL is not found when checking the sha256 due to an invalid format.
I changed it from:

{base_url}/{release}/{flavor} to {base_url}/{flavor}/{release}/

To test it, you can run for example:

python2 -m download-latest-test-iso -r bionic -v desktop -a amd64 -b pending -i ~/isos -f ubuntu-mate

To post a comment you must log in.
Revision history for this message
Joshua Powers (powersj) wrote :

LGTM appears to only touch desktop and the ubuntu case is covered with a default; so +1 I'll merge shortly

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'download-latest-test-iso.py'
--- download-latest-test-iso.py 2018-11-01 19:27:41 +0000
+++ download-latest-test-iso.py 2018-11-02 19:19:51 +0000
@@ -155,11 +155,11 @@
155 if options.flavor == 'ubuntu':155 if options.flavor == 'ubuntu':
156 flavor_part = ''156 flavor_part = ''
157 if options.variant == 'desktop':157 if options.variant == 'desktop':
158 l_url = os.path.join(options.base_url, release_part, flavor_part, 'daily-live')158 l_url = os.path.join(options.base_url, flavor_part, release_part, 'daily-live')
159 elif options.variant == 'alternate':159 elif options.variant == 'alternate':
160 l_url = os.path.join(options.base_url, release_part, flavor_part, 'daily')160 l_url = os.path.join(options.base_url, flavor_part, release_part, 'daily')
161 elif options.variant == 'dvd':161 elif options.variant == 'dvd':
162 l_url = os.path.join(options.base_url, release_part, flavor_part, 'dvd')162 l_url = os.path.join(options.base_url, flavor_part, release_part, 'dvd')
163 elif options.variant == 'live-server':163 elif options.variant == 'live-server':
164 l_url = os.path.join(options.base_url, 'ubuntu-server',release_part,164 l_url = os.path.join(options.base_url, 'ubuntu-server',release_part,
165 'daily-live')165 'daily-live')

Subscribers

People subscribed via source and target branches