Changeset 242967 in webkit

Timestamp:

Mar 14, 2019, 3:06:17 PM (7 years ago)

Author:

aakash_jain@apple.com

Message:

[ews-build] Generate status-bubble hover-over messages
​https://bugs.webkit.org/show_bug.cgi?id=195680

Reviewed by Lucas Forschler.

• BuildSlaveSupport/ews-app/ews/views/statusbubble.py:

(StatusBubble):
(StatusBubble._build_bubble): Generate bubbledetails_message which would be displayed
on hover-over.
(StatusBubble._iso_time):
(StatusBubble._steps_messages): Returns status messages from steps to be displayed in
hover-over message.
(StatusBubble._most_recent_step_message): Returns status message from most recent step.

Location:

trunk/Tools

Files:

• BuildSlaveSupport/ews-app/ews/views/statusbubble.py (modified) (4 diffs)
• ChangeLog (modified) (1 diff)

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

@@ -23 +23 @@
 from __future__ import unicode_literals
 
+import datetime
 import re
 
@@ -43 +44 @@
     def _build_bubble(self, patch, queue):
         bubble = {
-            "name": queue,
+            'name': queue,
         }
         build = self.get_latest_build_for_queue(patch, queue)
@@ -50 +51 @@
 
         if not build:
-            bubble["state"] = "none"
-            bubble["details_message"] = 'Waiting in queue, processing has not started yet.'
+            bubble['state'] = 'none'
+            bubble['details_message'] = 'Waiting in queue, processing has not started yet.'
             return bubble
 
-        bubble["url"] = 'https://{}/#/builders/{}/builds/{}'.format(config.BUILDBOT_SERVER_HOST, build.builder_id, build.number)
-        builder_full_name = build.builder_name
-        bubble["details_message"] = '{}\n{}'.format(builder_full_name, build.state_string)
-        if build.result is None:
-            bubble["state"] = "started"
+        bubble['url'] = 'https://{}/#/builders/{}/builds/{}'.format(config.BUILDBOT_SERVER_HOST, build.builder_id, build.number)
+        builder_full_name = build.builder_name.replace('-', ' ')
+
+        if build.result is None:  # In-progress build
+            bubble['state'] = 'started'
+            bubble['details_message'] = 'Recent messages:\n\n' + self._steps_messages(build) + '\n\n' + self._iso_time(build.step_set.last().started_at)
         elif build.result == Buildbot.SUCCESS:
-            bubble["state"] = "pass"
+            bubble['state'] = 'pass'
+            bubble['details_message'] = 'Pass\n\n' + self._iso_time(build.complete_at)
         elif build.result == Buildbot.WARNINGS:
-            bubble["state"] = "pass"
+            bubble['state'] = 'pass'
+            bubble['details_message'] = 'Warning\n\n' + self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at)
         elif build.result == Buildbot.FAILURE:
-            bubble["state"] = "fail"
+            bubble['state'] = 'fail'
+            bubble['details_message'] = self._most_recent_step_message(build) + '\n\n' + self._iso_time(build.complete_at)
         elif build.result == Buildbot.SKIPPED:
-            bubble["state"] = "none"
+            bubble['state'] = 'none'
+            bubble['details_message'] = 'The patch is no longer eligible for processing.'
+            if re.search(r'Bug .* is already closed', build.state_string):
+                bubble['details_message'] += ' Bug was already closed when EWS attempted to process it.'
+            elif re.search(r'Patch .* is marked r-', build.state_string):
+                bubble['details_message'] += ' Patch was already marked r- when EWS attempted to process it.'
+            elif re.search(r'Patch .* is obsolete', build.state_string):
+                bubble['details_message'] += ' Patch was obsolete when EWS attempted to process it.'
+            bubble['details_message'] += '\nSome messages were logged while the patch was still eligible:\n\n'
+            bubble['details_message'] += self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at)
+
         elif build.result == Buildbot.EXCEPTION:
-            bubble["state"] = "error"
+            bubble['state'] = 'error'
+            bubble['details_message'] += ('An unexpected error occured. Recent messages:\n\n'
+                + self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at))
         elif build.result == Buildbot.RETRY:
-            bubble["state"] = "provisional-fail"
+            bubble['state'] = 'provisional-fail'
+            bubble['details_message'] += ('Build is being retried. Recent messages:\n\n'
+                + self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at))
+        elif build.result == Buildbot.CANCELLED:
+            bubble['state'] = 'provisional-fail'
+            bubble['details_message'] += ('Build was cancelled. Recent messages:\n\n'
+                + self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at))
         else:
-            bubble["state"] = "fail"
+            bubble['state'] = 'error'
+            bubble['details_message'] += ('An unexpected error occured. Recent messages:\n\n'
+                + self._steps_messages(build) + '\n\n' + self._iso_time(build.complete_at))
+
+        if 'details_message' in bubble:
+            bubble['details_message'] = builder_full_name + '\n\n' + bubble['details_message']
 
         return bubble
+
+    def _iso_time(self, time):
+        return '[[' + datetime.datetime.fromtimestamp(time).isoformat() + 'Z]]'
+
+    def _steps_messages(self, build):
+        return '\n'.join([step.state_string for step in build.step_set.all()])
+
+    def _most_recent_step_message(self, build):
+        recent_step = build.step_set.last()
+        if not recent_step:
+            return ''
+        return recent_step.state_string
 
     def get_latest_build_for_queue(self, patch, queue):
@@ -120 +160 @@
 
         template_values = {
-            "bubbles": bubbles,
-            "patch_id": patch_id,
-            "show_submit_to_ews": show_submit_to_ews,
-            "show_failure_to_apply": show_failure_to_apply,
+            'bubbles': bubbles,
+            'patch_id': patch_id,
+            'show_submit_to_ews': show_submit_to_ews,
+            'show_failure_to_apply': show_failure_to_apply,
         }
         return render(request, 'statusbubble.html', template_values)

trunk/Tools/ChangeLog

@@ +1 @@
+2019-03-14  Aakash Jain  <aakash_jain@apple.com>
+
+        [ews-build] Generate status-bubble hover-over messages
+        https://bugs.webkit.org/show_bug.cgi?id=195680
+
+        Reviewed by Lucas Forschler.
+
+        * BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
+        (StatusBubble):
+        (StatusBubble._build_bubble): Generate bubble['details_message'] which would be displayed
+        on hover-over.
+        (StatusBubble._iso_time):
+        (StatusBubble._steps_messages): Returns status messages from steps to be displayed in
+        hover-over message.
+        (StatusBubble._most_recent_step_message): Returns status message from most recent step.
+
 2019-03-14  Aakash Jain  <aakash_jain@apple.com>