canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+462584@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:i386-no-retry-on-build-dep-failure into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)
Related bugs:
  Bug #2058062 in Auto Package Testing: "amd64 workers shouldn't abort after i386 test bed failures"
  https://bugs.launchpad.net/auto-package-testing/+bug/2058062

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/462584
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:i386-no-retry-on-build-dep-failure into autopkgtest-cloud:master.

diff --git a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
index f1bbac5..82a6a14 100755
--- a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
+++ b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/worker/worker
@@ -114,6 +114,10 @@ TEMPORARY_TEST_FAIL_STRINGS = [
     "OSError: [Errno 28] No space left on device",
 ]
 
+I386_NO_RETRY_STRINGS = [
+    "Can't resolve build dependencies on testbed"
+]
+
 # If we repeatedly time out when installing, there's probably a problem with
 # one of the packages' maintainer scripts.
 FAIL_STRINGS_REGEX = [
@@ -1134,6 +1138,10 @@ def request(msg):
                         )
                         if s in contents
                     ]
+                    if release.lower() == "i386":
+                        for I386_FAIL_STRING in I386_NO_RETRY_STRINGS:
+                            if I386_FAIL_STRING in contents:
+                                temp_fails = False
                     if temp_fails:
                         logging.warning(
                             "Saw %s in log, which is a sign of a temporary failure.",