2011-05-11 Eric Seidel <eric@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 May 2011 11:30:37 +0000 (11:30 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 11 May 2011 11:30:37 +0000 (11:30 +0000)
        Reviewed by Csaba Osztrogon√°c.

        queues.webkit.org should show 1000+ instead of 1000 when we hit the query limit
        https://bugs.webkit.org/show_bug.cgi?id=60614

        AppEngine has a result limit of 1000.  We hit that when trying
        to find out how many patches the style-queue (and other queues)
        have passed in the last 30 days.  So lets display 1000+ instead
        of 1000 to avoid confusion. (Ossy was asking about this in #webkit tonight.)

        I also made the status pages show "never" instead of just "ago" when we
        haven't seen a pass or boot message in the last 1000 messages.

        * QueueStatusServer/handlers/queuestatus.py:
        * QueueStatusServer/templates/queuestatus.html:

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

Tools/ChangeLog
Tools/QueueStatusServer/handlers/queuestatus.py
Tools/QueueStatusServer/templates/queuestatus.html

index 2f31e88..add9217 100644 (file)
@@ -1,3 +1,21 @@
+2011-05-11  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Csaba Osztrogon√°c.
+
+        queues.webkit.org should show 1000+ instead of 1000 when we hit the query limit
+        https://bugs.webkit.org/show_bug.cgi?id=60614
+
+        AppEngine has a result limit of 1000.  We hit that when trying
+        to find out how many patches the style-queue (and other queues)
+        have passed in the last 30 days.  So lets display 1000+ instead
+        of 1000 to avoid confusion. (Ossy was asking about this in #webkit tonight.)
+
+        I also made the status pages show "never" instead of just "ago" when we
+        haven't seen a pass or boot message in the last 1000 messages.
+
+        * QueueStatusServer/handlers/queuestatus.py:
+        * QueueStatusServer/templates/queuestatus.html:
+
 2011-05-11  Adam Barth  <abarth@webkit.org>
 
         Reviewed by Eric Seidel.
index 9eaef6f..9054fca 100644 (file)
@@ -82,6 +82,13 @@ class QueueStatus(webapp.RequestHandler):
         statuses.filter("message =", "Pass")
         return statuses.count()
 
+    def _fetch_trailing_days_pass_count_string(self, queue, bot_id, days):
+        status_count = self._fetch_trailing_days_pass_count(queue, bot_id, days)
+        # GQL has a result limit of 1000, so we return a special string to indicate we hit that limit.
+        if status_count == 1000:
+            status_count = "1000+"
+        return str(status_count)
+
     def _page_title(self, queue, bot_id):
         title = "%s Messages" % queue.display_name()
         if bot_id:
@@ -103,7 +110,7 @@ class QueueStatus(webapp.RequestHandler):
             "bot_id": bot_id,
             "last_pass": self._fetch_last_message_matching(queue, bot_id, "Pass"),
             "last_boot": self._fetch_last_message_matching(queue, bot_id, "Starting Queue"),
-            "trailing_month_pass_count": self._fetch_trailing_days_pass_count(queue, bot_id, 30),
-            "trailing_week_pass_count": self._fetch_trailing_days_pass_count(queue, bot_id, 7),
+            "trailing_month_pass_count": self._fetch_trailing_days_pass_count_string(queue, bot_id, 30),
+            "trailing_week_pass_count": self._fetch_trailing_days_pass_count_string(queue, bot_id, 7),
         }
         self.response.out.write(template.render("templates/queuestatus.html", template_values))
index f5b4523..f8eb61e 100644 (file)
@@ -9,13 +9,13 @@
 
 <h3>Summary</h3>
 <div>
-Last Pass: {{ last_pass.date|timesince }} ago
+Last Pass: {% if last_pass %}{{ last_pass.date|timesince }} ago{% else %}never{% endif %}
 {% if not bot_id and last_pass.bot_id %}
 by <a href="/queue-status/{{last_pass.queue_name}}/bots/{{last_pass.bot_id}}">{{ last_pass.bot_id }}</a>
 {% endif %}
 </div>
 <div>
-Last Boot: {{ last_boot.date|timesince }} ago
+Last Boot: {% if last_boot %}{{ last_boot.date|timesince }} ago{% else %}never{% endif %}
 {% if not bot_id and last_boot.bot_id %}
 by <a href="/queue-status/{{last_boot.queue_name}}/bots/{{last_boot.bot_id}}">{{ last_boot.bot_id }}</a>
 {% endif %}