[ews-app] Status bubble should not display certain infrastructure status messages
authoraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Apr 2019 20:27:12 +0000 (20:27 +0000)
committeraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Apr 2019 20:27:12 +0000 (20:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196550

Reviewed by Lucas Forschler.

Do not display the steps which are un-needed for end-user. This information is still avaiable
if the user click on the bubble and look at the Buildbot build.
* BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
(StatusBubble._steps_messages):
(StatusBubble._should_display_step): Method to decide whether to display the step or not.

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

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

index 9a5015b..072a29c 100644 (file)
@@ -41,6 +41,8 @@ class StatusBubble(View):
                     'mac-debug', 'mac-debug-wk1', 'mac-wk1', 'mac-wk2', 'style', 'webkitperl', 'webkitpy', 'win', 'wincairo', 'wpe']
     ENABLED_QUEUES = ['api-ios', 'api-mac', 'webkitperl']
 
+    STEPS_TO_HIDE = ['Killed old processes', 'Configured build', '^OS:.*Xcode:']
+
     def _build_bubble(self, patch, queue):
         bubble = {
             'name': queue,
@@ -120,7 +122,10 @@ class StatusBubble(View):
         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()])
+        return '\n'.join([step.state_string for step in build.step_set.all() if self._should_display_step(step)])
+
+    def _should_display_step(self, step):
+        return not filter(lambda step_to_hide: re.search(step_to_hide, step.state_string), StatusBubble.STEPS_TO_HIDE)
 
     def _most_recent_step_message(self, build):
         recent_step = build.step_set.last()
index a52673a..414334f 100644 (file)
@@ -1,5 +1,18 @@
 2019-04-03  Aakash Jain  <aakash_jain@apple.com>
 
+        [ews-app] Status bubble should not display certain infrastructure status messages
+        https://bugs.webkit.org/show_bug.cgi?id=196550
+
+        Reviewed by Lucas Forschler.
+
+        Do not display the steps which are un-needed for end-user. This information is still avaiable
+        if the user click on the bubble and look at the Buildbot build.
+        * BuildSlaveSupport/ews-app/ews/views/statusbubble.py:
+        (StatusBubble._steps_messages):
+        (StatusBubble._should_display_step): Method to decide whether to display the step or not.
+
+2019-04-03  Aakash Jain  <aakash_jain@apple.com>
+
         [ews-build] Use https for sending data to ews-app
         https://bugs.webkit.org/show_bug.cgi?id=196492