Merge lp:~danilo/launchpad/bug-125225 into lp:launchpad/db-devel

Proposed by Данило Шеган
Status: Merged
Approved by: Данило Шеган
Approved revision: no longer in the source branch.
Merged at revision: 9353
Proposed branch: lp:~danilo/launchpad/bug-125225
Merge into: lp:launchpad/db-devel
Diff against target: 26 lines (+5/-1)
2 files modified
database/schema/security.cfg (+4/-0)
scripts/copy-translations-from-parent.py (+1/-1)
To merge this branch: bzr merge lp:~danilo/launchpad/bug-125225
Reviewer Review Type Date Requested Status
Eleanor Berger (community) Approve
Review via email: mp+24981@code.launchpad.net

Commit message

Provide and use a separate DB user for copy-translations-from-parent script.

Description of the change

= Bug 125225 =

Provide and use a separate DB user for copy-translations-from-parent script: use the same permissions it used to use already by "inheriting" poimport permissions.

= Tests =

bin/test -vvct distroseries-translations-copy

= Demo & QA =

On your local installation try ./scripts/copy-translations-from-parent.py -d ubuntu -s breezy-autotest -vvv -f

Similar for testing on staging, though we'll probably need to create a new distroseries that has parent_series set to one of existing Ubuntu distroseries.

To post a comment you must log in.
Revision history for this message
Eleanor Berger (intellectronica) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'database/schema/security.cfg'
2--- database/schema/security.cfg 2010-05-03 06:59:06 +0000
3+++ database/schema/security.cfg 2010-05-10 11:32:33 +0000
4@@ -427,6 +427,10 @@
5 public.validpersoncache = SELECT
6 public.validpersonorteamcache = SELECT
7
8+[translations_distroseries_copy]
9+type=user
10+groups=poimport
11+
12 [translations_import_queue_gardener]
13 # Translations import queue management
14 type=user
15
16=== modified file 'scripts/copy-translations-from-parent.py'
17--- scripts/copy-translations-from-parent.py 2010-04-22 17:30:35 +0000
18+++ scripts/copy-translations-from-parent.py 2010-05-10 11:32:33 +0000
19@@ -98,7 +98,7 @@
20 if __name__ == '__main__':
21
22 script = TranslationsCopier(
23- 'copy-missing-translations', dbuser=config.poimport.dbuser)
24+ 'copy-missing-translations', dbuser='translations_distroseries_copy')
25
26 script.lock_and_run()
27

Subscribers

People subscribed via source and target branches

to status/vote changes: