Merge lp://staging/~frankban/juju-quickstart/hp-fixes into lp://staging/juju-quickstart

Proposed by Francesco Banconi
Status: Merged
Merged at revision: 84
Proposed branch: lp://staging/~frankban/juju-quickstart/hp-fixes
Merge into: lp://staging/juju-quickstart
Diff against target: 79 lines (+15/-11)
2 files modified
quickstart/__init__.py (+1/-1)
quickstart/models/envs.py (+14/-10)
To merge this branch: bzr merge lp://staging/~frankban/juju-quickstart/hp-fixes
Reviewer Review Type Date Requested Status
Juju GUI Hackers Pending
Review via email: mp+223445@code.staging.launchpad.net

Description of the change

Align to the latest HP Cloud config options.

https://codereview.appspot.com/105920046/

To post a comment you must log in.
Revision history for this message
Francesco Banconi (frankban) wrote :
Download full text (4.9 KiB)

Reviewers: mp+223445_code.launchpad.net,

Message:
Please take a look.

Description:
Align to the latest HP Cloud config options.

https://code.launchpad.net/~frankban/juju-quickstart/hp-fixes/+merge/223445

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/105920046/

Affected files (+17, -11 lines):
   A [revision details]
   M quickstart/__init__.py
   M quickstart/models/envs.py

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision:
<email address hidden>

Index: quickstart/__init__.py
=== modified file 'quickstart/__init__.py'
--- quickstart/__init__.py 2014-06-12 00:37:46 +0000
+++ quickstart/__init__.py 2014-06-17 16:07:50 +0000
@@ -45,7 +45,7 @@
  Once Juju has been installed, the command can also be run as a juju plugin,
  without the hyphen ("juju quickstart").
  """
-VERSION = (1, 3, 5)
+VERSION = (1, 4, 0)

  def get_version():

Index: quickstart/models/envs.py
=== modified file 'quickstart/models/envs.py'
--- quickstart/models/envs.py 2014-06-16 10:00:00 +0000
+++ quickstart/models/envs.py 2014-06-17 16:07:50 +0000
@@ -419,8 +419,7 @@
          'eu-west-1', 'sa-east-1',
          'us-east-1', 'us-west-1', 'us-west-2',
      )
- hp_regions = (
- 'az-1.region-a.geo-1', 'az-2.region-a.geo-1', 'az-3.region-a.geo-1')
+ hp_regions = ('region-a.geo-1', 'region-b.geo-1')
      azure_locations = (
          'East US', 'West US', 'West Europe', 'North Europe',
          'Southeast Asia', 'East Asia')
@@ -512,7 +511,8 @@
                  help='Specifies whether the use of a floating IP address
is '
                       'required to give the nodes a public IP address. '
                       'Some installations assign public IP addresses by '
- 'default without requiring a floating IP address.'),
+ 'default without requiring a floating IP address. '
+ 'For HP Public Cloud, floating IP must be
activated.'),
              fields.AutoGeneratedStringField(
                  'control-bucket', label='control bucket', required=False,
                  help='the globally unique swift bucket name'),
@@ -526,7 +526,7 @@
                  'tenant-name', label='tenant name', required=True,
                  help='The OpenStack tenant name. For HP Public Cloud, this
is '
                       'listed as the project name on the '
- 'https://account.hpcloud.com/projects page.'),
+ 'https://horizon.hpcloud.com/landing/ page.'),
              fields.SuggestionsStringField(
                  'region', label='region', required=True,
                  help='The OpenStack region to use. '
@@ -538,15 +538,17 @@
                  help='The way Juju authenticates to OpenStack. The
userpass '
                       'authentication requires you to fill in your user
name '
                       'and password. The keypair mode requires access ke...

Read more...

Revision history for this message
Brad Crittenden (bac) wrote :

LGTM. No QA.

https://codereview.appspot.com/105920046/diff/1/quickstart/models/envs.py
File quickstart/models/envs.py (left):

https://codereview.appspot.com/105920046/diff/1/quickstart/models/envs.py#oldcode542
quickstart/models/envs.py:542: 'these information can be retrieved on
the '
wow, glad that's gone!

https://codereview.appspot.com/105920046/

Revision history for this message
Francesco Banconi (frankban) wrote :

*** Submitted:

Align to the latest HP Cloud config options.

R=bac
CC=
https://codereview.appspot.com/105920046

https://codereview.appspot.com/105920046/

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