Merge autopkgtest-cloud:download-results-notes into autopkgtest-cloud:master

Proposed by Brian Murray
Status: Merged
Merged at revision: 6820f8b804522a28caa951213239eba3abeed53d
Proposed branch: autopkgtest-cloud:download-results-notes
Merge into: autopkgtest-cloud:master
Diff against target: 14 lines (+2/-1)
1 file modified
charms/focal/autopkgtest-web/webcontrol/download-results (+2/-1)
Reviewer Review Type Date Requested Status
Tim Andersson Approve
Review via email: mp+452475@code.launchpad.net

Description of the change

I thought the comment would be more helpful if we were to explicitly state what a container for PPA results looks like.

To post a comment you must log in.
Revision history for this message
Tim Andersson (andersson123) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/charms/focal/autopkgtest-web/webcontrol/download-results b/charms/focal/autopkgtest-web/webcontrol/download-results
2index 0f14155..d97b98a 100755
3--- a/charms/focal/autopkgtest-web/webcontrol/download-results
4+++ b/charms/focal/autopkgtest-web/webcontrol/download-results
5@@ -58,7 +58,8 @@ def process_message(msg, db_con):
6 version = info["testpkg_version"]
7 (_, _, _, _, run_id) = info["swift_dir"].split("/")
8
9- # we don't handle PPA requests
10+ # we don't handle PPA requests and their container names are
11+ # formatted like autopkgtest-$release-$user-$ppaname
12 if container != ("autopkgtest-{}".format(release)):
13 logging.debug("Ignoring non-distro request: {}".format(info))
14 msg.channel.basic_ack(msg.delivery_tag)

Subscribers

People subscribed via source and target branches