Title: [148146] trunk/Tools
Revision
148146
Author
[email protected]
Date
2013-04-10 16:11:24 -0700 (Wed, 10 Apr 2013)

Log Message

Remove Chromium EWS bots from the queue status server
https://bugs.webkit.org/show_bug.cgi?id=113988

Reviewed by Benjamin Poulain.

Removed Chromium EWS bots. Also updated it to use webkit-queues.appspot.com.

* QueueStatusServer/app.yaml:
* QueueStatusServer/config/queues.py:
* QueueStatusServer/model/queues.py:
(Queue.short_name):
(Queue.display_name):
* QueueStatusServer/model/queues_unittest.py:
(QueueTest.test_short_name):
(QueueTest.test_display_name):
(QueueTest.test_name_with_underscores):
* QueueStatusServer/templates/activebots.html:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (148145 => 148146)


--- trunk/Tools/ChangeLog	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/ChangeLog	2013-04-10 23:11:24 UTC (rev 148146)
@@ -1,3 +1,23 @@
+2013-04-10  Ryosuke Niwa  <[email protected]>
+
+        Remove Chromium EWS bots from the queue status server
+        https://bugs.webkit.org/show_bug.cgi?id=113988
+
+        Reviewed by Benjamin Poulain.
+
+        Removed Chromium EWS bots. Also updated it to use webkit-queues.appspot.com.
+
+        * QueueStatusServer/app.yaml:
+        * QueueStatusServer/config/queues.py:
+        * QueueStatusServer/model/queues.py:
+        (Queue.short_name):
+        (Queue.display_name):
+        * QueueStatusServer/model/queues_unittest.py:
+        (QueueTest.test_short_name):
+        (QueueTest.test_display_name):
+        (QueueTest.test_name_with_underscores):
+        * QueueStatusServer/templates/activebots.html:
+
 2013-04-10  Ojan Vafai  <[email protected]>
 
         Change all instances of test-results.appspot.com to webkit-test-results.appspot.com

Modified: trunk/Tools/QueueStatusServer/app.yaml (148145 => 148146)


--- trunk/Tools/QueueStatusServer/app.yaml	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/QueueStatusServer/app.yaml	2013-04-10 23:11:24 UTC (rev 148146)
@@ -1,4 +1,4 @@
-application: webkit-commit-queue
+application: webkit-queues
 version: 108593 # Bugzilla bug ID of last major change
 runtime: python
 api_version: 1

Modified: trunk/Tools/QueueStatusServer/config/queues.py (148145 => 148146)


--- trunk/Tools/QueueStatusServer/config/queues.py	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/QueueStatusServer/config/queues.py	2013-04-10 23:11:24 UTC (rev 148146)
@@ -30,9 +30,6 @@
 all_queue_names = [
     "commit-queue",
     "style-queue",
-    "chromium-ews",  # aka cr-linux-ews
-    "cr-linux-debug-ews",
-    "cr-android-ews",
     "qt-ews",
     "qt-wk2-ews",
     "gtk-ews",

Modified: trunk/Tools/QueueStatusServer/model/queues.py (148145 => 148146)


--- trunk/Tools/QueueStatusServer/model/queues.py	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/QueueStatusServer/model/queues.py	2013-04-10 23:11:24 UTC (rev 148146)
@@ -68,18 +68,12 @@
 
     # For use in status bubbles or table headers
     def short_name(self):
-        # HACK: chromium-ews is incorrectly named.
-        short_name = self._name.replace("chromium-ews", "Cr-Linux-ews")
-        short_name = short_name.replace("-ews", "")
+        short_name = self._name.replace("-ews", "")
         short_name = short_name.replace("-queue", "")
         return self._caplitalize_after_dash(short_name.capitalize())
 
     def display_name(self):
-        # HACK: chromium-ews is incorrectly named.
-        display_name = self._name.replace("chromium-ews", "cr-linux-ews")
-
-        display_name = display_name.replace("-", " ")
-        display_name = display_name.replace("cr", "chromium")
+        display_name = self._name.replace("-", " ")
         display_name = display_name.title()
         display_name = display_name.replace("Ews", "EWS")
         return display_name

Modified: trunk/Tools/QueueStatusServer/model/queues_unittest.py (148145 => 148146)


--- trunk/Tools/QueueStatusServer/model/queues_unittest.py	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/QueueStatusServer/model/queues_unittest.py	2013-04-10 23:11:24 UTC (rev 148146)
@@ -47,7 +47,6 @@
 
     def test_short_name(self):
         self._assert_short_name("mac-ews", "Mac")
-        self._assert_short_name("chromium-ews", "Cr-Linux")
         self._assert_short_name("commit-queue", "Commit")
         self._assert_short_name("style-queue", "Style")
 
@@ -56,7 +55,6 @@
 
     def test_display_name(self):
         self._assert_display_name("mac-ews", "Mac EWS")
-        self._assert_display_name("chromium-ews", "Chromium Linux EWS")
         self._assert_display_name("commit-queue", "Commit Queue")
         self._assert_display_name("style-queue", "Style Queue")
 
@@ -65,7 +63,6 @@
 
     def test_name_with_underscores(self):
         self._assert_name_with_underscores("mac-ews", "mac_ews")
-        self._assert_name_with_underscores("chromium-ews", "chromium_ews")
         self._assert_name_with_underscores("commit-queue", "commit_queue")
 
     def test_style_queue_is_ews(self):

Modified: trunk/Tools/QueueStatusServer/templates/activebots.html (148145 => 148146)


--- trunk/Tools/QueueStatusServer/templates/activebots.html	2013-04-10 22:47:07 UTC (rev 148145)
+++ trunk/Tools/QueueStatusServer/templates/activebots.html	2013-04-10 23:11:24 UTC (rev 148146)
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
 <head>
-<title>WebKit Queue Status</title>
+<title>WebKit Queues</title>
 <link type="text/css" rel="stylesheet" href="" />
 <style>
 /* Override the generic table styles from dashboard.css */
@@ -11,7 +11,7 @@
 </style>
 </head>
 <body>
-<h1>WebKit Queue Status</h1>
+<h1>WebKit Queues</h1>
 <table>
     <thead>
         <tr>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to