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

[Libreoffice Gerrit user]

 qa/bugzillaChecker.py |   30 ++++++++++++++++++++++++------
 1 file changed, 24 insertions(+), 6 deletions(-)

New commits:
commit 37823b54dd0b1adc95268952e2f815c46b3346bd
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Wed Nov 28 17:56:37 2018 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Wed Nov 28 17:56:37 2018 +0100

    QA: check inactive unconfirmed bugs which last comment is from the reporter

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 55f2dfd..55e8d49 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -29,15 +29,23 @@ memberBugs = 50
 oldUserPeriodDays = 180
 oldUserBugs = 20
 
+# bugs which last comment is from the Commit Notification
 pingFixedBugPeriodDays = 30
 
+# Unconfirmed bugs which last comment is not written by the reporter
+# or not comments at all
 retestUnconfirmedPeriodDays = 30
 
+# Unconfirmed bugs which last comment is written by the reporter
+inactiveUnconfirmedPeriodDays = 180
+
+# Needinfo bugs which last comment is written by the reporter
 retestNeedinfoPeriodDays = 60
 
+# Assigned bugs without activity
 inactiveAssignedPeriodDays = 90
 
-#tuple of versions to check whether the version has been changed at confirmation time
+# tuple of versions to check whether the version has been changed at confirmation time
 versionsToCheck = ('5', '6')
 
 crashReportDomain = "https://crashreport.libreoffice.org/stats/signature/"
@@ -295,13 +303,18 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
             if bSameAuthor and rowStatus == 'UNCONFIRMED' and \
                     datetime.datetime.strptime(row['last_change_time'], "%Y-%m-%dT%H:%M:%SZ") < cfg['retestUnconfirmedPeriod']:
                 value = [ rowId, row['last_change_time'], creatorMail ]
-                util_add_to_result(lResults, 'unconfirmed_not_triaged', value)
+                util_add_to_result(lResults, 'unconfirmed_without_comments', value)
 
             if len(comments) > 0:
-                if rowStatus == 'UNCONFIRMED' and comments[-1]['creator'] != creatorMail and \
+                if rowStatus == 'UNCONFIRMED':
+                    if comments[-1]['creator'] != creatorMail and \
                         datetime.datetime.strptime(row['last_change_time'], "%Y-%m-%dT%H:%M:%SZ") < cfg['retestUnconfirmedPeriod']:
-                    value = [ rowId, row['last_change_time'], comments[-1]['creator'] ]
-                    util_add_to_result(lResults, 'untouched_unconfirmed', value)
+                        value = [ rowId, row['last_change_time'], comments[-1]['creator'] ]
+                        util_add_to_result(lResults, 'unconfirmed_last_comment_not_from_reporter', value)
+                    elif comments[-1]['creator'] == creatorMail and \
+                        datetime.datetime.strptime(row['last_change_time'], "%Y-%m-%dT%H:%M:%SZ") < cfg['inactiveUnconfirmedPeriod']:
+                        value = [ rowId, row['last_change_time'], comments[-1]['creator'] ]
+                        util_add_to_result(lResults, 'unconfirmed_last_comment_from_reporter', value)
 
                 elif rowStatus == 'NEEDINFO' and comments[-1]['creator'] == creatorMail and \
                         datetime.datetime.strptime(row['last_change_time'], "%Y-%m-%dT%H:%M:%SZ") >= cfg['retestNeedinfoPeriod']:
@@ -402,9 +415,12 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
                     if dKey == 'inactive_assignee':
                         if dValue[idx][1] >= cfg['coloredInactiveAssignedPeriod']:
                             background = Back.GREEN
-                    elif dKey == 'untouched_unconfirmed' or dKey == 'unconfirmed_not_triaged':
+                    elif dKey == 'unconfirmed_last_comment_not_from_reporter' or dKey == 'unconfirmed_without_comments':
                         if dValue[idx][1] >= cfg['coloredRetestUnconfirmedPeriod']:
                             background = Back.GREEN
+                    elif dKey == 'unconfirmed_last_comment_from_reporter':
+                        if dValue[idx][1] >= cfg['coloredInactiveUnconfirmedPeriod']:
+                            background = Back.GREEN
                     elif dKey == 'ping_bug_fixed':
                         if dValue[idx][1] >= cfg['coloredFixBugPingPeriod']:
                             background = Back.GREEN
@@ -460,6 +476,8 @@ def runCfg():
     cfg['pingFixedBugDiff'] = common.util_convert_days_to_datetime(cfg, pingFixedBugPeriodDays + reportPeriodDays)
     cfg['coloredFixBugPingPeriod'] = common.util_convert_days_to_datetime(cfg, coloredPeriodDays + pingFixedBugPeriodDays)
     cfg['retestUnconfirmedPeriod'] = common.util_convert_days_to_datetime(cfg, retestUnconfirmedPeriodDays)
+    cfg['coloredInactiveUnconfirmedPeriod'] = common.util_convert_days_to_datetime(cfg, coloredPeriodDays + inactiveUnconfirmedPeriodDays)
+    cfg['inactiveUnconfirmedPeriod'] = common.util_convert_days_to_datetime(cfg, inactiveUnconfirmedPeriodDays)
     cfg['coloredRetestUnconfirmedPeriod'] = common.util_convert_days_to_datetime(cfg, coloredPeriodDays + retestUnconfirmedPeriodDays)
     cfg['retestNeedinfoPeriod'] = common.util_convert_days_to_datetime(cfg, retestNeedinfoPeriodDays)
     cfg['inactiveAssignedPeriod'] = common.util_convert_days_to_datetime(cfg, inactiveAssignedPeriodDays)