canonical-ubuntu-qa team mailing list archive

[Brian Murray <mp+469241@xxxxxxxxxxxxxxxxxx>]

Brian Murray has proposed merging autopkgtest-cloud:faster-stats 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/469241

The extract_data function already downloads the log files for a test run so we don't a separate call to that function. Additionally, we don't need to call extract_data twice i.e. once before the try / except.
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of autopkgtest-cloud:faster-stats into autopkgtest-cloud:master.

diff --git a/dev-tools/stats.py b/dev-tools/stats.py
index 43aa039..82f6f43 100755
--- a/dev-tools/stats.py
+++ b/dev-tools/stats.py
@@ -219,8 +219,6 @@ def get_stats(db_con, since_days_ago, until_days_ago, limit):
             """
         ):
             r = Run(row)
-            r.download_log()
-            r.extract_data()
             try:
                 r.extract_data()
                 db_con.execute(