Changeset 174158 in webkit

Timestamp:

Oct 1, 2014 10:18:10 AM (10 years ago)

Author:

ap@apple.com

Message:

EWS is too eager to say that a patch does not apply to trunk of repository
​https://bugs.webkit.org/show_bug.cgi?id=137290

Reviewed by Ryosuke Niwa.

• QueueStatusServer/app.yaml: Updated app version.

• QueueStatusServer/handlers/statusbubble.py:

(StatusBubble._build_bubble):
(StatusBubble._build_bubbles_for_attachment):
Rewrote to make more sense.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• QueueStatusServer/app.yaml (modified) (1 diff)
• QueueStatusServer/handlers/statusbubble.py (modified) (4 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2014-10-01  Alexey Proskuryakov  <ap@apple.com>
+
+        EWS is too eager to say that a patch does not apply to trunk of repository
+        https://bugs.webkit.org/show_bug.cgi?id=137290
+
+        Reviewed by Ryosuke Niwa.
+
+        * QueueStatusServer/app.yaml: Updated app version.
+
+        * QueueStatusServer/handlers/statusbubble.py:
+        (StatusBubble._build_bubble):
+        (StatusBubble._build_bubbles_for_attachment):
+        Rewrote to make more sense.
+
 2014-09-30  Eva Balazsfalvi  <evab.u-szeged@partner.samsung.com>
 

trunk/Tools/QueueStatusServer/app.yaml

@@ -1 +1 @@
 application: webkit-queues
-version: 174124 # Bugzilla bug ID of last major change
+version: 174158 # Bugzilla bug ID of last major change
 runtime: python
 api_version: 1

trunk/Tools/QueueStatusServer/handlers/statusbubble.py

@@ -116 +116 @@
         statuses = QueueStatus.all().filter('queue_name =', queue.name()).filter('active_patch_id =', attachment.id).order('-date').fetch(limit=10)
         if not statuses:
+            bubble["had_resultative_status_other_than_failure_to_apply"] = False
             if attachment.id in queue.active_work_items().item_ids:
                 bubble["state"] = "started"
                 bubble["details_message"] = "Started processing, no output yet.\n\n" + self._iso_time(queue.active_work_items().time_for_item(attachment.id))
-                bubble["may_fail_to_apply"] = True
             else:
                 real_queue_position = self._real_queue_position(queue, queue_position)
@@ -125 +125 @@
                 bubble["details_message"] = "Waiting in queue, processing has not started yet.\n\nPosition in queue: " + str(real_queue_position)
                 bubble["queue_position"] = real_queue_position
-                bubble["may_fail_to_apply"] = True
         else:
             latest_resultative_status = self._latest_resultative_status(statuses)
+            bubble["had_resultative_status_other_than_failure_to_apply"] = any(map(lambda status:
+                latest_resultative_status and latest_resultative_status.message != "Error: " + queue.name() + " unable to apply patch.",
+                statuses))
             if not latest_resultative_status:
                 bubble["state"] = "started"
@@ -147 +149 @@
                         bubble["details_message"] += " Some messages were logged while the patch was still eligible:\n\n"
                     bubble["details_message"] += "\n".join([status.message for status in statuses[1:]]) + "\n\n" + self._iso_time(statuses[0].date)
-                bubble["may_fail_to_apply"] = True
             elif statuses[0].message == "Error: " + queue.name() + " unable to apply patch.":
                 bubble["state"] = "fail"
@@ -190 +191 @@
 
         failed_to_apply = any(map(lambda bubble: "failed_to_apply" in bubble, bubbles))
-        had_output = all(map(lambda bubble: not "may_fail_to_apply" in bubble and not "failed_to_apply" in bubble, bubbles))
-
-        return (bubbles, show_submit_to_ews, failed_to_apply and (not had_output) and (not show_submit_to_ews))
+        had_resultative_status_other_than_failure_to_apply = any(map(lambda bubble: bubble["had_resultative_status_other_than_failure_to_apply"], bubbles))
+
+        return (bubbles, show_submit_to_ews, failed_to_apply and not had_resultative_status_other_than_failure_to_apply)
 
     def get(self, attachment_id_string):