Launchpad itself

process-accepted, publish-distro, process-death-row, buildd-retry-depwait and generate-contents-files should use their own DB users

Bug #815753 reported by William Grant
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Stuart Bishop

Bug Description

The big publication scripts (process-accepted, publish-distro, process-death-row, and generate-contents-files) should run as distinct DB users, to allow people and scripts to distinguish their DB connections.

Related branches

lp:~stub/launchpad/distinct-db-users
Gavin Panella (community): Approve (code)
Raphaël Badin (community): Approve (code*)
Diff: 14 lines (+7/-0)
1 file modified
database/schema/security.cfg (+7/-0)
Revision history for this message
Stuart Bishop (stub) wrote :

https://code.launchpad.net/~stub/launchpad/db-cleanups/+merge/69038 fixes generate-contents-files and publish-ftpmaster

Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in db-stable r10815 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/db-stable/revision/10815>.

Changed in launchpad:
assignee: nobody → Stuart Bishop (stub)
tags: added: qa-needstesting
Changed in launchpad:
status: Triaged → Fix Committed
Revision history for this message
William Grant (wgrant) wrote :

Note that publish-ftpmaster is not in use yet.

Stuart Bishop (stub)
Changed in launchpad:
assignee: Stuart Bishop (stub) → nobody
status: Fix Committed → Triaged
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in db-stable r10923 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/db-stable/revision/10923>.

Changed in launchpad:
assignee: nobody → Stuart Bishop (stub)
status: Triaged → Fix Committed
Stuart Bishop (stub)
Changed in launchpad:
status: Fix Committed → Triaged
assignee: Stuart Bishop (stub) → nobody
tags: removed: qa-needstesting
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Why was this set to Critical? Was it causing production issues?

tags: added: critical-analysis
Revision history for this message
Stuart Bishop (stub) wrote :

Its a fastdowntime issue. These processes where all connecting as the same user (archivepublisher), but only some of them need to be shutdown manually before a fastdowntime deployment.

tags: added: qa-untestable
Revision history for this message
Stuart Bishop (stub) wrote :

Also buildd-retry-depwait

summary: - process-accepted, publish-distro, process-death-row and generate-
- contents-files should use their own DB users
+ process-accepted, publish-distro, process-death-row, buildd-retry-
+ depwait and generate-contents-files should use their own DB users
Revision history for this message
Stuart Bishop (stub) wrote :

Also process-upload.py

Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in stable r14013 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/14013>.

Changed in launchpad:
assignee: nobody → Stuart Bishop (stub)
tags: added: qa-needstesting
removed: qa-untestable
Changed in launchpad:
status: Triaged → Fix Committed
William Grant (wgrant)
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.