Removed ENABLED_QUEUES logic from ews-app
authoraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Nov 2019 16:03:35 +0000 (16:03 +0000)
committeraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Nov 2019 16:03:35 +0000 (16:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=204683

Unreviewed minor fix.

* BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
(StatusBubble): Removed ENABLED_QUEUES.
(StatusBubble._build_bubbles_for_patch):
(StatusBubble._should_show_bubble_for_queue): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@252926 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/BuildSlaveSupport/ews-app/ews/views/statusbubble.py
Tools/ChangeLog

index 34591c7..a7bad31 100644 (file)
@@ -41,8 +41,6 @@ class StatusBubble(View):
     # FIXME: Auto-generate this list https://bugs.webkit.org/show_bug.cgi?id=195640
     ALL_QUEUES = ['style', 'ios', 'ios-sim', 'mac', 'mac-debug', 'gtk', 'wpe', 'wincairo', 'win',
                   'ios-wk2', 'mac-wk1', 'mac-wk2', 'mac-debug-wk1', 'api-ios', 'api-mac', 'bindings', 'jsc', 'jsc-armv7', 'jsc-mips', 'webkitperl', 'webkitpy', 'services']
-    ENABLED_QUEUES = ['style', 'ios', 'ios-sim', 'mac', 'mac-debug', 'gtk', 'wpe', 'wincairo', 'win',
-                      'ios-wk2', 'mac-wk1', 'mac-wk2', 'mac-debug-wk1', 'api-ios', 'api-mac', 'bindings', 'jsc', 'jsc-armv7', 'jsc-mips', 'webkitperl', 'webkitpy', 'services']
     # FIXME: Auto-generate the queue's trigger relationship
     QUEUE_TRIGGERS = {
         'api-ios': 'ios-sim',
@@ -249,9 +247,6 @@ class StatusBubble(View):
             return False
         return True
 
-    def _should_show_bubble_for_queue(self, queue):
-        return queue in StatusBubble.ENABLED_QUEUES
-
     def _queue_position(self, patch, queue, parent_queue=None):
         # FIXME: Handle retried builds and cancelled build-requests as well.
         from_timestamp = timezone.now() - datetime.timedelta(days=StatusBubble.DAYS_TO_CHECK)
@@ -290,9 +285,6 @@ class StatusBubble(View):
             return (None, show_submit_to_ews, failed_to_apply, show_retry)
 
         for queue in StatusBubble.ALL_QUEUES:
-            if not self._should_show_bubble_for_queue(queue):
-                continue
-
             bubble = self._build_bubble(patch, queue, hide_icons)
             if bubble:
                 show_submit_to_ews = False
index 35e4ea1..e3c18e4 100644 (file)
@@ -1,5 +1,17 @@
 2019-11-28  Aakash Jain  <aakash_jain@apple.com>
 
+        Removed ENABLED_QUEUES logic from ews-app
+        https://bugs.webkit.org/show_bug.cgi?id=204683
+
+        Unreviewed minor fix.
+
+        * BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
+        (StatusBubble): Removed ENABLED_QUEUES.
+        (StatusBubble._build_bubbles_for_patch):
+        (StatusBubble._should_show_bubble_for_queue): Deleted.
+
+2019-11-28  Aakash Jain  <aakash_jain@apple.com>
+
         [ews] Enable Windows queue on new EWS
         https://bugs.webkit.org/show_bug.cgi?id=204243