[LNT] r317388 - Log when we remove previous runs because we got a duplicate submission

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 3 16:35:08 PDT 2017


Author: matze
Date: Fri Nov  3 16:35:08 2017
New Revision: 317388

URL: http://llvm.org/viewvc/llvm-project?rev=317388&view=rev
Log:
Log when we remove previous runs because we got a duplicate submission

Modified:
    lnt/trunk/lnt/server/db/testsuitedb.py

Modified: lnt/trunk/lnt/server/db/testsuitedb.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/db/testsuitedb.py?rev=317388&r1=317387&r2=317388&view=diff
==============================================================================
--- lnt/trunk/lnt/server/db/testsuitedb.py (original)
+++ lnt/trunk/lnt/server/db/testsuitedb.py Fri Nov  3 16:35:08 2017
@@ -16,6 +16,7 @@ from sqlalchemy import *
 from sqlalchemy.orm import relation
 from sqlalchemy.orm.exc import ObjectDeletedError
 from typing import List
+from lnt.util import logger
 
 import testsuite
 import lnt.testing.profile.profile as profile
@@ -970,8 +971,9 @@ class TestSuiteDB(object):
                     raise ValueError("Duplicate submission for '%s'" %
                                      order.name)
                 elif merge == 'replace':
-                    for run in existing_runs:
-                        session.delete(run)
+                    for previous_run in existing_runs:
+                        logger.info("Duplicate submission for order %r: deleting previous run %r" % (order, previous_run))
+                        session.delete(previous_run)
                 else:
                     raise ValueError('Invalid Run mergeStrategy %r' % merge)
 




More information about the llvm-commits mailing list