Merge lp:~laney/ubuntu-archive-tools/retry-autopkgtest-regressions-bileto-v2 into lp:ubuntu-archive-tools

Proposed by Iain Lane on 2016-12-06
Status: Merged
Merged at revision: 1069
Proposed branch: lp:~laney/ubuntu-archive-tools/retry-autopkgtest-regressions-bileto-v2
Merge into: lp:ubuntu-archive-tools
Diff against target: 18 lines (+3/-3)
1 file modified
retry-autopkgtest-regressions (+3/-3)
To merge this branch: bzr merge lp:~laney/ubuntu-archive-tools/retry-autopkgtest-regressions-bileto-v2
Reviewer Review Type Date Requested Status
Sebastien Bacher 2016-12-06 Pending
Ubuntu Package Archive Administrators 2016-12-06 Pending
Review via email: mp+312537@code.launchpad.net

Description of the Change

bileto's v1 api has gone away

To post a comment you must log in.
Robert Bruce Park (robru) wrote :

Api for v2 is correct. Apologies for not fixing this myself when i migrated everything else off of v1.

Please file a bug against lp:bileto to list this project among the known bileto json api consumers so i remember to maintain it during future API transitions.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'retry-autopkgtest-regressions'
2--- retry-autopkgtest-regressions 2016-11-14 20:45:38 +0000
3+++ retry-autopkgtest-regressions 2016-12-06 09:25:20 +0000
4@@ -101,11 +101,11 @@
5
6 if args.bileto:
7 url_root = 'https://bileto.ubuntu.com'
8- ticket_url = url_root + '/v1/ticket/%s' % args.bileto
9+ ticket_url = url_root + '/v2/ticket/%s' % args.bileto
10 excuses_url = None
11 with urllib.request.urlopen(ticket_url) as f:
12- ticket = json.loads(f.read().decode('utf-8'))['requests'][0]
13- ppa_name = ticket.get('siloname', '').split('/')[-1]
14+ ticket = json.loads(f.read().decode('utf-8'))['tickets'][0]
15+ ppa_name = ticket.get('ppa', '')
16 for line in ticket.get('autopkgtest', '').splitlines():
17 if args.series in line:
18 excuses_url = line

Subscribers

People subscribed via source and target branches