canonical-ubuntu-qa team mailing list archive

[Skia <mp+457903@xxxxxxxxxxxxxxxxxx>]

Skia has proposed merging ~hyask/autopkgtest-cloud:skia/fix_dl-a-r into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~hyask/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/457903
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~hyask/autopkgtest-cloud:skia/fix_dl-a-r into autopkgtest-cloud:master.

diff --git a/.gitignore b/.gitignore
index 6a76737..0682011 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,3 @@
 docs/_build
 *.charm
+__pycache__
diff --git a/charms/focal/autopkgtest-web/webcontrol/download-all-results b/charms/focal/autopkgtest-web/webcontrol/download-all-results
index 53a3983..412771a 100755
--- a/charms/focal/autopkgtest-web/webcontrol/download-all-results
+++ b/charms/focal/autopkgtest-web/webcontrol/download-all-results
@@ -208,7 +208,7 @@ def fetch_container(release, container_url):
                 os.path.join(container_url, known_results[run_id])
             )
     except urllib.error.HTTPError as e:
-        if e.code == 401:
+        if e.code == 401 or e.code == 404:
             LOGGER.warning(f"Couldn't access {container_url} - doesn't exist?")
             return
         raise