From 4ab5f75e1a470090172c68fbcf822906417036f7 Mon Sep 17 00:00:00 2001 From: Anthony Johnson Date: Wed, 25 Apr 2018 09:09:36 -0600 Subject: [PATCH] Remove build queue length warning on build list page This piece of UI was never updated to fit somewhere more cohesive in the UI and I feel is a strange thing to expose directly to users. I think a better application for this is in an eventual status page. A status page is not a priority atm though. --- readthedocs/builds/views.py | 8 -------- readthedocs/templates/builds/build_list.html | 12 ------------ 2 files changed, 20 deletions(-) diff --git a/readthedocs/builds/views.py b/readthedocs/builds/views.py index 4be37d268b3..c6335fe36f7 100644 --- a/readthedocs/builds/views.py +++ b/readthedocs/builds/views.py @@ -16,8 +16,6 @@ from readthedocs.core.utils import trigger_build from readthedocs.projects.models import Project -from redis import Redis, ConnectionError - log = logging.getLogger(__name__) @@ -67,12 +65,6 @@ def get_context_data(self, **kwargs): context['versions'] = Version.objects.public(user=self.request.user, project=self.project) context['build_qs'] = self.get_queryset() - try: - redis = Redis.from_url(settings.BROKER_URL) - context['queue_length'] = redis.llen('celery') - except ConnectionError: - context['queue_length'] = None - return context diff --git a/readthedocs/templates/builds/build_list.html b/readthedocs/templates/builds/build_list.html index 7f4445c7c9f..77da12d8e4b 100644 --- a/readthedocs/templates/builds/build_list.html +++ b/readthedocs/templates/builds/build_list.html @@ -17,18 +17,6 @@ {% block content %} - -{% if queue_length > 10 %} -
-

- {% blocktrans with queue_length as queue_length %} - Whoa! The build queue is {{ queue_length }} tasks long, new builds may take a while. - {% endblocktrans %} -

-
-{% endif %} - -
{% autopaginate build_qs 15 %}