Merge lp:~allenap/maas/rpc-test-failure--bug-1288001 into lp:~maas-committers/maas/trunk

Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: no longer in the source branch.
Merged at revision: 2079
Proposed branch: lp:~allenap/maas/rpc-test-failure--bug-1288001
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 38 lines (+3/-6)
2 files modified
src/maasserver/utils/tests/test_async.py (+3/-3)
src/maasserver/views/tests/test_rpc.py (+0/-3)
To merge this branch: bzr merge lp:~allenap/maas/rpc-test-failure--bug-1288001
Reviewer Review Type Date Requested Status
Gavin Panella (community) Approve
Review via email: mp+209432@code.launchpad.net

Commit message

In the test suite, don't start the event-loop unless as a managed resource.

In other words, it can be started, but a clean-up needs to be arranged to stop it again, or its presence will interfere with other tests.

To post a comment you must log in.
Revision history for this message
Gavin Panella (allenap) :
review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :
Download full text (15.3 KiB)

The attempt to merge lp:~allenap/maas/rpc-test-failure--bug-1288001 into lp:maas failed. Below is the output from the failed tests.

Ign http://security.ubuntu.com trusty-security InRelease
Hit http://security.ubuntu.com trusty-security Release.gpg
Hit http://security.ubuntu.com trusty-security Release
Ign http://nova.clouds.archive.ubuntu.com trusty InRelease
Ign http://nova.clouds.archive.ubuntu.com trusty-updates InRelease
Get:1 http://nova.clouds.archive.ubuntu.com trusty Release.gpg [933 B]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates Release.gpg
Get:2 http://nova.clouds.archive.ubuntu.com trusty Release [58.5 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates Release
Hit http://security.ubuntu.com trusty-security/main Sources
Hit http://security.ubuntu.com trusty-security/universe Sources
Hit http://security.ubuntu.com trusty-security/main amd64 Packages
Hit http://security.ubuntu.com trusty-security/universe amd64 Packages
Hit http://security.ubuntu.com trusty-security/main Translation-en
Hit http://security.ubuntu.com trusty-security/universe Translation-en
Get:3 http://nova.clouds.archive.ubuntu.com trusty/main Sources [1,062 kB]
Ign http://security.ubuntu.com trusty-security/main Translation-en_US
Ign http://security.ubuntu.com trusty-security/universe Translation-en_US
Get:4 http://nova.clouds.archive.ubuntu.com trusty/universe Sources [6,408 kB]
Get:5 http://nova.clouds.archive.ubuntu.com trusty/main amd64 Packages [1,346 kB]
Get:6 http://nova.clouds.archive.ubuntu.com trusty/universe amd64 Packages [5,877 kB]
Get:7 http://nova.clouds.archive.ubuntu.com trusty/main Translation-en [763 kB]
Get:8 http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en [4,051 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main Sources
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe Sources
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en
Ign http://nova.clouds.archive.ubuntu.com trusty/main Translation-en_US
Ign http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en_US
Ign http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en_US
Ign http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en_US
Fetched 19.6 MB in 7s (2,579 kB/s)
Reading package lists...
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
     --no-install-recommends install apache2 bind9 bind9utils build-essential curl daemontools distro-info dnsutils firefox freeipmi-tools ipython isc-dhcp-common libjs-raphael libjs-yui3-full libjs-yui3-min libpq-dev make postgresql python-amqplib python-bzrlib python-celery python-convoy python-cssselect python-curtin python-dev python-distro-info python-django python-django-piston python-django-south python-djorm-ext-pgarray python-docutils python-formencode python-httplib2 python-jinja2 python-jsonschema python-lockfile python-lxml python-netaddr python-netifaces python-oa...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/maasserver/utils/tests/test_async.py'
2--- src/maasserver/utils/tests/test_async.py 2014-03-05 00:40:03 +0000
3+++ src/maasserver/utils/tests/test_async.py 2014-03-05 12:05:03 +0000
4@@ -17,7 +17,7 @@
5 from functools import partial
6 from time import time
7
8-from maasserver import eventloop
9+import crochet
10 from maasserver.utils import async
11 from maastesting.testcase import MAASTestCase
12 from mock import sentinel
13@@ -33,8 +33,8 @@
14 from twisted.internet.task import deferLater
15 from twisted.python.failure import Failure
16
17-
18-eventloop.start()
19+# These tests need a running reactor.
20+crochet.setup()
21
22
23 class TestGather(MAASTestCase):
24
25=== modified file 'src/maasserver/views/tests/test_rpc.py'
26--- src/maasserver/views/tests/test_rpc.py 2014-03-05 01:26:35 +0000
27+++ src/maasserver/views/tests/test_rpc.py 2014-03-05 12:05:03 +0000
28@@ -41,10 +41,7 @@
29
30 class RPCViewTest(MAASServerTestCase):
31
32- # XXX 2014-03-05 gmb bug=1288001:
33- # This fails due to an apparent isolation problem.
34 def test_rpc_info(self):
35- self.skip("Disabled due to a test isolation problem.")
36 response = self.client.get(reverse('rpc-info'))
37 self.assertEqual("application/json", response["Content-Type"])
38 info = json.loads(response.content)