Merge into trunk : useful-noconnectionerrors : Code : MAAS

Status:	Merged
Approved by:	Graham Binns on 2014-09-11
Approved revision:	no longer in the source branch.
Merged at revision:	2962
Proposed branch:	lp:~gmb/maas/useful-noconnectionerrors
Merge into:	lp:~maas-committers/maas/trunk
Diff against target:	64 lines (+10/-2) 3 files modified src/maasserver/models/tests/test_nodegroup.py (+4/-0) src/maasserver/rpc/__init__.py (+3/-1) src/maasserver/rpc/regionservice.py (+3/-1)
To merge this branch:	bzr merge lp:~gmb/maas/useful-noconnectionerrors
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Graham Binns (community)		Approve on 2014-09-11
Raphaël Badin (community)	2014-09-11	Approve on 2014-09-11
Review via email: mp+234319@code.launchpad.net

Commit message

Make the NoConnectionsAvailable errors raised by NodeGroup methods return a useful error message for the sake of reporting things back to API clients.

Revision history for this message

Raphaël Badin (rvb) wrote on 2014-09-11:

#

Wouldn't it be enough to only change src/maasserver/rpc/__init__.py:getClientFor and src/maasserver/rpc/regionservice.py:getClientFor ?

review: Needs Information

Revision history for this message

Graham Binns (gmb) wrote on 2014-09-11:

#

Err… yes. And I saw that. And then I got sidetracked and forgot about it :).

Revision history for this message

Raphaël Badin (rvb) wrote on 2014-09-11:

#

Looks good, thanks.

review: Approve

Revision history for this message

MAAS Lander (maas-lander) wrote on 2014-09-11:

#

Download full text (21.5 KiB)

The attempt to merge lp:~gmb/maas/useful-noconnectionerrors into lp:maas failed. Below is the output from the failed tests.

Ign http://security.ubuntu.com trusty-security InRelease
Get:1 http://security.ubuntu.com trusty-security Release.gpg [933 B]
Get:2 http://security.ubuntu.com trusty-security Release [59.7 kB]
Ign http://nova.clouds.archive.ubuntu.com trusty InRelease
Ign http://nova.clouds.archive.ubuntu.com trusty-updates InRelease
Hit http://nova.clouds.archive.ubuntu.com trusty Release.gpg
Get:3 http://nova.clouds.archive.ubuntu.com trusty-updates Release.gpg [933 B]
Hit http://nova.clouds.archive.ubuntu.com trusty Release
Get:4 http://nova.clouds.archive.ubuntu.com trusty-updates Release [59.7 kB]
Get:5 http://security.ubuntu.com trusty-security/main Sources [43.9 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/main Sources
Get:6 http://security.ubuntu.com trusty-security/universe Sources [10.8 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Sources
Get:7 http://security.ubuntu.com trusty-security/main amd64 Packages [138 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/main amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/universe amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en
Get:8 http://security.ubuntu.com trusty-security/universe amd64 Packages [47.2 kB]
Hit http://security.ubuntu.com trusty-security/main Translation-en
Hit http://security.ubuntu.com trusty-security/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
Get:9 http://nova.clouds.archive.ubuntu.com trusty-updates/main Sources [118 kB]
Get:10 http://nova.clouds.archive.ubuntu.com trusty-updates/universe Sources [83.7 kB]
Get:11 http://nova.clouds.archive.ubuntu.com trusty-updates/main amd64 Packages [317 kB]
Get:12 http://nova.clouds.archive.ubuntu.com trusty-updates/universe amd64 Packages [201 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en
Fetched 1,080 kB in 0s (1,886 kB/s)
Reading package lists...
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
--no-install-recommends install apache2 authbind bind9 bind9utils build-essential bzr-builddeb curl daemontools debhelper dh-apport distro-info dnsutils firefox freeipmi-tools ipython isc-dhcp-common libjs-raphael libjs-yui3-full libjs-yui3-min libpq-dev make pep8 postgresql pyflakes python-amqplib python-bzrlib python-celery python-convoy python-crochet python-cssselect python-curtin python-dev python-distro-info python-django python-django-piston python-django-south python-djorm-ext-pgarray python-docutils python-extras python-fixtures python-flake8 python-formencode python-hivex python-httplib2 python-jinja2 python-jsonschema python-lockfile python-lxml python-mimeparse python-mock python-netaddr python-netifaces python-nose python-oauth python-oops python-oops-amqp python-oops-datedir-repo python-oops-twisted python-oops-wsgi python-ope...

The attempt to merge lp:~gmb/maas/useful-noconnectionerrors into lp:maas failed. Below is the output from the failed tests.

Ign http://security.ubuntu.com trusty-security InRelease
Get:1 http://security.ubuntu.com trusty-security Release.gpg [933 B]
Get:2 http://security.ubuntu.com trusty-security Release [59.7 kB]
Ign http://nova.clouds.archive.ubuntu.com trusty InRelease
Ign http://nova.clouds.archive.ubuntu.com trusty-updates InRelease
Hit http://nova.clouds.archive.ubuntu.com trusty Release.gpg
Get:3 http://nova.clouds.archive.ubuntu.com trusty-updates Release.gpg [933 B]
Hit http://nova.clouds.archive.ubuntu.com trusty Release
Get:4 http://nova.clouds.archive.ubuntu.com trusty-updates Release [59.7 kB]
Get:5 http://security.ubuntu.com trusty-security/main Sources [43.9 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/main Sources
Get:6 http://security.ubuntu.com trusty-security/universe Sources [10.8 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Sources
Get:7 http://security.ubuntu.com trusty-security/main amd64 Packages [138 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty/main amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/universe amd64 Packages
Hit http://nova.clouds.archive.ubuntu.com trusty/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty/universe Translation-en
Get:8 http://security.ubuntu.com trusty-security/universe amd64 Packages [47.2 kB]
Hit http://security.ubuntu.com trusty-security/main Translation-en
Hit http://security.ubuntu.com trusty-security/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
Get:9 http://nova.clouds.archive.ubuntu.com trusty-updates/main Sources [118 kB]
Get:10 http://nova.clouds.archive.ubuntu.com trusty-updates/universe Sources [83.7 kB]
Get:11 http://nova.clouds.archive.ubuntu.com trusty-updates/main amd64 Packages [317 kB]
Get:12 http://nova.clouds.archive.ubuntu.com trusty-updates/universe amd64 Packages [201 kB]
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/main Translation-en
Hit http://nova.clouds.archive.ubuntu.com trusty-updates/universe Translation-en
Fetched 1,080 kB in 0s (1,886 kB/s)
Reading package lists...
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
	    --no-install-recommends install apache2 authbind bind9 bind9utils build-essential bzr-builddeb curl daemontools debhelper dh-apport distro-info dnsutils firefox freeipmi-tools ipython isc-dhcp-common libjs-raphael libjs-yui3-full libjs-yui3-min libpq-dev make pep8 postgresql pyflakes python-amqplib python-bzrlib python-celery python-convoy python-crochet python-cssselect python-curtin python-dev python-distro-info python-django python-django-piston python-django-south python-djorm-ext-pgarray python-docutils python-extras python-fixtures python-flake8 python-formencode python-hivex python-httplib2 python-jinja2 python-jsonschema python-lockfile python-lxml python-mimeparse python-mock python-netaddr python-netifaces python-nose python-oauth python-oops python-oops-amqp python-oops-datedir-repo python-oops-twisted python-oops-wsgi python-openssl python-paramiko python-pexpect python-pip python-pocket-lint python-psycopg2 python-pyinotify python-seamicroclient python-simplejson python-simplestreams python-sphinx python-subunit python-tempita python-testresources python-testscenarios python-testtools python-twisted python-txamqp python-txlongpoll python-txtftp python-tz python-virtualenv python-yaml python-zope.interface rabbitmq-server syslinux-common tgt ubuntu-cloudimage-keyring wget xvfb
Reading package lists...
Building dependency tree...
Reading state information...
authbind is already the newest version.
bind9 is already the newest version.
bind9utils is already the newest version.
build-essential is already the newest version.
curl is already the newest version.
debhelper is already the newest version.
distro-info is already the newest version.
dnsutils is already the newest version.
isc-dhcp-common is already the newest version.
libjs-raphael is already the newest version.
libjs-yui3-full is already the newest version.
libjs-yui3-min is already the newest version.
make is already the newest version.
pep8 is already the newest version.
postgresql is already the newest version.
pyflakes is already the newest version.
python-amqplib is already the newest version.
python-celery is already the newest version.
python-convoy is already the newest version.
python-crochet is already the newest version.
python-curtin is already the newest version.
python-dev is already the newest version.
python-distro-info is already the newest version.
python-django-piston is already the newest version.
python-django-south is already the newest version.
python-djorm-ext-pgarray is already the newest version.
python-docutils is already the newest version.
python-extras is already the newest version.
python-fixtures is already the newest version.
python-formencode is already the newest version.
python-httplib2 is already the newest version.
python-jinja2 is already the newest version.
python-jsonschema is already the newest version.
python-lockfile is already the newest version.
python-mimeparse is already the newest version.
python-mock is already the newest version.
python-netaddr is already the newest version.
python-netifaces is already the newest version.
python-nose is already the newest version.
python-oauth is already the newest version.
python-oops is already the newest version.
python-oops-amqp is already the newest version.
python-oops-datedir-repo is already the newest version.
python-oops-twisted is already the newest version.
python-oops-wsgi is already the newest version.
python-openssl is already the newest version.
python-paramiko is already the newest version.
python-psycopg2 is already the newest version.
python-pyinotify is already the newest version.
python-simplejson is already the newest version.
python-subunit is already the newest version.
python-tempita is already the newest version.
python-testresources is already the newest version.
python-testscenarios is already the newest version.
python-testtools is already the newest version.
python-twisted is already the newest version.
python-txamqp is already the newest version.
python-txlongpoll is already the newest version.
python-txtftp is already the newest version.
python-tz is already the newest version.
python-yaml is already the newest version.
python-zope.interface is already the newest version.
rabbitmq-server is already the newest version.
syslinux-common is already the newest version.
tgt is already the newest version.
ubuntu-cloudimage-keyring is already the newest version.
wget is already the newest version.
bzr-builddeb is already the newest version.
daemontools is already the newest version.
ipython is already the newest version.
python-cssselect is already the newest version.
python-flake8 is already the newest version.
python-hivex is already the newest version.
python-pip is already the newest version.
python-pocket-lint is already the newest version.
python-virtualenv is already the newest version.
apache2 is already the newest version.
dh-apport is already the newest version.
firefox is already the newest version.
freeipmi-tools is already the newest version.
libpq-dev is already the newest version.
python-bzrlib is already the newest version.
python-django is already the newest version.
python-lxml is already the newest version.
python-pexpect is already the newest version.
python-seamicroclient is already the newest version.
python-simplestreams is already the newest version.
python-sphinx is already the newest version.
xvfb is already the newest version.
The following package was automatically installed and is no longer required:
  librabbitmq1
Use 'apt-get autoremove' to remove it.
0 upgraded, 0 newly installed, 0 to remove and 16 not upgraded.
sudo DEBIAN_FRONTEND=noninteractive apt-get -y \
	    purge python-librabbitmq
Reading package lists...
Building dependency tree...
Reading state information...
Package 'python-librabbitmq' is not installed, so not removed
The following package was automatically installed and is no longer required:
  librabbitmq1
Use 'apt-get autoremove' to remove it.
0 upgraded, 0 newly installed, 0 to remove and 16 not upgraded.
make -C acceptance clean
make[1]: Entering directory `/tmp/tarmac/branch.BDnXu0/acceptance'
rm -f -r source build *.build *.changes *.deb
make[1]: Leaving directory `/tmp/tarmac/branch.BDnXu0/acceptance'
find . -type f -name '*.py[co]' -print0 | xargs -r0 rm -f
find . -type f -name '*~' -print0 | xargs -r0 rm -f
find . -type f -name dropin.cache -print0 | xargs -r0 rm -f
rm -f -r media/demo/* media/development
rm -f src/maasserver/static/js/enums.js
rm -f *.log
rm -f docs/api.rst
rm -f -r docs/_autosummary docs/_build
rm -f -r man/.doctrees
rm -f -r bin include lib local
rm -f -r eggs develop-eggs
rm -f -r build dist logs/* parts
rm -f tags TAGS .installed.cfg
rm -f -r *.egg *.egg-info src/*.egg-info
rm -f -r run/* services/*/supervise
virtualenv --python=python2.7 --system-site-packages /tmp/tarmac/branch.BDnXu0
New python executable in /tmp/tarmac/branch.BDnXu0/bin/python2.7
Also creating executable in /tmp/tarmac/branch.BDnXu0/bin/python
Installing setuptools, pip...done.
Running virtualenv with interpreter /usr/bin/python2.7
bin/python -m pip --quiet install --ignore-installed \
	    --no-dependencies bootstrap/zc.buildout-1.5.2.tar.gz
rm -f -f README.txt  # zc.buildout installs an annoying README.txt.
bin/buildout install database
Creating directory '/tmp/tarmac/branch.BDnXu0/parts'.
Creating directory '/tmp/tarmac/branch.BDnXu0/develop-eggs'.
Installing database.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/database'.
bin/buildout install repl
Installing repl.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/ipy'.
Generated interpreter '/tmp/tarmac/branch.BDnXu0/bin/py'.
bin/py -m src/maasserver/utils/jsenums src/maasserver/enum.py src/metadataserver/enum.py > src/maasserver/static/js/enums.js
bin/buildout install maas
Installing maas.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/celeryd.region'.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/maas-region-admin'.
bin/buildout install maas-test
Installing maas-test.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/test.maas'.
bin/buildout install maascli
Installing maascli.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/maas'.
bin/buildout install maascli-test
Installing maascli-test.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/test.maascli'.
bin/buildout install maastesting-test
Installing maastesting-test.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/test.maastesting'.
bin/buildout install pserv
Installing pserv.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/celeryd.cluster'.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/maas-provision'.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/twistd.pserv'.
bin/buildout install pserv-test
Installing pserv-test.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/test.pserv'.
bin/buildout install config-test
Installing config-test.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/test.config'.
bin/buildout install maas-probe-dhcp
Installing maas-probe-dhcp.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/maas-probe-dhcp'.
bin/buildout install txlongpoll
Installing txlongpoll.
Generated script '/tmp/tarmac/branch.BDnXu0/bin/twistd.txlongpoll'.
make -C acceptance clean
make[1]: Entering directory `/tmp/tarmac/branch.BDnXu0/acceptance'
rm -f -r source build *.build *.changes *.deb
make[1]: Leaving directory `/tmp/tarmac/branch.BDnXu0/acceptance'
find . -type f -name '*.py[co]' -print0 | xargs -r0 rm -f
find . -type f -name '*~' -print0 | xargs -r0 rm -f
find . -type f -name dropin.cache -print0 | xargs -r0 rm -f
rm -f -r media/demo/* media/development
rm -f src/maasserver/static/js/enums.js
rm -f *.log
rm -f docs/api.rst
rm -f -r docs/_autosummary docs/_build
rm -f -r man/.doctrees
bin/py -m src/maasserver/utils/jsenums src/maasserver/enum.py src/metadataserver/enum.py > src/maasserver/static/js/enums.js
bin/buildout install maas
Updating maas.
bin/buildout install maas-test
Updating maas-test.
bin/buildout install maas
Updating maas.
echo bin/test.config bin/test.maas bin/test.maascli bin/test.maastesting bin/test.pserv | xargs -n1 env
nosetests --exclude=provisioningserver --exclude=maastesting --exclude=maascli --verbosity=1
Creating test database for alias 'default'...
Destroying test database for alias 'default'...

..............................................
----------------------------------------------------------------------
Ran 46 tests in 0.470s

OK
..S...............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................F..F...F..F..............................................................................................................................................................................................................................................................................................................................................................................................................S...............................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
======================================================================
FAIL: maasserver.models.tests.test_nodegroup.TestNodeGroup.test_add_seamicro15k_raises_if_no_connection_to_cluster
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.BDnXu0/src/maasserver/models/tests/test_nodegroup.py", line 544, in test_add_seamicro15k_raises_if_no_connection_to_cluster
    "available." % nodegroup.uuid, unicode(exception))
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 321, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: !=:
reference = u'Unable to connect to cluster 26968f24-39ca-11e4-91e3-fa163ee59350; no connections available.'
actual    = u''

======================================================================
FAIL: maasserver.models.tests.test_nodegroup.TestNodeGroup.test_add_virsh_raises_if_no_connection_to_cluster
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.BDnXu0/src/maasserver/models/tests/test_nodegroup.py", line 487, in test_add_virsh_raises_if_no_connection_to_cluster
    "available." % nodegroup.uuid, unicode(exception))
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 321, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: !=:
reference = u'Unable to connect to cluster 26a8e3c2-39ca-11e4-91e3-fa163ee59350; no connections available.'
actual    = u''

======================================================================
FAIL: maasserver.models.tests.test_nodegroup.TestNodeGroup.test_enlist_nodes_from_mscm_raises_if_no_connection_to_cluster
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.BDnXu0/src/maasserver/models/tests/test_nodegroup.py", line 597, in test_enlist_nodes_from_mscm_raises_if_no_connection_to_cluster
    "available." % nodegroup.uuid, unicode(exception))
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 321, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: !=:
reference = u'Unable to connect to cluster 26c5bfce-39ca-11e4-91e3-fa163ee59350; no connections available.'
actual    = u''

======================================================================
FAIL: maasserver.models.tests.test_nodegroup.TestNodeGroup.test_enlist_nodes_from_ucsm_raises_if_no_connection_to_cluster
----------------------------------------------------------------------
_StringException: Traceback (most recent call last):
  File "/tmp/tarmac/branch.BDnXu0/src/maasserver/models/tests/test_nodegroup.py", line 650, in test_enlist_nodes_from_ucsm_raises_if_no_connection_to_cluster
    "available." % nodegroup.uuid, unicode(exception))
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 321, in assertEqual
    self.assertThat(observed, matcher, message)
  File "/usr/lib/python2.7/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
MismatchError: !=:
reference = u'Unable to connect to cluster 26d7f6d0-39ca-11e4-91e3-fa163ee59350; no connections available.'
actual    = u''

----------------------------------------------------------------------
Ran 3339 tests in 319.449s

FAILED (SKIP=2, failures=4)
...............................................................................................................
----------------------------------------------------------------------
Ran 111 tests in 1.123s

OK
..............................................................S........................................................................................
----------------------------------------------------------------------
Ran 151 tests in 28.041s

OK (SKIP=1)
...................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
----------------------------------------------------------------------
Ran 1491 tests in 17.203s

OK
make: *** [test] Error 123

Revision history for this message

Graham Binns (gmb) on 2014-09-11:

#

review: Approve

1	=== modified file 'src/maasserver/models/tests/test_nodegroup.py'
2	--- src/maasserver/models/tests/test_nodegroup.py 2014-09-11 13:02:20 +0000
3	+++ src/maasserver/models/tests/test_nodegroup.py 2014-09-11 16:06:39 +0000
4	@@ -478,6 +478,7 @@
5	nodegroup = factory.make_NodeGroup()
6	poweraddr = factory.make_name('poweraddr')
7	password = factory.make_name('password')
8	+
9	self.assertRaises(
10	NoConnectionsAvailable, nodegroup.add_virsh, poweraddr,
11	password)
12	@@ -531,6 +532,7 @@
13	username = factory.make_name('user')
14	password = factory.make_name('password')
15	power_control = factory.make_name('power_control')
16	+
17	self.assertRaises(
18	NoConnectionsAvailable, nodegroup.add_seamicro15k,
19	mac, username, password, power_control)
20	@@ -580,6 +582,7 @@
21	host = factory.make_name('host')
22	username = factory.make_name('user')
23	password = factory.make_name('password')
24	+
25	self.assertRaises(
26	NoConnectionsAvailable, nodegroup.enlist_nodes_from_mscm,
27	host, username, password)
28	@@ -629,6 +632,7 @@
29	url = factory.make_url()
30	username = factory.make_name('user')
31	password = factory.make_name('password')
32	+
33	self.assertRaises(
34	NoConnectionsAvailable, nodegroup.enlist_nodes_from_ucsm,
35	url, username, password)
36
37	=== modified file 'src/maasserver/rpc/__init__.py'
38	--- src/maasserver/rpc/__init__.py 2014-09-01 22:25:21 +0000
39	+++ src/maasserver/rpc/__init__.py 2014-09-11 16:06:39 +0000
40	@@ -39,7 +39,9 @@
41	try:
42	service = eventloop.services.getServiceNamed("rpc")
43	except KeyError:
44	- raise exceptions.NoConnectionsAvailable()
45	+ raise exceptions.NoConnectionsAvailable(
46	+ "Unable to connect to cluster %s; no connections available." %
47	+ uuid)
48	else:
49	return service.getClientFor(uuid, timeout=timeout)
50
51
52	=== modified file 'src/maasserver/rpc/regionservice.py'
53	--- src/maasserver/rpc/regionservice.py 2014-09-05 08:10:06 +0000
54	+++ src/maasserver/rpc/regionservice.py 2014-09-11 16:06:39 +0000
55	@@ -463,7 +463,9 @@
56
57	def cancelled(failure):
58	failure.trap(CancelledError)
59	- raise exceptions.NoConnectionsAvailable()
60	+ raise exceptions.NoConnectionsAvailable(
61	+ "Unable to connect to cluster %s; no connections available." %
62	+ uuid)
63
64	return d.addCallbacks(common.Client, cancelled)
65

MAAS

Merge lp:~gmb/maas/useful-noconnectionerrors into lp:~maas-committers/maas/trunk

Commit message

Description of the change

Preview Diff

Subscribers