Merge lp:~gary/launchpad/fix-578-patch into lp:launchpad

[Gary Poster]

[release-critical=wallyworld][r=bac,gary][no-qa] adds a missing SQL insert statement

This branch adds a missing SQL insert statement. It takes about 2 minutes to run on staging, according to stub. It was supposed to be a part of https://code.launchpad.net/~gary/launchpad/bug548-db-2/+merge/48318 but we missed it.

Without it, the branch to be deployed gives https://pastebin.canonical.com/42925/ as the output. With it, as verified by stub applying on staging, we get emails properly again.

[Ian Booth (wallyworld)]

review: Approve (release-critical)

[Gary Poster (gary)]

stub gave me an rs on IRC earlier, so I'll make the call here:

gary: ok, I'll point them to this conversation then. Are you going to be around long enough for me to make 38-1 and get your blessing then, stub, or should I break protocol?
stub: rs=stub.
gary: got it
gary: thanks
stub: i might be here but its not rocketscience this one

review: Approve (db)

[Brad Crittenden (bac)]

The patch looks good to me. (I think an additional set of eyes is good but I don't think a code review is necessary here.)

review: Approve (code)

=== added file 'database/schema/patch-2208-38-1.sql'
--- database/schema/patch-2208-38-1.sql	1970-01-01 00:00:00 +0000
+++ database/schema/patch-2208-38-1.sql	2011-02-07 15:12:19 +0000
@@ -0,0 +1,8 @@
+SET client_min_messages=ERROR;
+
+INSERT INTO PersonSettings (person)
+    SELECT id FROM Person WHERE teamowner IS NULL;
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2208, 38, 1);
+
+