launchpad-reviewers team mailing list archive

[Gary Poster <gary.poster@xxxxxxxxxxxxx>]

Gary Poster has proposed merging lp:~gary/launchpad/bug994158 into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #994158 in Launchpad itself: "lp.registry.browser.tests.test_distroseriesdifference_views.DistroSeriesDifferenceTestCase.test_binary_summaries_for_source_pub fails intermittently/rarely in parallel tests with distroreleasepackagecache_pkey error"
  https://bugs.launchpad.net/launchpad/+bug/994158

For more details, see:
https://code.launchpad.net/~gary/launchpad/bug994158/+merge/104618

This simple change mirrors the changes done for related bug 849056 and bug 987904.  cronscripts do not mark the db as dirty in the current process, so we need to explicitly do so so that the db is reset.
-- 
https://code.launchpad.net/~gary/launchpad/bug994158/+merge/104618
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~gary/launchpad/bug994158 into lp:launchpad.

=== modified file 'lib/lp/soyuz/doc/package-cache-script.txt'
--- lib/lp/soyuz/doc/package-cache-script.txt	2011-12-29 05:29:36 +0000
+++ lib/lp/soyuz/doc/package-cache-script.txt	2012-05-03 20:30:26 +0000
@@ -72,3 +72,7 @@
   >>> warty.searchPackages('foobar').count()
   0
 
+Explicitly mark the database as dirty so that it is cleaned (see bug 994158).
+
+  >>> from lp.testing.layers import DatabaseLayer
+  >>> DatabaseLayer.force_dirty_database()