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

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Tue Mar 19 12:04:01 UTC 2019


 qa/createBlogReport.py |  446 ++++++++++++++++++++++++-------------------------
 1 file changed, 226 insertions(+), 220 deletions(-)

New commits:
commit 15a03618d7335b82510e1c89cdf220dff85556a6
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 12:50:18 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: Add enhancement info to the blog report

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index 2ce9879..64d2e1d 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -118,6 +118,11 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                 creationDay = str(creationDate.strftime("%Y-%m-%d"))
                 util_increase_action(statList['created'], rowId, creatorMail, creationDay)
 
+                if row['severity'] == 'enhancement':
+                    if 'enhancement' not in statList['created']:
+                        statList['created']['enhancement'] = 0
+                    statList['created']['enhancement'] += 1
+
             common.util_check_bugzilla_mail(
                     statList, creatorMail, row['creator_detail']['real_name'], creationDate, rowId)
 
@@ -496,9 +501,14 @@ def createPlot(valueDict, plotType, plotTitle, plotLabel, plotColor):
 
 def createSection(fp, value, sectionName, action, actionPerson, plotColor):
     print(makeH2(sectionName), file=fp)
-    print("{} bugs have been {} by {} people.".format(
-        makeStrong(len(value["id"])), action,
-        makeStrong(len(value["author"]))), file=fp)
+    if 'enhancement' in value:
+        print("{} bugs, {} of which are enhancements, have been {} by {} people.".format(
+            makeStrong(len(value["id"])), makeStrong(value['enhancement']), action,
+            makeStrong(len(value["author"]))), file=fp)
+    else:
+        print("{} bugs have been {} by {} people.".format(
+            makeStrong(len(value["id"])), action,
+            makeStrong(len(value["author"]))), file=fp)
 
     print(file=fp)
     print(makeStrong("Top 10 " + actionPerson), file=fp)
@@ -557,7 +567,7 @@ def createReport(statList):
     createList(fp, statList['crashFixed'], "List of crashes fixed")
     createList(fp, statList['oldBugsFixed'], "List of old bugs ( more than {} years old ) fixed".format(oldBugsYears))
     createSection(fp, statList['verified'], "Verified bug fixes", "verified", "Verifiers", "palegreen")
-    createSection(fp, statList['wfm'], "WORKSFORME bugs", "retested", "testers", "palegreen")
+    createSection(fp, statList['wfm'], "WORKSFORME bugs", "retested", "testers", "m")
     createSection(fp, statList['metabug'], "Categorized Bugs", "categorized with a metabug", "Categorizers", "lightpink")
     createSection(fp, statList['keywords']['regression'], "Regression Bugs", "set as regressions", "", "mediumpurple")
     createSection(fp, statList['keywords']['bisected'], "Bisected Bugs", "bisected", "Bisecters", "orange")
commit ab9163e29ea3535ae0c50b45567ebcb233e3cb73
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 12:20:58 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: Add WORKSFORME info to the report

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index 81ed552..2ce9879 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -32,6 +32,7 @@ def util_create_statList():
         'created': util_create_basic_schema(),
         'confirmed': util_create_basic_schema(),
         'verified': util_create_basic_schema(),
+        'wfm': util_create_basic_schema(),
         'fixed': util_create_basic_schema(),
         'criticalFixed': {},
         'crashFixed': {},
@@ -121,12 +122,15 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                     statList, creatorMail, row['creator_detail']['real_name'], creationDate, rowId)
 
             isFixed = False
+            isWFM = False
             isConfirmed = False
             isVerified = False
             dayConfirmed = None
             dayVerified = None
+            dayWFM = None
             authorConfirmed = None
             authorVerified = None
+            authorWFM = None
             isRegression = False
             isRegressionClosed = False
             isBibisectRequest = False
@@ -290,11 +294,19 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                             if addedResolution == 'FIXED':
                                 fixedBugs[rowId] = actionDate
                                 isFixed = True
-
                             elif removedResolution == 'FIXED' and isFixed:
                                 del fixedBugs[rowId]
                                 isFixed = False
 
+                            if addedResolution == 'WORKSFORME':
+                                isWFM = True
+                                dayWFM = actionDay
+                                authorWFM = actionMail
+                                util_increase_action(statList['wfm'], rowId, actionMail, actionDay, diffTime)
+                            elif removedResolution == 'WORKSFORME' and isWFM:
+                                util_decrease_action(statList['wfm'], authorWFM, dayWFM)
+                                isWFM = False
+
                     elif change['field_name'] == 'keywords':
                         keywordsAdded = change['added'].lower().split(", ")
                         keywordsRemoved = change['removed'].lower().split(", ")
@@ -545,6 +557,7 @@ def createReport(statList):
     createList(fp, statList['crashFixed'], "List of crashes fixed")
     createList(fp, statList['oldBugsFixed'], "List of old bugs ( more than {} years old ) fixed".format(oldBugsYears))
     createSection(fp, statList['verified'], "Verified bug fixes", "verified", "Verifiers", "palegreen")
+    createSection(fp, statList['wfm'], "WORKSFORME bugs", "retested", "testers", "palegreen")
     createSection(fp, statList['metabug'], "Categorized Bugs", "categorized with a metabug", "Categorizers", "lightpink")
     createSection(fp, statList['keywords']['regression'], "Regression Bugs", "set as regressions", "", "mediumpurple")
     createSection(fp, statList['keywords']['bisected'], "Bisected Bugs", "bisected", "Bisecters", "orange")
commit 13f170a7e8c86381e45a577542521b608bd7a433
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 11:23:55 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: just care about comments in the period of time

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index 76e0661..81ed552 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -351,7 +351,7 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                 common.util_check_bugzilla_mail(
                         statList, commentMail, '', commentDate, rowId)
 
-                if rowId in fixedBugs:
+                if check_date(commentDate, cfg) and rowId in fixedBugs:
                     if commentMail == "libreoffice-commits at lists.freedesktop.org":
                         commentText = comment['text']
                         author =  commentText.split(' committed a patch related')[0]
commit b8533eb38cadacf8b511bd2a400229991aa1d614
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 11:05:06 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: Fix indent

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index 166da74..76e0661 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -148,202 +148,202 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                 diffTime = (actionDate - creationDate).days
 
                 for change in action['changes']:
-                        if change['field_name'] == 'priority':
-                            addedPriority = change['added']
-                            removedPriority = change['removed']
-
-                            # Sometimes the priority is increased to highest after the bug is fixed
-                            # Ignore those cases
-                            if not isThisBugClosed and not isHighestClosed:
-                                if not isHighest and addedPriority == "highest":
-                                    if actionDay not in highestCountPerDay:
-                                        highestCountPerDay[actionDay] = 0
-                                    highestCountPerDay[actionDay] += 1
-                                    isHighest = True
-
-                                if isHighest and removedPriority == "highest":
-                                    if actionDay not in highestCountPerDay:
-                                        highestCountPerDay[actionDay] = 0
-                                    highestCountPerDay[actionDay] -= 1
-                                    isHighest = False
-
-                            # TODO: IsThisBugClosed should be check here, but the result is not accurate
-                            if not isHighClosed:
-                                if not isHigh and addedPriority == "high":
-                                    if actionDay not in highCountPerDay:
-                                        highCountPerDay[actionDay] = 0
-                                    highCountPerDay[actionDay] += 1
-                                    isHigh = True
-
-                                if isHigh and removedPriority == "high":
-                                    if actionDay not in highCountPerDay:
-                                        highCountPerDay[actionDay] = 0
-                                    highCountPerDay[actionDay] -= 1
-                                    isHigh = False
-
-                        if change['field_name'] == 'status':
-                            addedStatus = change['added']
-                            removedStatus = change['removed']
-
-                            if common.isOpen(addedStatus):
-                                isThisBugClosed = False
-                            else:
-                                isThisBugClosed = True
-
-                            #See above
-                            if rowId >= 89589:
-                                if removedStatus == "UNCONFIRMED":
-                                    if actionDay not in unconfirmedCountPerDay:
-                                        unconfirmedCountPerDay[actionDay] = 0
-                                    unconfirmedCountPerDay[actionDay] -= 1
-
-                                elif addedStatus == 'UNCONFIRMED':
-                                    if actionDay not in unconfirmedCountPerDay:
-                                        unconfirmedCountPerDay[actionDay] = 0
-                                    unconfirmedCountPerDay[actionDay] += 1
-
-                            if isRegression:
-                                # the regression is being reopened
-                                if isRegressionClosed and not isThisBugClosed:
-                                    if actionDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[actionDay] = 0
-                                    regressionsCountPerDay[actionDay] += 1
-                                    isRegressionClosed = False
-
-                                # the regression is being closed
-                                if not isRegressionClosed and isThisBugClosed:
-                                    if actionDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[actionDay] = 0
-                                    regressionsCountPerDay[actionDay] -= 1
-                                    isRegressionClosed = True
-
-                            if isBibisectRequest:
-                                # the bibisectRequest is being reopened
-                                if isBibisectRequestClosed and not isThisBugClosed:
-                                    if actionDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[actionDay] = 0
-                                    bibisectRequestCountPerDay[actionDay] += 1
-                                    isBibisectRequestClosed = False
-
-                                # the bibisectRequest is being closed
-                                if not isBibisectRequestClosed and isThisBugClosed:
-                                    if actionDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[actionDay] = 0
-                                    bibisectRequestCountPerDay[actionDay] -= 1
-                                    isBibisectRequestClosed = True
-
-                            if isHighest:
-                                # the Highest priority bug is being reopened
-                                if isHighestClosed and not isThisBugClosed:
-                                    if actionDay not in highestCountPerDay:
-                                        highestCountPerDay[actionDay] = 0
-                                    highestCountPerDay[actionDay] += 1
-                                    isHighestClosed = False
-
-                                # the Highest priority bug is being closed
-                                if not isHighestClosed and isThisBugClosed:
-                                    if actionDay not in highestCountPerDay:
-                                        highestCountPerDay[actionDay] = 0
-                                    highestCountPerDay[actionDay] -= 1
-                                    isHighestClosed = True
-
-                            if isHigh:
-                                # the High priority bug is being reopened
-                                if isHighClosed and not isThisBugClosed:
-                                    if actionDay not in highCountPerDay:
-                                        highCountPerDay[actionDay] = 0
-                                    highCountPerDay[actionDay] += 1
-                                    isHighClosed = False
-
-                                # the High priority bug is being closed
-                                if not isHighClosed and isThisBugClosed:
-                                    if actionDay not in highCountPerDay:
-                                        highCountPerDay[actionDay] = 0
-                                    highCountPerDay[actionDay] -= 1
-                                    isHighClosed = True
-
-                            if check_date(actionDate, cfg):
-                                if removedStatus == "UNCONFIRMED":
-                                    util_increase_action(statList['confirmed'], rowId, actionMail, actionDay, diffTime)
-                                    dayConfirmed = actionDay
-                                    authorConfirmed = actionMail
-                                    isConfirmed = True
-
-                                elif addedStatus == 'UNCONFIRMED' and isConfirmed:
-                                    util_decrease_action(statList['confirmed'], authorConfirmed, dayConfirmed)
-                                    isConfirmed = False
-
-                                if addedStatus == 'VERIFIED':
-                                    util_increase_action(statList['verified'], rowId, actionMail, actionDay, diffTime)
-                                    dayVerified = actionDay
-                                    authorVerified = actionMail
-                                    isVerified = True
-
-                                elif removedStatus == 'VERIFIED' and isVerified and common.isOpen(addedStatus):
-                                    util_decrease_action(statList['verified'], authorVerified, dayVerified)
-                                    isVerified = False
-
-                        elif change['field_name'] == 'resolution':
-                            if check_date(actionDate, cfg):
-                                addedResolution = change['added']
-                                removedResolution = change['removed']
-                                if addedResolution == 'FIXED':
-                                    fixedBugs[rowId] = actionDate
-                                    isFixed = True
-
-                                elif removedResolution == 'FIXED' and isFixed:
-                                    del fixedBugs[rowId]
-                                    isFixed = False
-
-                        elif change['field_name'] == 'keywords':
-                            keywordsAdded = change['added'].lower().split(", ")
-                            keywordsRemoved = change['removed'].lower().split(", ")
-
-                            if check_date(actionDate, cfg):
-                                for keyword in keywordsAdded:
-                                    if keyword in lKeywords:
-                                        util_increase_action(statList['keywords'][keyword], rowId, actionMail, actionDay, diffTime)
-
-                            # TODO: IsThisBugClosed should be check here, but the result is not accurate
-                            if not isRegressionClosed:
-                                if not isRegression and 'regression' in keywordsAdded:
-                                    if actionDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[actionDay] = 0
-                                    regressionsCountPerDay[actionDay] += 1
-                                    isRegression = True
-
-                                if isRegression and 'regression' in keywordsRemoved:
-                                    if actionDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[actionDay] = 0
-                                    regressionsCountPerDay[actionDay] -= 1
-                                    isRegression = False
-
-                            # In the past, 'bibisectRequest' was added after the bug got fixed
-                            # to find the commit fixing it. Ignore them
-                            if not isThisBugClosed and not isBibisectRequestClosed:
-                                if not isBibisectRequest and 'bibisectrequest' in keywordsAdded:
-                                    if actionDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[actionDay] = 0
-                                    bibisectRequestCountPerDay[actionDay] += 1
-                                    isBibisectRequest = True
-
-                                if isBibisectRequest and 'bibisectrequest' in keywordsRemoved:
-                                    if actionDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[actionDay] = 0
-                                    bibisectRequestCountPerDay[actionDay] -= 1
-                                    isBibisectRequest = False
-
-                        elif change['field_name'] == 'blocks':
-                            if check_date(actionDate, cfg):
-                                if change['added']:
-                                    for metabug in change['added'].split(', '):
-                                        if int(metabug) in row['blocks']:
-                                            util_increase_action(statList['metabug'], rowId, actionMail, actionDay, diffTime)
+                    if change['field_name'] == 'priority':
+                        addedPriority = change['added']
+                        removedPriority = change['removed']
+
+                        # Sometimes the priority is increased to highest after the bug is fixed
+                        # Ignore those cases
+                        if not isThisBugClosed and not isHighestClosed:
+                            if not isHighest and addedPriority == "highest":
+                                if actionDay not in highestCountPerDay:
+                                    highestCountPerDay[actionDay] = 0
+                                highestCountPerDay[actionDay] += 1
+                                isHighest = True
+
+                            if isHighest and removedPriority == "highest":
+                                if actionDay not in highestCountPerDay:
+                                    highestCountPerDay[actionDay] = 0
+                                highestCountPerDay[actionDay] -= 1
+                                isHighest = False
+
+                        # TODO: IsThisBugClosed should be check here, but the result is not accurate
+                        if not isHighClosed:
+                            if not isHigh and addedPriority == "high":
+                                if actionDay not in highCountPerDay:
+                                    highCountPerDay[actionDay] = 0
+                                highCountPerDay[actionDay] += 1
+                                isHigh = True
+
+                            if isHigh and removedPriority == "high":
+                                if actionDay not in highCountPerDay:
+                                    highCountPerDay[actionDay] = 0
+                                highCountPerDay[actionDay] -= 1
+                                isHigh = False
+
+                    if change['field_name'] == 'status':
+                        addedStatus = change['added']
+                        removedStatus = change['removed']
+
+                        if common.isOpen(addedStatus):
+                            isThisBugClosed = False
+                        else:
+                            isThisBugClosed = True
+
+                        #See above
+                        if rowId >= 89589:
+                            if removedStatus == "UNCONFIRMED":
+                                if actionDay not in unconfirmedCountPerDay:
+                                    unconfirmedCountPerDay[actionDay] = 0
+                                unconfirmedCountPerDay[actionDay] -= 1
+
+                            elif addedStatus == 'UNCONFIRMED':
+                                if actionDay not in unconfirmedCountPerDay:
+                                    unconfirmedCountPerDay[actionDay] = 0
+                                unconfirmedCountPerDay[actionDay] += 1
+
+                        if isRegression:
+                            # the regression is being reopened
+                            if isRegressionClosed and not isThisBugClosed:
+                                if actionDay not in regressionsCountPerDay:
+                                    regressionsCountPerDay[actionDay] = 0
+                                regressionsCountPerDay[actionDay] += 1
+                                isRegressionClosed = False
+
+                            # the regression is being closed
+                            if not isRegressionClosed and isThisBugClosed:
+                                if actionDay not in regressionsCountPerDay:
+                                    regressionsCountPerDay[actionDay] = 0
+                                regressionsCountPerDay[actionDay] -= 1
+                                isRegressionClosed = True
+
+                        if isBibisectRequest:
+                            # the bibisectRequest is being reopened
+                            if isBibisectRequestClosed and not isThisBugClosed:
+                                if actionDay not in bibisectRequestCountPerDay:
+                                    bibisectRequestCountPerDay[actionDay] = 0
+                                bibisectRequestCountPerDay[actionDay] += 1
+                                isBibisectRequestClosed = False
+
+                            # the bibisectRequest is being closed
+                            if not isBibisectRequestClosed and isThisBugClosed:
+                                if actionDay not in bibisectRequestCountPerDay:
+                                    bibisectRequestCountPerDay[actionDay] = 0
+                                bibisectRequestCountPerDay[actionDay] -= 1
+                                isBibisectRequestClosed = True
+
+                        if isHighest:
+                            # the Highest priority bug is being reopened
+                            if isHighestClosed and not isThisBugClosed:
+                                if actionDay not in highestCountPerDay:
+                                    highestCountPerDay[actionDay] = 0
+                                highestCountPerDay[actionDay] += 1
+                                isHighestClosed = False
+
+                            # the Highest priority bug is being closed
+                            if not isHighestClosed and isThisBugClosed:
+                                if actionDay not in highestCountPerDay:
+                                    highestCountPerDay[actionDay] = 0
+                                highestCountPerDay[actionDay] -= 1
+                                isHighestClosed = True
+
+                        if isHigh:
+                            # the High priority bug is being reopened
+                            if isHighClosed and not isThisBugClosed:
+                                if actionDay not in highCountPerDay:
+                                    highCountPerDay[actionDay] = 0
+                                highCountPerDay[actionDay] += 1
+                                isHighClosed = False
+
+                            # the High priority bug is being closed
+                            if not isHighClosed and isThisBugClosed:
+                                if actionDay not in highCountPerDay:
+                                    highCountPerDay[actionDay] = 0
+                                highCountPerDay[actionDay] -= 1
+                                isHighClosed = True
+
+                        if check_date(actionDate, cfg):
+                            if removedStatus == "UNCONFIRMED":
+                                util_increase_action(statList['confirmed'], rowId, actionMail, actionDay, diffTime)
+                                dayConfirmed = actionDay
+                                authorConfirmed = actionMail
+                                isConfirmed = True
+
+                            elif addedStatus == 'UNCONFIRMED' and isConfirmed:
+                                util_decrease_action(statList['confirmed'], authorConfirmed, dayConfirmed)
+                                isConfirmed = False
+
+                            if addedStatus == 'VERIFIED':
+                                util_increase_action(statList['verified'], rowId, actionMail, actionDay, diffTime)
+                                dayVerified = actionDay
+                                authorVerified = actionMail
+                                isVerified = True
+
+                            elif removedStatus == 'VERIFIED' and isVerified and common.isOpen(addedStatus):
+                                util_decrease_action(statList['verified'], authorVerified, dayVerified)
+                                isVerified = False
+
+                    elif change['field_name'] == 'resolution':
+                        if check_date(actionDate, cfg):
+                            addedResolution = change['added']
+                            removedResolution = change['removed']
+                            if addedResolution == 'FIXED':
+                                fixedBugs[rowId] = actionDate
+                                isFixed = True
+
+                            elif removedResolution == 'FIXED' and isFixed:
+                                del fixedBugs[rowId]
+                                isFixed = False
+
+                    elif change['field_name'] == 'keywords':
+                        keywordsAdded = change['added'].lower().split(", ")
+                        keywordsRemoved = change['removed'].lower().split(", ")
+
+                        if check_date(actionDate, cfg):
+                            for keyword in keywordsAdded:
+                                if keyword in lKeywords:
+                                    util_increase_action(statList['keywords'][keyword], rowId, actionMail, actionDay, diffTime)
+
+                        # TODO: IsThisBugClosed should be check here, but the result is not accurate
+                        if not isRegressionClosed:
+                            if not isRegression and 'regression' in keywordsAdded:
+                                if actionDay not in regressionsCountPerDay:
+                                    regressionsCountPerDay[actionDay] = 0
+                                regressionsCountPerDay[actionDay] += 1
+                                isRegression = True
+
+                            if isRegression and 'regression' in keywordsRemoved:
+                                if actionDay not in regressionsCountPerDay:
+                                    regressionsCountPerDay[actionDay] = 0
+                                regressionsCountPerDay[actionDay] -= 1
+                                isRegression = False
+
+                        # In the past, 'bibisectRequest' was added after the bug got fixed
+                        # to find the commit fixing it. Ignore them
+                        if not isThisBugClosed and not isBibisectRequestClosed:
+                            if not isBibisectRequest and 'bibisectrequest' in keywordsAdded:
+                                if actionDay not in bibisectRequestCountPerDay:
+                                    bibisectRequestCountPerDay[actionDay] = 0
+                                bibisectRequestCountPerDay[actionDay] += 1
+                                isBibisectRequest = True
+
+                            if isBibisectRequest and 'bibisectrequest' in keywordsRemoved:
+                                if actionDay not in bibisectRequestCountPerDay:
+                                    bibisectRequestCountPerDay[actionDay] = 0
+                                bibisectRequestCountPerDay[actionDay] -= 1
+                                isBibisectRequest = False
+
+                    elif change['field_name'] == 'blocks':
+                        if check_date(actionDate, cfg):
+                            if change['added']:
+                                for metabug in change['added'].split(', '):
+                                    if int(metabug) in row['blocks']:
+                                        util_increase_action(statList['metabug'], rowId, actionMail, actionDay, diffTime)
 
             commentMail = None
             comments = row['comments'][1:]
             bugFixers = []
-            commitNoticiation=False
+            commitNoticiation = False
             for idx, comment in enumerate(comments):
                 commentMail = comment['creator']
                 commentDate = datetime.strptime(comment['time'], "%Y-%m-%dT%H:%M:%SZ")
@@ -360,7 +360,8 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                             diffTime = (commentDate - creationDate).days
                             commentDay = commentDate.strftime("%Y-%m-%d")
                             util_increase_action(statList['fixed'], rowId, author, commentDay, diffTime)
-                            commitNoticiation=True
+                            commitNoticiation = True
+
                             if row['priority'] == "highest":
                                 statList['criticalFixed'][rowId]= {'summary': row['summary'], 'author': author}
                             elif 'crash' in row['summary'].lower():
commit edc500e381a1ac03a14245d8480a1eaffcfa27df
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 10:50:07 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: remove unneeded variable

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index e25b6e7..166da74 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -100,10 +100,10 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
             #Some old bugs were directly created as NEW, skipping the UNCONFIRMED status
             #Use the oldest bug ID in the unconfirmed list
             if rowId >= 89589:
-                strDay = creationDate.strftime("%Y-%m-%d")
-                if strDay not in unconfirmedCountPerDay:
-                    unconfirmedCountPerDay[strDay] = 0
-                unconfirmedCountPerDay[strDay] += 1
+                actionDay = creationDate.strftime("%Y-%m-%d")
+                if actionDay not in unconfirmedCountPerDay:
+                    unconfirmedCountPerDay[actionDay] = 0
+                unconfirmedCountPerDay[actionDay] += 1
 
             rowStatus = row['status']
             rowResolution = row['resolution']
@@ -156,33 +156,29 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                             # Ignore those cases
                             if not isThisBugClosed and not isHighestClosed:
                                 if not isHighest and addedPriority == "highest":
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highestCountPerDay:
-                                        highestCountPerDay[strDay] = 0
-                                    highestCountPerDay[strDay] += 1
+                                    if actionDay not in highestCountPerDay:
+                                        highestCountPerDay[actionDay] = 0
+                                    highestCountPerDay[actionDay] += 1
                                     isHighest = True
 
                                 if isHighest and removedPriority == "highest":
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highestCountPerDay:
-                                        highestCountPerDay[strDay] = 0
-                                    highestCountPerDay[strDay] -= 1
+                                    if actionDay not in highestCountPerDay:
+                                        highestCountPerDay[actionDay] = 0
+                                    highestCountPerDay[actionDay] -= 1
                                     isHighest = False
 
                             # TODO: IsThisBugClosed should be check here, but the result is not accurate
                             if not isHighClosed:
                                 if not isHigh and addedPriority == "high":
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highCountPerDay:
-                                        highCountPerDay[strDay] = 0
-                                    highCountPerDay[strDay] += 1
+                                    if actionDay not in highCountPerDay:
+                                        highCountPerDay[actionDay] = 0
+                                    highCountPerDay[actionDay] += 1
                                     isHigh = True
 
                                 if isHigh and removedPriority == "high":
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highCountPerDay:
-                                        highCountPerDay[strDay] = 0
-                                    highCountPerDay[strDay] -= 1
+                                    if actionDay not in highCountPerDay:
+                                        highCountPerDay[actionDay] = 0
+                                    highCountPerDay[actionDay] -= 1
                                     isHigh = False
 
                         if change['field_name'] == 'status':
@@ -197,83 +193,73 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                             #See above
                             if rowId >= 89589:
                                 if removedStatus == "UNCONFIRMED":
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in unconfirmedCountPerDay:
-                                        unconfirmedCountPerDay[strDay] = 0
-                                    unconfirmedCountPerDay[strDay] -= 1
+                                    if actionDay not in unconfirmedCountPerDay:
+                                        unconfirmedCountPerDay[actionDay] = 0
+                                    unconfirmedCountPerDay[actionDay] -= 1
 
                                 elif addedStatus == 'UNCONFIRMED':
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in unconfirmedCountPerDay:
-                                        unconfirmedCountPerDay[strDay] = 0
-                                    unconfirmedCountPerDay[strDay] += 1
+                                    if actionDay not in unconfirmedCountPerDay:
+                                        unconfirmedCountPerDay[actionDay] = 0
+                                    unconfirmedCountPerDay[actionDay] += 1
 
                             if isRegression:
                                 # the regression is being reopened
                                 if isRegressionClosed and not isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[strDay] = 0
-                                    regressionsCountPerDay[strDay] += 1
+                                    if actionDay not in regressionsCountPerDay:
+                                        regressionsCountPerDay[actionDay] = 0
+                                    regressionsCountPerDay[actionDay] += 1
                                     isRegressionClosed = False
 
                                 # the regression is being closed
                                 if not isRegressionClosed and isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[strDay] = 0
-                                    regressionsCountPerDay[strDay] -= 1
+                                    if actionDay not in regressionsCountPerDay:
+                                        regressionsCountPerDay[actionDay] = 0
+                                    regressionsCountPerDay[actionDay] -= 1
                                     isRegressionClosed = True
 
                             if isBibisectRequest:
                                 # the bibisectRequest is being reopened
                                 if isBibisectRequestClosed and not isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[strDay] = 0
-                                    bibisectRequestCountPerDay[strDay] += 1
+                                    if actionDay not in bibisectRequestCountPerDay:
+                                        bibisectRequestCountPerDay[actionDay] = 0
+                                    bibisectRequestCountPerDay[actionDay] += 1
                                     isBibisectRequestClosed = False
 
                                 # the bibisectRequest is being closed
                                 if not isBibisectRequestClosed and isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[strDay] = 0
-                                    bibisectRequestCountPerDay[strDay] -= 1
+                                    if actionDay not in bibisectRequestCountPerDay:
+                                        bibisectRequestCountPerDay[actionDay] = 0
+                                    bibisectRequestCountPerDay[actionDay] -= 1
                                     isBibisectRequestClosed = True
 
                             if isHighest:
                                 # the Highest priority bug is being reopened
                                 if isHighestClosed and not isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highestCountPerDay:
-                                        highestCountPerDay[strDay] = 0
-                                    highestCountPerDay[strDay] += 1
+                                    if actionDay not in highestCountPerDay:
+                                        highestCountPerDay[actionDay] = 0
+                                    highestCountPerDay[actionDay] += 1
                                     isHighestClosed = False
 
                                 # the Highest priority bug is being closed
                                 if not isHighestClosed and isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highestCountPerDay:
-                                        highestCountPerDay[strDay] = 0
-                                    highestCountPerDay[strDay] -= 1
+                                    if actionDay not in highestCountPerDay:
+                                        highestCountPerDay[actionDay] = 0
+                                    highestCountPerDay[actionDay] -= 1
                                     isHighestClosed = True
 
                             if isHigh:
                                 # the High priority bug is being reopened
                                 if isHighClosed and not isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highCountPerDay:
-                                        highCountPerDay[strDay] = 0
-                                    highCountPerDay[strDay] += 1
+                                    if actionDay not in highCountPerDay:
+                                        highCountPerDay[actionDay] = 0
+                                    highCountPerDay[actionDay] += 1
                                     isHighClosed = False
 
                                 # the High priority bug is being closed
                                 if not isHighClosed and isThisBugClosed:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in highCountPerDay:
-                                        highCountPerDay[strDay] = 0
-                                    highCountPerDay[strDay] -= 1
+                                    if actionDay not in highCountPerDay:
+                                        highCountPerDay[actionDay] = 0
+                                    highCountPerDay[actionDay] -= 1
                                     isHighClosed = True
 
                             if check_date(actionDate, cfg):
@@ -321,34 +307,30 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                             # TODO: IsThisBugClosed should be check here, but the result is not accurate
                             if not isRegressionClosed:
                                 if not isRegression and 'regression' in keywordsAdded:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[strDay] = 0
-                                    regressionsCountPerDay[strDay] += 1
+                                    if actionDay not in regressionsCountPerDay:
+                                        regressionsCountPerDay[actionDay] = 0
+                                    regressionsCountPerDay[actionDay] += 1
                                     isRegression = True
 
                                 if isRegression and 'regression' in keywordsRemoved:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in regressionsCountPerDay:
-                                        regressionsCountPerDay[strDay] = 0
-                                    regressionsCountPerDay[strDay] -= 1
+                                    if actionDay not in regressionsCountPerDay:
+                                        regressionsCountPerDay[actionDay] = 0
+                                    regressionsCountPerDay[actionDay] -= 1
                                     isRegression = False
 
                             # In the past, 'bibisectRequest' was added after the bug got fixed
                             # to find the commit fixing it. Ignore them
                             if not isThisBugClosed and not isBibisectRequestClosed:
                                 if not isBibisectRequest and 'bibisectrequest' in keywordsAdded:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[strDay] = 0
-                                    bibisectRequestCountPerDay[strDay] += 1
+                                    if actionDay not in bibisectRequestCountPerDay:
+                                        bibisectRequestCountPerDay[actionDay] = 0
+                                    bibisectRequestCountPerDay[actionDay] += 1
                                     isBibisectRequest = True
 
                                 if isBibisectRequest and 'bibisectrequest' in keywordsRemoved:
-                                    strDay = actionDate.strftime("%Y-%m-%d")
-                                    if strDay not in bibisectRequestCountPerDay:
-                                        bibisectRequestCountPerDay[strDay] = 0
-                                    bibisectRequestCountPerDay[strDay] -= 1
+                                    if actionDay not in bibisectRequestCountPerDay:
+                                        bibisectRequestCountPerDay[actionDay] = 0
+                                    bibisectRequestCountPerDay[actionDay] -= 1
                                     isBibisectRequest = False
 
                         elif change['field_name'] == 'blocks':
commit 9ad8b68705eee7f57f935b2a12a3a428e3b7970a
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Mar 19 10:38:38 2019 +0100
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 19 13:03:20 2019 +0100

    QA: ignore uitests commits

diff --git a/qa/createBlogReport.py b/qa/createBlogReport.py
index ddfe502..e25b6e7 100755
--- a/qa/createBlogReport.py
+++ b/qa/createBlogReport.py
@@ -373,7 +373,7 @@ def analyze_bugzilla_data(statList, bugzillaData, cfg):
                     if commentMail == "libreoffice-commits at lists.freedesktop.org":
                         commentText = comment['text']
                         author =  commentText.split(' committed a patch related')[0]
-                        if author not in bugFixers:
+                        if author not in bugFixers and 'uitest' not in commentText.lower():
                             bugFixers.append(author)
                             diffTime = (commentDate - creationDate).days
                             commentDay = commentDate.strftime("%Y-%m-%d")


More information about the Libreoffice-commits mailing list