[llvm-commits] [zorg] r123588 - in /zorg/trunk/lnt/lnt: db/perfdbsummary.py util/NTEmailReport.py

[Daniel Dunbar]

Author: ddunbar
Date: Sun Jan 16 11:32:49 2011
New Revision: 123588

URL: http://llvm.org/viewvc/llvm-project?rev=123588&view=rev
Log:
LNT/import: Fix PR8972, a crash when importing runs with no 'run_order' info.

Modified:
    zorg/trunk/lnt/lnt/db/perfdbsummary.py
    zorg/trunk/lnt/lnt/util/NTEmailReport.py

Modified: zorg/trunk/lnt/lnt/db/perfdbsummary.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/lnt/lnt/db/perfdbsummary.py?rev=123588&r1=123587&r2=123588&view=diff
==============================================================================
--- zorg/trunk/lnt/lnt/db/perfdbsummary.py (original)
+++ zorg/trunk/lnt/lnt/db/perfdbsummary.py Sun Jan 16 11:32:49 2011
@@ -192,6 +192,9 @@
         return (not db.modified_run and
                 self.revision == db.get_revision_number("RunInfo"))
 
+    def contains_run(self, run_id):
+        return run_id in self.machine_id_by_run
+
     def get_run_order(self, run_id):
         return self.order_by_run.get(run_id)
 

Modified: zorg/trunk/lnt/lnt/util/NTEmailReport.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/lnt/lnt/util/NTEmailReport.py?rev=123588&r1=123587&r2=123588&view=diff
==============================================================================
--- zorg/trunk/lnt/lnt/util/NTEmailReport.py (original)
+++ zorg/trunk/lnt/lnt/util/NTEmailReport.py Sun Jan 16 11:32:49 2011
@@ -98,6 +98,10 @@
     # Get the run summary.
     run_summary = perfdbsummary.SimpleSuiteRunSummary.get_summary(db, tag)
 
+    # Ignore run's which don't appear in the summary, for whatever reason.
+    if not run_summary.contains_run(run.id):
+        return ("No report for run", "No report for run", None)
+
     # Load the test suite summary.
     ts_summary = perfdbsummary.get_simple_suite_summary(db, tag)