[Libreoffice-commits] dev-tools.git: 3 commits - qa/bugzillaChecker.py

[Xisco Fauli]

 qa/bugzillaChecker.py |   12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

New commits:
commit a47a35415ccc5bf1bffe11df9cb4a01a8e859be0
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed Jun 6 12:07:35 2018 +0200

    QA: this info is not really useful. Remove it for now

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 2625524..9a40a96 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -243,10 +243,6 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
                                         whiteboard in row['whiteboard'] and common.isOpen(rowStatus):
                                     util_add_to_result(lResults, 'backport_added', resultValue)
 
-                            for whiteboard in change['removed'].split(' '):
-                                if 'target:' in whiteboard.lower() and whiteboard.split(":")[1] not in row["whiteboard"]:
-                                    util_add_to_result(lResults, 'target_removed', resultValue)
-
                     elif change['field_name'] == 'cf_crashreport':
                         crashSignature = row['cf_crashreport']
                         if crashSignature and not crashSignature.startswith('["'):
commit 963177c50714756ef92c362cc95359bce4ebcfde
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed Jun 6 12:05:18 2018 +0200

    QA: don't use this variable

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index bab6c4c..2625524 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -26,7 +26,6 @@ memberPeriodDays = 365
 memberBugs = 50
 
 oldUserPeriodDays = 180
-oldUserPeriod2Days = 365
 oldUserBugs = 20
 
 fixBugPingPeriodDays = 30
@@ -414,7 +413,7 @@ def runCfg():
     cfg['coloredReportPeriod'] = common.util_convert_days_to_datetime(cfg, coloredPeriodDays)
     cfg['newUserPeriod'] = common.util_convert_days_to_datetime(cfg, newUserPeriodDays)
     cfg['oldUserPeriod'] = common.util_convert_days_to_datetime(cfg, oldUserPeriodDays)
-    cfg['oldUserPeriod2'] = common.util_convert_days_to_datetime(cfg, oldUserPeriod2Days)
+    cfg['oldUserPeriod2'] = common.util_convert_days_to_datetime(cfg, oldUserPeriodDays + reportPeriodDays)
     cfg['memberPeriod'] = common.util_convert_days_to_datetime(cfg, memberPeriodDays)
     cfg['fixBugPingPeriod'] = common.util_convert_days_to_datetime(cfg, fixBugPingPeriodDays)
     cfg['fixBugPingDiff'] = common.util_convert_days_to_datetime(cfg, fixBugPingPeriodDays + reportPeriodDays)
commit b5c768651cd13ce795113aeca6b45be87aa8a628
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Wed Jun 6 11:13:28 2018 +0200

    QA: remove broken code

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 5f025b7..bab6c4c 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -37,8 +37,6 @@ retestNeedinfoPeriodDays = 60
 
 inactiveAssignedPeriodDays = 90
 
-reopened6MonthsComment = "This bug has been in RESOLVED FIXED status for more than 6 months."
-
 #tuple of versions to check whether the version has been changed at confirmation time
 versionsToCheck = ('5', '6')
 
@@ -276,9 +274,6 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
 
                 common.util_check_bugzilla_mail(statList, commentMail, '', commentDate, rowId)
 
-                if common.isOpen(rowStatus) and reopened6MonthsComment in comment['text']:
-                    util_add_to_result(lResults, 'reopened_6_months', [rowId, '', ''])
-
             if len(comments) > 0:
                 if rowStatus == 'UNCONFIRMED' and comments[-1]['creator'] != creatorMail and \
                         datetime.datetime.strptime(row['last_change_time'], "%Y-%m-%dT%H:%M:%SZ") < cfg['retestUnconfirmedPeriod']: