the puller scheduler's handling of broken netstrings is broken

Bug #371467 reported by Michael Hudson-Doyle on 2009-05-04
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Critical
Unassigned

Bug Description

Thanks in part to a Twisted issue (http://twistedmatrix.com/trac/ticket/2851), the puller's handling of malformed netstrings doesn't actually work, as seen in OOPS-1219SMPM3.

It's a bit hard to see what to do about this, apart from fixing Twisted.

One possible cause of this issue are 'mirrorFailed' messages with extremely long (> 100k) explanations.

Jonathan Lange (jml) wrote :

OOPS-1219SMPM3 has the error specific to netstring handling.

OOPS-1219SMPM4 is presumably a later response to the same root error in the worker.

OOPS-1219SMPM01 lends support to Michael's theory of an extremely long mirrorFailed message (warning: OOPS report is very large) and OOPS-1219SMPM09 is from the same "branch".

Changed in launchpad-code:
importance: Undecided → High
status: New → Triaged
tags: added: branch-puller oops
Tim Penhey (thumper) wrote :

Michael, is this still a problem?

Changed in launchpad-code:
status: Triaged → Incomplete
Michael Hudson-Doyle (mwhudson) wrote :

I think so, although it happens so extreeeeemely rarely we should lower the priority.

Changed in launchpad-code:
importance: High → Low
status: Incomplete → Triaged
Changed in launchpad:
importance: Low → Critical
Robert Collins (lifeless) wrote :

FTR last seen - Last seen: Oct. 12, 2010, 2:19 p.m. OOPS-1746SMS382. This is a while ago now, but mwhudson did say very rare. @mwhudson, do you think its > 1 year frequency and still existing, or perhaps fixed by some twisted upgrade?

description: updated
Robert Collins (lifeless) wrote :

Oh, and if jelmer completes his work to make the puller go away, using regular code imports, we can close this as no longer relevant to LP.

Michael Hudson-Doyle (mwhudson) wrote :

I'm pretty sure the bug is still there. Twisted has changed a bit so we could probably fix this by overriding a private method -- but we don't (and it's not clear that this would be a good idea -- changing twisted to make the method public would be better).

Agree that Jelmer is close to making this bug irrelevant.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers