Bug 1404018 - Do not reset priority of owner-triaged bugs when already P2/P3; r=jmaher
authorGeoff Brown <gbrown@mozilla.com>
Thu, 28 Sep 2017 11:27:56 -0600
changeset 343 1e42f8946368b488d387d66621eaa7f7f34fbf9a
parent 342 247a7020da025219e7bcbe5b5f7bddfa2e6feb16
child 344 27840d346dfac131af23020468337f6d3f575736
push id291
push usergbrown@mozilla.com
push dateThu, 28 Sep 2017 17:28:21 +0000
reviewersjmaher
bugs1404018
Bug 1404018 - Do not reset priority of owner-triaged bugs when already P2/P3; r=jmaher
woo_commenter.py
--- a/woo_commenter.py
+++ b/woo_commenter.py
@@ -258,19 +258,18 @@ def main():
                     bug_info = get_triage_info_for_bug(bmo_session, bug_id)
                 whiteboard = bug_info['whiteboard']
                 if (((bug_info['product'], bug_info['component']) in components) and
                     ('intermittent-failure' in bug_info['keywords']) and
                     ('[stockwell fixed' not in whiteboard) and
                     ('[stockwell disabled' not in whiteboard) and
                     ('[stockwell infra' not in whiteboard)):
                     # do not update priority on bugs already awaiting triage or
-                    # already a high priority (P1 should be re-triaged weekly)
-                    if ((bug_info['priority'] != '--') and
-                        (bug_info['priority'] != 'P1')):
+                    # already P1/P2/P3 (hopefully recently triaged)
+                    if bug_info['priority'] not in ['--', 'P1', 'P2', 'P3']:
                         params['priority'] = '--'
                     # remove any [stockwell xxx] from whiteboard, then add
                     # [stockwell needswork:owner], unless it is already there
                     whiteboard = bug_info['whiteboard']
                     if not WHITEBOARD_NEEDSWORK_OWNER in whiteboard:
                         whiteboard = stockwell_whiteboard(whiteboard, WHITEBOARD_NEEDSWORK_OWNER)
         # When skip-threshold exceeded, also update whiteboard
         if priority == 3: