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

Issue 83530044: provider/joyent: key-file => private-key-path

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 1 month ago by jameinel
Modified:
10 years, 1 month ago
Reviewers:
mp+213818, wallyworld
Visibility:
Public.

Description

provider/joyent: key-file => private-key-path This only loosely depends on the nil-key-file patch (just because otherwise they conflict, and I wanted it to be easy to land.) This is a backwards incompatible change, so I wanted to land it early before we have Joyent in an official release. Doing this change makes the config fit how we do those settings elsewhere (ca-cert-path vs ca-cert, etc). It also avoids using the name "key-file" which is easier for us developers, since keyFile is a term we use WRT MongoDB which means something entirely different. https://code.launchpad.net/~jameinel/juju-core/1.18-private-key-path-1301315/+merge/213818 Requires: https://code.launchpad.net/~jameinel/juju-core/1.18-nil-key-file-1300846/+merge/213805 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 2

Patch Set 2 : provider/joyent: key-file => private-key-path #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -84 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M provider/joyent/config.go View 1 3 chunks +46 lines, -44 lines 0 comments Download
M provider/joyent/config_test.go View 1 5 chunks +28 lines, -28 lines 0 comments Download
M provider/joyent/joyent_test.go View 1 chunk +12 lines, -12 lines 0 comments Download

Messages

Total messages: 3
jameinel
Please take a look.
10 years, 1 month ago (2014-04-02 11:31:13 UTC) #1
wallyworld
LGTM with comment fix https://codereview.appspot.com/83530044/diff/1/provider/joyent/config.go File provider/joyent/config.go (right): https://codereview.appspot.com/83530044/diff/1/provider/joyent/config.go#newcode42 provider/joyent/config.go:42: # You can also "private-key" ...
10 years, 1 month ago (2014-04-02 12:04:58 UTC) #2
jameinel
10 years, 1 month ago (2014-04-02 12:35:17 UTC) #3
Please take a look.

https://codereview.appspot.com/83530044/diff/1/provider/joyent/config.go
File provider/joyent/config.go (right):

https://codereview.appspot.com/83530044/diff/1/provider/joyent/config.go#newc...
provider/joyent/config.go:42: # You can also "private-key" to the *content* of
the private key, instead.
On 2014/04/02 12:04:58, wallyworld wrote:
> Wording seems a little off?

Done.
Sign in to reply to this message.

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