Tools used by the Ubuntu QA Team

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

  1. Git
  2. lp:~sespiros/ubuntu-qa-tools/+git/ubuntu-qa-tools
  3. fix-trusty
  4. Merge into 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
1diff --git a/vm-tools/uvt b/vm-tools/uvt
2index 6ca51e7..4a3f4a9 100755
3--- a/vm-tools/uvt
4+++ b/vm-tools/uvt
5@@ -2915,7 +2915,7 @@ def find_latest_release(release_num, iso_type):
6 matches = []
7 for line in text:
8 if line.startswith(' ' + release_num):
9- matches.append(line[0:].split('/')[0][1:])
10+ matches.append(re.search(r'\d+\.\d+(\.\d+)? ', line).group().strip())
11
12 if len(matches) > 0:
13 latest_release = sorted(matches)[-1]

Subscribers

People subscribed via source and target branches