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
=== modified file 'database/schema/security.cfg'
--- database/schema/security.cfg 2010-05-03 06:59:06 +0000
+++ database/schema/security.cfg 2010-05-10 11:32:33 +0000
@@ -427,6 +427,10 @@
427public.validpersoncache = SELECT427public.validpersoncache = SELECT
428public.validpersonorteamcache = SELECT428public.validpersonorteamcache = SELECT
429429
430[translations_distroseries_copy]
431type=user
432groups=poimport
433
430[translations_import_queue_gardener]434[translations_import_queue_gardener]
431# Translations import queue management435# Translations import queue management
432type=user436type=user
433437
=== modified file 'scripts/copy-translations-from-parent.py'
--- scripts/copy-translations-from-parent.py 2010-04-22 17:30:35 +0000
+++ scripts/copy-translations-from-parent.py 2010-05-10 11:32:33 +0000
@@ -98,7 +98,7 @@
98if __name__ == '__main__':98if __name__ == '__main__':
9999
100 script = TranslationsCopier(100 script = TranslationsCopier(
101 'copy-missing-translations', dbuser=config.poimport.dbuser)101 'copy-missing-translations', dbuser='translations_distroseries_copy')
102102
103 script.lock_and_run()103 script.lock_and_run()
104104

Subscribers

People subscribed via source and target branches

to status/vote changes: