Test error in test_import_bzrsvn

Bug #541526 reported by Steve Kowalik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Jelmer Vernooij

Bug Description

Running a test suite on an EC2 instance resulted in the test_import_bzrsvn test erroring, with the following log around the error:

/var/launchpad/test/bin/py -t ./test_on_merge.py -vv
2010-03-18 16:47:39 WARNING No permissions specified for [u'public.lp_teamparticipation', u'public.lp_personlocation', u'public.bug_update_latest_patch_uploaded(integer)', u'public.lp_person']
xvfb-run -s '-screen 0 1024x768x24' /var/launchpad/test/bin/test -vv

 testSMTPServerIsAvailable (canonical.testing.ftests.test_layers.LayerProcessControllerInvariantsTestCase)No handlers could be found for logger "lazr.smtptest"

Error in test test_import_bzrsvn (lp.codehosting.codeimport.tests.test_workermonitor.TestWorkerMonitorIntegration)
Traceback (most recent call last):
DirtyReactorAggregateError: Reactor was unclean.
Selectables:
<<class 'twisted.internet.tcp.Client'> to ('xmlrpc-private.launchpad.dev', 8087) at c24b410>

in 0.506 seconds.
in 0.509 seconds.

Started ['/tmp/tmpRQtj6H.mozrunner/mozrunner-firefox', '-profile', '/tmp/tmpRQtj6H.mozrunner', 'http://bugs.launchpad.dev:8085/windmill-serv/start.html']
in 1.007 seconds.
in 0.505 seconds.

Related branches

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

This seems reasonably impossible :/

Maybe there's a buglet in twisted's http client where the response is parsed before the connection to the server is fully dropped?

tags: added: code-import spurious-test-failure
Changed in launchpad-code:
status: New → In Progress
importance: Undecided → High
Curtis Hovey (sinzui)
Changed in launchpad-code:
status: In Progress → Triaged
Revision history for this message
Jonathan Lange (jml) wrote :

Yes, that's exactly the bug Michael. It has been fixed in Twisted 10.2 for XML-RPC. When we upgrade, we should re-enable this test.

Revision history for this message
William Grant (wgrant) wrote :

Are we running a sufficient Twisted yet?

Revision history for this message
Jonathan Lange (jml) wrote :

We are.

Jelmer Vernooij (jelmer)
Changed in launchpad:
assignee: nobody → Jelmer Vernooij (jelmer)
Jelmer Vernooij (jelmer)
Changed in launchpad:
status: Triaged → In Progress
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Jelmer Vernooij (jelmer)
tags: added: qa-untestable
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.

Other bug subscribers

Remote bug watches

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