Code review comment for lp://staging/~jameinel/juju-core/hpcloud-1229480

Revision history for this message
John A Meinel (jameinel) wrote :

Reviewers: mp+187485_code.launchpad.net,

Message:
Please take a look.

Description:
fix boilerplate for hpcloud

We were saying AWS_ACCESS_KEY which we don't support and
AWS_SECRET_KEY which we do support, but probably isn't what we would
actually prefer to use for Openstack env variables

https://code.launchpad.net/~jameinel/juju-core/hpcloud-1229480/+merge/187485

(do not edit description out of merge proposal)

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

Affected files (+4, -2 lines):
   A [revision details]
   M provider/openstack/provider.go

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: tarmac-20130917234003-54re4ia2qmlq7xt3
+New revision: <email address hidden>

Index: provider/openstack/provider.go
=== modified file 'provider/openstack/provider.go'
--- provider/openstack/provider.go 2013-09-11 06:27:05 +0000
+++ provider/openstack/provider.go 2013-09-25 10:40:26 +0000
@@ -114,9 +114,9 @@
    # region: <your region>
    # The following are used for keypair authentication
    # auth-mode: keypair
- # Usually set via the env variable AWS_ACCESS_KEY_ID, but can be
specified here
+ # Usually set via the env variable OS_ACCESS_KEY, but can be specified
here
    # access-key: <secret>
- # Usually set via the env variable AWS_SECRET_ACCESS_KEY, but can be
specified here
+ # Usually set via the env variable OS_SECRET_KEY, but can be specified
here
    # secret-key: <secret>

  `[1:]

« Back to merge proposal