Merge lp:~jtv/launchpad/bug-623865 into lp:launchpad

[Jeroen T. Vermeulen]

Raise TranslationTemplatesBuildJob priority.

= Bug 623865 =

Our TranslationTemplateBuildFarmJobs have been starved out of the build farm since Friday morning. This raises new jobs' priority a bit, as per discussion with bigjools.

To test:
{{{
./bin/test -vvc -m lp.translations.tests.test_translationtemplatesbuildfarmjob
}}}

To Q/A, confirm that these jobs are being processed. One way of doing it is to run this against the production db: http://paste.ubuntu.com/482149/

…and verifying that accumulation of jobs in status 0 ("waiting") does not continue. The older jobs will probably stick around for a while as they still have the older priority of 1000.

No lint.

Jeroen

[Julian Edwards (julian-edwards)]

review: Approve

[Данило Шеган (danilo)]

Don't forget to update the comment in the test which mentions "1,000" (at least when landing on devel).

review: Approve (production-change)

[Jeroen T. Vermeulen (jtv)]

Thanks for the approval, and the comment gotcha!

=== modified file 'lib/lp/translations/model/translationtemplatesbuildjob.py'
--- lib/lp/translations/model/translationtemplatesbuildjob.py	2010-08-20 20:31:18 +0000
+++ lib/lp/translations/model/translationtemplatesbuildjob.py	2010-08-25 11:11:02 +0000
@@ -69,9 +69,10 @@
 
     def score(self):
         """See `IBuildFarmJob`."""
-        # Hard-code score for now; anything other than 1000 is probably
-        # inappropriate.
-        return 1000
+        # Hard-code score for now.  Most PPA jobs start out at 2505;
+        # TranslationTemplateBuildJobs are fast so we want them at a
+        # higher priority.
+        return 2510
 
     def getLogFileName(self):
         """See `IBuildFarmJob`."""

=== modified file 'lib/lp/translations/tests/test_translationtemplatesbuildjob.py'
--- lib/lp/translations/tests/test_translationtemplatesbuildjob.py	2010-08-20 20:31:18 +0000
+++ lib/lp/translations/tests/test_translationtemplatesbuildjob.py	2010-08-25 11:11:02 +0000
@@ -117,9 +117,9 @@
             self.specific_job.getLogFileName(), other_job.getLogFileName())
 
     def test_score(self):
-        # For now, these jobs always score themselves at 1,000.  In the
+        # For now, these jobs always score themselves at 2510.  In the
         # future however the scoring system is to be revisited.
-        self.assertEqual(1000, self.specific_job.score())
+        self.assertEqual(2510, self.specific_job.score())
 
     def test_cleanUp(self):
         # TranslationTemplatesBuildJob has its own customized cleanup