Merge ~sespiros/ubuntu-qa-tools/+git/ubuntu-qa-tools:fix-trusty into ubuntu-qa-tools:master

Proposed by Spyros Seimenis
Status: Merged
Merged at revision: c6ff8402f7e47bb9e16125722720877bb2f80cdb
Proposed branch: ~sespiros/ubuntu-qa-tools/+git/ubuntu-qa-tools:fix-trusty
Merge into: ubuntu-qa-tools:master
Diff against target: 13 lines (+1/-1)
1 file modified
vm-tools/uvt (+1/-1)
Reviewer Review Type Date Requested Status
Ubuntu Bug Control Pending
Review via email: mp+411863@code.launchpad.net

Description of the change

Fixes the following case:

uvt new trusty amd64 test

This commit is also part of https://code.launchpad.net/~sespiros/ubuntu-qa-tools/+git/ubuntu-qa-tools/+merge/409517 which was partially merged (the xenial case fix).

To post a comment you must log in.
Revision history for this message
Paulo Flabiano Smorigo (pfsmorigo) wrote :

LGTM

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/vm-tools/uvt b/vm-tools/uvt
index 6ca51e7..4a3f4a9 100755
--- a/vm-tools/uvt
+++ b/vm-tools/uvt
@@ -2915,7 +2915,7 @@ def find_latest_release(release_num, iso_type):
2915 matches = []2915 matches = []
2916 for line in text:2916 for line in text:
2917 if line.startswith(' ' + release_num):2917 if line.startswith(' ' + release_num):
2918 matches.append(line[0:].split('/')[0][1:])2918 matches.append(re.search(r'\d+\.\d+(\.\d+)? ', line).group().strip())
29192919
2920 if len(matches) > 0:2920 if len(matches) > 0:
2921 latest_release = sorted(matches)[-1]2921 latest_release = sorted(matches)[-1]

Subscribers

People subscribed via source and target branches