Launchpad itself

Merge lp:~gary/launchpad/bug994158 into lp:launchpad

  1. bug994158
  2. Merge into devel
Proposed by Gary Poster on 2012-05-03
Status: Merged
Approved by: Brad Crittenden on 2012-05-03
Approved revision: no longer in the source branch.
Merged at revision: 15200
Proposed branch: lp:~gary/launchpad/bug994158
Merge into: lp:launchpad
Diff against target: 11 lines (+4/-0)
1 file modified
lib/lp/soyuz/doc/package-cache-script.txt (+4/-0)
To merge this branch: bzr merge lp:~gary/launchpad/bug994158
Reviewer Review Type Date Requested Status
Brad Crittenden (community) code 2012-05-03 Approve on 2012-05-03
Review via email: mp+104618@code.launchpad.net

Commit Message

Mark the db as dirty after a cronscript test so our automatic processes clean the db up for the next test.

Description of the Change

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.

To post a comment you must log in.
Brad Crittenden (bac) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/soyuz/doc/package-cache-script.txt'
2--- lib/lp/soyuz/doc/package-cache-script.txt 2011-12-29 05:29:36 +0000
3+++ lib/lp/soyuz/doc/package-cache-script.txt 2012-05-03 20:30:26 +0000
4@@ -72,3 +72,7 @@
5 >>> warty.searchPackages('foobar').count()
6 0
7
8+Explicitly mark the database as dirty so that it is cleaned (see bug 994158).
9+
10+ >>> from lp.testing.layers import DatabaseLayer
11+ >>> DatabaseLayer.force_dirty_database()