launchpad-reviewers team mailing list archive

[William Grant <me@xxxxxxxxxxxxxxxxxx>]

William Grant has proposed merging lp:~wgrant/launchpad/gpgkey-fks-db-pre into lp:launchpad.

Commit message:
Add DB permissions to make person merge not crash during the GPGKey foreign key migration.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~wgrant/launchpad/gpgkey-fks-db-pre/+merge/287931

Add DB permissions to make person merge not crash during the GPGKey foreign key migration.

It'll still be slow until we create the indexes in a hot patch a few minutes after the FDT, but that's not so bad.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~wgrant/launchpad/gpgkey-fks-db-pre into lp:launchpad.

=== modified file 'database/schema/security.cfg'
--- database/schema/security.cfg	2016-03-02 21:21:26 +0000
+++ database/schema/security.cfg	2016-03-03 13:11:48 +0000
@@ -2212,6 +2212,7 @@
 public.packagediff                      = SELECT, UPDATE
 public.packageset                       = SELECT, UPDATE
 public.packagesetgroup                  = SELECT, UPDATE
+public.packageupload                    = SELECT, UPDATE
 public.packaging                        = SELECT, UPDATE
 public.person                           = SELECT, UPDATE
 public.personlanguage                   = SELECT, UPDATE
@@ -2234,6 +2235,7 @@
 public.questionjob                      = SELECT, UPDATE
 public.questionreopening                = SELECT, UPDATE
 public.questionsubscription             = SELECT, UPDATE, DELETE
+public.revision                         = SELECT, UPDATE
 public.revisionauthor                   = SELECT, UPDATE
 public.seriessourcepackagebranch        = SELECT, UPDATE
 public.sharingjob                       = SELECT, INSERT, UPDATE