launchpad-reviewers team mailing list archive

[Stuart Bishop <stuart.bishop@xxxxxxxxxxxxx>]

Stuart Bishop has proposed merging lp:~stub/launchpad/distinct-db-users into lp:launchpad with lp:~stub/launchpad/trivial as a prerequisite.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #134012 in Launchpad itself: "script monitor needs a database user"
  https://bugs.launchpad.net/launchpad/+bug/134012
  Bug #815753 in Launchpad itself: "process-accepted, publish-distro, process-death-row, buildd-retry-depwait and generate-contents-files should use their own DB users"
  https://bugs.launchpad.net/launchpad/+bug/815753

For more details, see:
https://code.launchpad.net/~stub/launchpad/distinct-db-users/+merge/79915

= Summary =

Create a user for the pgmetrics statistics gathering script to connect as.
-- 
https://code.launchpad.net/~stub/launchpad/distinct-db-users/+merge/79915
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stub/launchpad/distinct-db-users into lp:launchpad.

=== modified file 'database/schema/security.cfg'
--- database/schema/security.cfg	2011-10-03 18:34:08 +0000
+++ database/schema/security.cfg	2011-10-20 06:46:27 +0000
@@ -2264,3 +2264,10 @@
 [publish_ftpmaster]
 type=user
 groups=archivepublisher
+
+[pgmetrics]
+type=user
+groups=read
+public.accountpassword = SELECT
+public.oauthnonce = SELECT
+public.openidconsumernonce = SELECT