https://launchpad.net/ubuntu/+ppas broken when recent uploads are to disabled ppas [we think]

Bug #576168 reported by Robert Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
William Grant

Bug Description

When a recent upload to a PPA is to a disabled PPA, https://launchpad.net/ubuntu/+ppas gets broken. It gives the following traceback to devs:
/edge/launchpad-rev-9336/lib/lp/soyuz/browser/../templates/distribution-ppa-list.pt instance at 0xe4783d0>
...
      __traceback_info__: (<SourcePackagePublishingHistory at 0xda4e210>, 'displayname', [])
Unauthorized: (<SourcePackagePublishingHistory at 0xda4e210>, 'displayname', 'launchpad.View')<br />

13:09 < lifeless> wgrant: could it be a masked/rethrown error from the db ?
13:10 < wgrant> The code actually lives in Registry, but that's an accident.
13:11 < wgrant> Oh.
13:11 < wgrant> I guess it's probably because one of the last few uploads is in a disabled PPA.
13:11 < wgrant> spm: Can you confirm that?
13:11 < wgrant> One of the PPAs in the list should show up greyed out.
13:12 < spm> 2 most recent. haha
13:13 < wgrant> So it'll resolve itself in a few minutes, probably.

Related branches

Revision history for this message
Steve McInerney (spm) wrote :

https://edge.launchpad.net/~vperetokin/+archive/unofficial-gnome-shell is the grey'ed out PPA in question.
See attached screenshot via superdooper powers.

William Grant (wgrant)
Changed in soyuz:
status: New → In Progress
assignee: nobody → William Grant (wgrant)
tags: added: oops
Changed in soyuz:
importance: Undecided → High
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
Changed in soyuz:
milestone: none → 10.05
status: In Progress → Fix Committed
tags: added: qa-needstesting
Revision history for this message
Michael Nelson (michael.nelson) wrote :

QA'd by uploading a package to my PPA, then disabling the PPA and accessing:

https://launchpad.net/ubuntu/+ppas

Got someone else to verify that production oopses as expected, while edge is fine (just doesn't display the package).

Thanks William :)

tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in soyuz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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