New EWS: patches on recently added queues listed as #1 for older bugs

https://bugs.webkit.org/show_bug.cgi?id=197496

Reviewed by Lucas Forschler.

* BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
(StatusBubble._build_bubble):
(StatusBubble._queue_position):


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

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

@@ -62,6 +62,8 @@ def _build_bubble(self, patch, queue):
             bubble['state'] = 'none'
             queue_position = self._queue_position(patch, queue, self._get_parent_queue(queue))
             bubble['queue_position'] = queue_position
+            if not queue_position:
+                return None
             bubble['details_message'] = 'Waiting in queue, processing has not started yet.\n\nPosition in queue: {}'.format(queue_position)
             return bubble
 
@@ -181,6 +183,12 @@ def _queue_position(self, patch, queue, parent_queue=None):
         DAYS_TO_CHECK = 3
         from_timestamp = timezone.now() - datetime.timedelta(days=DAYS_TO_CHECK)
 
+        if patch.modified < from_timestamp:
+            # Do not display bubble for old patch for which no build has been reported on given queue.
+            # Most likely the patch would never be processed on this queue, since either the queue was
+            # added after the patch was submitted, or build request for that patch was cancelled.
+            return None
+
         previously_sent_patches = set(Patch.objects
                                           .filter(modified__gte=from_timestamp)
                                           .filter(sent_to_buildbot=True)

Tools/ChangeLog

@@ -1,3 +1,14 @@
+2019-05-03  Aakash Jain  <aakash_jain@apple.com>
+
+        New EWS: patches on recently added queues listed as #1 for older bugs
+        https://bugs.webkit.org/show_bug.cgi?id=197496
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
+        (StatusBubble._build_bubble):
+        (StatusBubble._queue_position):
+
 2019-05-03  Aakash Jain  <aakash_jain@apple.com>
 
         webkit-patch --no-review upload does not submit patch to New EWS