TranslationBranchApprover: permission error on TranslationMessage
Bug #487447 reported by
Jeroen T. Vermeulen
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Jeroen T. Vermeulen |
Bug Description
We've had some errors in the translations branch approver:
ProgrammingError: permission denied for relation translationmessage
OOPS-1421RSBR1, OOPS-1421RSBR2, OOPS-1421RSBR3
Related branches
lp:~jtv/launchpad/bug-487447
- Abel Deuring (community): Approve (code)
-
Diff: 357 lines (+120/-67)10 files modifieddatabase/schema/security.cfg (+25/-23)
lib/lp/testing/factory.py (+0/-1)
lib/lp/translations/browser/potemplate.py (+6/-5)
lib/lp/translations/interfaces/pofile.py (+3/-0)
lib/lp/translations/interfaces/potemplate.py (+1/-2)
lib/lp/translations/model/pofile.py (+13/-16)
lib/lp/translations/model/potemplate.py (+5/-11)
lib/lp/translations/model/translationimportqueue.py (+3/-2)
lib/lp/translations/scripts/tests/test_message_sharing_migration.py (+3/-1)
lib/lp/translations/tests/test_autoapproval.py (+61/-6)
lp:~jtv/launchpad/son-of-bug-487447
- Henning Eggers (community): Approve (code)
-
Diff: 224 lines (+71/-4)2 files modifieddatabase/schema/security.cfg (+1/-0)
lib/lp/translations/tests/test_translationbranchapprover.py (+70/-4)
summary: |
- Job system error: permission denied for relation TranslationMessage + TranslationBranchApprover: permission error on TranslationMessage |
description: | updated |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in rosetta: | |
status: | Fix Committed → Fix Released |
Changed in rosetta: | |
milestone: | 3.1.11 → 3.1.13 |
status: | Triaged → In Progress |
Changed in rosetta: | |
status: | Fix Committed → Fix Released |
Changed in rosetta: | |
milestone: | 3.1.13 → 10.01 |
To post a comment you must log in.
It goes wrong when the translations branch scanner needs to create a new POFile. I'm probably missing something, but ISTM this would also require INSERT rights on POFile, and possibly more.