Code review comment for lp:~arosales/charms/precise/ubuntu/add-basic-amulet-test

Revision history for this message
Antonio Rosales (arosales) wrote :

Note I am getting the following error:

$ juju test 10-deploy-test.py
juju-test INFO : Starting test run on hp-scale using Juju 1.17.0
Launching instance
 - 3461791
Waiting for address
Attempting to connect to 10.5.31.158:22
Attempting to connect to 15.185.125.174:22
Connection to 15.185.125.174 closed.
2014-01-24 20:05:35 Starting deployment of hp-scale
2014-01-24 20:05:39 Deploying services...
2014-01-24 20:05:41 Deploying service ubuntu-sentry using local:precise/ubuntu-sentry
2014-01-24 20:05:44 Exposing service 'ubuntu-sentry'
2014-01-24 20:05:44 Deploying service relation-sentry using local:precise/relation-sentry
2014-01-24 20:05:48 Exposing service 'relation-sentry'
2014-01-24 20:05:48 Deploying service ubuntu using local:precise/ubuntu
2014-01-24 20:08:12 Config specifies num units for subordinate: ubuntu-sentry
2014-01-24 20:08:12 Adding relations...
2014-01-24 20:08:13 Adding relation ubuntu:juju-info <-> ubuntu-sentry:juju-info
2014-01-24 20:09:20 Deployment complete in 225.05 seconds
Traceback (most recent call last):
  File "tests/10-deploy-test.py", line 50, in <module>
    output, code = d.sentry.unit['ubuntu/0'].run(lsb_release_command)
  File "/usr/lib/python3/dist-packages/amulet/sentry.py", line 98, in run
    r = self.query('/run', data=command)
  File "/usr/lib/python3/dist-packages/amulet/sentry.py", line 47, in query
    return self._fetch(self.config['address'], endpoint, query, data)
  File "/usr/lib/python3/dist-packages/amulet/sentry.py", line 50, in _fetch
    url = "%s/%s?%s" % (address, endpoint, urllib.parse.urlencode(query))
AttributeError: 'module' object has no attribute 'parse'
juju-test.conductor.10-deploy-test.py RESULT : ✘
juju-test INFO : Results: 0 passed, 1 failed, 0 errored
ERROR exit status 1

However, I think this may be a bug that Marco has already fixed in Amulet but has not been released in the package yet.

-thanks,
Antonio

« Back to merge proposal