Merge lp://staging/~abentley/juju-release-tools/boto-compat into lp://staging/juju-release-tools

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 282
Proposed branch: lp://staging/~abentley/juju-release-tools/boto-compat
Merge into: lp://staging/juju-release-tools
Diff against target: 54 lines (+23/-2)
2 files modified
make_aws_image_streams.py (+7/-1)
tests/test_make_aws_image_streams.py (+16/-1)
To merge this branch: bzr merge lp://staging/~abentley/juju-release-tools/boto-compat
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+289529@code.staging.launchpad.net

Commit message

Fix incompatibilities with old boto.

Description of the change

This branch fixes incompatibilities with ci-master's boto.

It supplies credentials to ec2.regions (which is not required on my machine), and it skips two regions that claim authentication failed. (Probably a new authentication method not supported by this boto.)

To post a comment you must log in.
284. By Aaron Bentley

Add comments.

Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you.

I have seen the auth issue in korea before. I am this is a case of libs need to catch up.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches