Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4632)

Issue 6003046: Disable constraints on private clouds

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by fwereade
Modified:
12 years ago
Reviewers:
mp+101592
Visibility:
Public.

Description

Disable constraints on private clouds Private clouds (defined as those with an ec2-uri config setting) now expose no constraints at all; everything in the ec2 provider that uses constraints now knows not to assume that any keys are present other than provider-type and ubuntu-series. https://code.launchpad.net/~fwereade/juju/no-private-cloud-constraints/+merge/101592 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Disable constraints on private clouds #

Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -14 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M juju/providers/ec2/__init__.py View 1 chunk +1 line, -2 lines 0 comments Download
M juju/providers/ec2/launch.py View 1 chunk +1 line, -1 line 0 comments Download
M juju/providers/ec2/tests/common.py View 1 chunk +7 lines, -6 lines 0 comments Download
M juju/providers/ec2/tests/test_launch.py View 1 chunk +29 lines, -0 lines 0 comments Download
M juju/providers/ec2/tests/test_provider.py View 1 chunk +1 line, -3 lines 0 comments Download
M juju/providers/ec2/tests/test_utils.py View 3 chunks +22 lines, -1 line 0 comments Download
M juju/providers/ec2/utils.py View 2 chunks +3 lines, -1 line 0 comments Download

Messages

Total messages: 3
fwereade
Please take a look.
12 years ago (2012-04-11 15:52:16 UTC) #1
hazmat
unfortunate but nesc. lgtm.
12 years ago (2012-04-11 16:20:51 UTC) #2
fwereade
12 years ago (2012-04-11 16:23:36 UTC) #3
*** Submitted:

Disable constraints on private clouds

Private clouds (defined as those with an ec2-uri config setting) now expose
no constraints at all; everything in the ec2 provider that uses constraints
now knows not to assume that any keys are present other than provider-type
and ubuntu-series.

R=hazmat
CC=
https://codereview.appspot.com/6003046
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b