Launchpad itself

+ppa-packages timeout

Bug #1071581 reported by Ian Booth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Ian Booth

Bug Description

This url times out for example: https://launchpad.net/~fta/+ppa-packages

OOPS-6f888dbc487d02ee1a54723848132090 show 95% of the sql time is getting the SourcePackageReleases.

See original description
Tags: ppa qa-ok timeout

Related branches

lp:~wallyworld/launchpad/ppa-packages-timeout-1071581
Curtis Hovey (community): Approve (code)
Diff: 857 lines (+534/-57)
11 files modified
database/sampledata/current-dev.sql (+47/-0)
database/sampledata/current.sql (+43/-0)
database/schema/security.cfg (+3/-0)
lib/lp/registry/browser/person.py (+10/-8)
lib/lp/registry/doc/person.txt (+1/-1)
lib/lp/registry/model/person.py (+40/-48)
lib/lp/scripts/garbo.py (+194/-0)
lib/lp/scripts/tests/test_garbo.py (+111/-0)
lib/lp/soyuz/configure.zcml (+6/-0)
lib/lp/soyuz/interfaces/reporting.py (+30/-0)
lib/lp/soyuz/model/reporting.py (+49/-0)
lp:~wallyworld/launchpad/latestpublishedreleases-1071581
Ian Booth (community): Approve
William Grant: Pending requested
Stuart Bishop: Pending (db) requested
Diff: 60 lines (+56/-0)
1 file modified
database/schema/patch-2209-38-0.sql (+56/-0)
Superseded for merging into lp:launchpad
Stuart Bishop (community): Approve (db)
William Grant: Pending requested
Diff: 60 lines (+56/-0)
1 file modified
database/schema/patch-2209-38-0.sql (+56/-0)
lp:~wallyworld/launchpad/person-fk-handling-1071581
Curtis Hovey (community): Approve (code)
Diff: 287 lines (+73/-18)
7 files modified
database/schema/security.cfg (+2/-1)
lib/lp/registry/browser/tests/test_person.py (+14/-0)
lib/lp/registry/model/person.py (+11/-4)
lib/lp/scripts/tests/test_garbo.py (+5/-4)
lib/lp/soyuz/interfaces/reporting.py (+1/-1)
lib/lp/soyuz/model/reporting.py (+1/-1)
lib/lp/soyuz/stories/soyuz/xx-person-packages.txt (+39/-7)
lp:~wallyworld/launchpad/latestpublishedreleases-person-fk-1071581
Stuart Bishop (community): Approve (db)
William Grant: Pending requested
Diff: 18 lines (+14/-0)
1 file modified
database/schema/patch-2209-38-1.sql (+14/-0)
lp:~wallyworld/launchpad/fast-related-software-feature-flag
Steve Kowalik (community): Approve (code)
Diff: 175 lines (+116/-0)
3 files modified
lib/lp/registry/browser/tests/test_person.py (+10/-0)
lib/lp/registry/model/person.py (+99/-0)
lib/lp/services/features/flags.py (+7/-0)
lp:~wallyworld/launchpad/improve-latestpersonspr-garbo-job-1071581
j.c.sackett (community): Approve
Diff: 103 lines (+25/-7)
2 files modified
lib/lp/scripts/garbo.py (+15/-7)
lib/lp/scripts/tests/test_garbo.py (+10/-0)
lp:~wallyworld/launchpad/another-reporting-cache-garbojob-1071581
William Grant: Approve (code)
Diff: 848 lines (+263/-189)
10 files modified
lib/lp/registry/browser/tests/test_person.py (+3/-3)
lib/lp/registry/model/person.py (+15/-16)
lib/lp/scripts/garbo.py (+155/-137)
lib/lp/scripts/tests/test_garbo.py (+14/-16)
lib/lp/services/database/bulk.py (+5/-4)
lib/lp/services/database/stormexpr.py (+58/-0)
lib/lp/soyuz/configure.zcml (+2/-2)
lib/lp/soyuz/interfaces/reporting.py (+3/-3)
lib/lp/soyuz/model/reporting.py (+6/-6)
lib/lp/soyuz/stories/soyuz/xx-person-packages.txt (+2/-2)
lp:~wallyworld/launchpad/report-cache-index-1071581
William Grant: Approve (db)
Diff: 14 lines (+10/-0)
1 file modified
database/schema/patch-2209-38-2.sql (+10/-0)
Ian Booth (wallyworld)
description: updated
Curtis Hovey (sinzui)
tags: added: ppa
description: updated
Ian Booth (wallyworld)
Changed in launchpad:
status: Triaged → In Progress
assignee: nobody → Ian Booth (wallyworld)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

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

tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

r12110 in db-stable (http://bazaar.launchpad.net/~launchpad-pqm/launchpad/db-stable/revision/12110) is part of this bug's fix.

Changed in launchpad:
status: Fix Committed → In Progress
Ian Booth (wallyworld)
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

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

tags: added: qa-needstesting
removed: qa-ok
Changed in launchpad:
status: In Progress → Fix Committed
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

r16247 in stable (http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/16247) is part of this bug's fix.

Changed in launchpad:
status: Fix Committed → In Progress
Ian Booth (wallyworld)
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

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

tags: added: qa-needstesting
removed: qa-ok
Changed in launchpad:
status: In Progress → Fix Committed
Ian Booth (wallyworld)
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

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

tags: added: qa-needstesting
removed: qa-ok
Ian Booth (wallyworld)
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad:
status: Fix Committed → In Progress
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

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

tags: added: qa-needstesting
removed: qa-ok
Changed in launchpad:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
William Grant (wgrant) wrote :

There's a minor change in behaviour from the previous implementation. If someone uploads a package that they don't maintain, but later in the same archive+series takes over its maintainership, it won't show up on +uploaded-packages. The only record in LPSPRC for that (archive, distroseries, sourcepackagename, creator) will be an SPR where the maintainer and creator are the same, so the uploader_only branch of _releasesQueryFilter will exclude it.

This is only a minor issue and won't affect us much in practice, so we can ignore it. I suspect that any future fix would be to simply remove the created-but-not-maintained implication of the uploader_only flag.

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.