Merge lp:~laney/ubuntu-archive-scripts/generate-team-p-m-xz-me-harder into lp:ubuntu-archive-scripts

Proposed by Iain Lane
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 283
Merged at revision: 285
Proposed branch: lp:~laney/ubuntu-archive-scripts/generate-team-p-m-xz-me-harder
Merge into: lp:ubuntu-archive-scripts
Diff against target: 28 lines (+5/-2)
2 files modified
generate-team-p-m (+4/-1)
run-proposed-migration (+1/-1)
To merge this branch: bzr merge lp:~laney/ubuntu-archive-scripts/generate-team-p-m-xz-me-harder
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+387371@code.launchpad.net

Description of the change

I've cowboyed this on snakefruit, will uncowboy once merged!

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for fixing that one!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'generate-team-p-m'
2--- generate-team-p-m 2020-07-01 08:19:40 +0000
3+++ generate-team-p-m 2020-07-14 14:22:54 +0000
4@@ -240,7 +240,10 @@
5
6 print("fetching yaml")
7 if args.excuses_yaml:
8- yaml_text = open(args.excuses_yaml)
9+ if args.excuses_yaml.endswith('.xz'):
10+ yaml_text = lzma.open(args.excuses_yaml)
11+ else:
12+ yaml_text = open(args.excuses_yaml)
13 else:
14 try:
15 yaml_text = lzma.open(urlopen("https://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.yaml.xz"))
16
17=== modified file 'run-proposed-migration'
18--- run-proposed-migration 2020-04-24 13:12:18 +0000
19+++ run-proposed-migration 2020-07-14 14:22:54 +0000
20@@ -54,7 +54,7 @@
21 code/b1/britney $actions >"$logfile" 2>&1
22 if [ "$SERIES" = "$DEFAULT_SERIES" ]; then
23 cd ~/public_html/proposed-migration/
24- generate-team-p-m --excuses-yaml $SERIES/update_excuses.yaml \
25+ generate-team-p-m --excuses-yaml $SERIES/update_excuses.yaml.xz \
26 --subscribers-json ~/public_html/package-team-mapping.json \
27 $SERIES/update_excuses_by_team.html $SERIES/update_excuses_by_team.yaml
28 tmpdir="$(mktemp -d)"

Subscribers

People subscribed via source and target branches