Build status lists buildds twice after an automatic refresh

Bug #330451 reported by Loïc Minier
18
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Michael Nelson

Bug Description

Hi,

I had a P3A page open and after an automatic refresh it showed active builds twice.

Attaching screenshot.

Bye

Tags: lp-soyuz ppa
Revision history for this message
Loïc Minier (lool) wrote :
Changed in soyuz:
assignee: nobody → michael.nelson
importance: Undecided → High
milestone: none → 2.2.2
status: New → Triaged
Revision history for this message
Michael Nelson (michael.nelson) wrote :

Hi Loïc,

AFAICS, this isn't a presentation bug, but rather I think it's a bug in our logic that collects the builds for a source package. Is it the case that when you saw the attached screenshot, the one source package was being built on multiple DistroSerieses at once?

I've been through the code that generates the list, and we are unfortunately grabbing all builds for a particular source, rather than just the builds for that DistroSeries. So I'm guessing that that's what is going on in the screenshot.

BTW: I've fixed the small presentation bug that meant there were no spaces between each build.

Revision history for this message
Loïc Minier (lool) wrote :

No, it was a build in a jaunty P3A and only there; it was the only active build.

Revision history for this message
Loïc Minier (lool) wrote :

Hmm it might have been than an older version of the same source in the same dist was still building (as I reuploaded a newer version before a build completed), this happened at least once, but I rather remember taking the screenshot only when the newest source was building.

Changed in soyuz:
milestone: 2.2.2 → 2.2.3
Changed in soyuz:
milestone: 2.2.3 → 2.2.4
Changed in soyuz:
status: Triaged → In Progress
Revision history for this message
Michael Nelson (michael.nelson) wrote :

Fixed with r8186.

Seems to be related to a YUI bug (http://yuilibrary.com/projects/yui3/ticket/2526023). Our code that was iterating over a nodelist to delete the old links wasn't getting all the nodes during the iteration. Fixed by instead clearing the td element completely before adding the new links.

Changed in soyuz:
status: In Progress → Fix Committed
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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