canonical-ubuntu-qa team mailing list archive

[Brian Murray <mp+468816@xxxxxxxxxxxxxxxxxx>]

Brian Murray has proposed merging autopkgtest-cloud:pass-on-empty into autopkgtest-cloud:master.

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

For more details, see:
https://code.launchpad.net/~ubuntu-release/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/468816

When I was searching the log files in early May I encountered a situation where lines was empty. Unfortunately, I lost the example log file but I suspect it was just full of errors. Regardless, if lines is empty we shouldn't try searching it for a regex.

-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of autopkgtest-cloud:pass-on-empty into autopkgtest-cloud:master.

diff --git a/dev-tools/stats.py b/dev-tools/stats.py
index 1f251c7..743e7a6 100755
--- a/dev-tools/stats.py
+++ b/dev-tools/stats.py
@@ -109,6 +109,8 @@ class Run:
                 if "command line: /home/ubuntu/autopkgtest/runner/autopkgtest"
                 in l
             ]
+            if not lines:
+                return
             m = self.host_pattern.search(lines[0])
             if m:
                 self.host = m.group(1)