← Back to team overview

ubuntu-server-iso-testing-dev team mailing list archive

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

 

Heber Parrucci has proposed merging lp:~heber013/ubuntu-server-iso-testing/fixing-url-format into lp:ubuntu-server-iso-testing.

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

Requested reviews:
  Joshua Powers (powersj)

For more details, see:
https://code.launchpad.net/~heber013/ubuntu-server-iso-testing/fixing-url-format/+merge/358251

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
-- 
Your team Ubuntu Server Iso Testing Developers is subscribed to branch lp:ubuntu-server-iso-testing.
=== 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 @@
     if options.flavor == 'ubuntu':
         flavor_part = ''
     if options.variant == 'desktop':
-        l_url = os.path.join(options.base_url, release_part, flavor_part, 'daily-live')
+        l_url = os.path.join(options.base_url, flavor_part, release_part, 'daily-live')
     elif options.variant == 'alternate':
-        l_url = os.path.join(options.base_url, release_part, flavor_part, 'daily')
+        l_url = os.path.join(options.base_url, flavor_part, release_part, 'daily')
     elif options.variant == 'dvd':
-        l_url = os.path.join(options.base_url, release_part, flavor_part, 'dvd')
+        l_url = os.path.join(options.base_url, flavor_part, release_part, 'dvd')
     elif options.variant == 'live-server':
         l_url = os.path.join(options.base_url, 'ubuntu-server',release_part,
                              'daily-live')


Follow ups