./test --functional errors if EC2 credentials aren't in the global environment

Bug #917228 reported by Graham Binns
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pyjuju
Triaged
Low
Unassigned

Bug Description

When running ./test --functional (the errors also occur if you run ./test without options, but --functional makes it fail quicker), I see a whole mess of errors. This looks to be because my AWS credentials aren't in the global environment. For example:

Traceback (most recent call last):
  File "/home/graham/launchpad/workspace/juju/trunk/juju/ftests/test_ec2_provider.py", line 218, in setUp
    super(EC2StorageTest, self).setUp()
  File "/home/graham/launchpad/workspace/juju/trunk/juju/ftests/test_ec2_provider.py", line 114, in setUp "juju-branch": juju_branch})
  File "/home/graham/launchpad/workspace/juju/trunk/juju/providers/ec2/__init__.py", line 37, in __init__
    s3_uri=config.get("s3-uri", ""))
  File "/usr/lib/python2.7/dist-packages/txaws/service.py", line 102, in __init__
    creds = AWSCredentials(access_key, secret_key)
  File "/usr/lib/python2.7/dist-packages/txaws/credentials.py", line 34, in __init__
    raise ValueError("Could not find %s" % ENV_ACCESS_KEY)
exceptions.ValueError: Could not find AWS_ACCESS_KEY_ID

I've attached the full output from ./test --functional for reference.

Revision history for this message
Graham Binns (gmb) wrote :
Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

I believe that the functional tests actually do require the use of EC2, so it doesn't look like this is an actual issue.

As Clint points out, the message could be improved, though, and the traceback avoided.

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Right, I'd suggest that the EC2 functional tests be skipped entirely unless these variables are set. We should also look at running these as part of the wtf.labix.org tests.

Changed in juju:
status: New → Confirmed
importance: Undecided → Low
Curtis Hovey (sinzui)
Changed in juju:
status: Confirmed → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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