Code review comment for lp:~thumper/juju-core/fix-local-bootstrap

Revision history for this message
Tim Penhey (thumper) wrote :

Reviewers: mp+191522_code.launchpad.net,

Message:
Please take a look.

Description:
Fix local provider bootstrap.

The bootstrap storage was not set up for the local
provider.

https://code.launchpad.net/~thumper/juju-core/fix-local-bootstrap/+merge/191522

(do not edit description out of merge proposal)

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

Affected files (+7, -0 lines):
   A [revision details]
   M provider/local/environ.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-20131010031125-8lgphpzdl93berr7
+New revision: <email address hidden>

Index: provider/local/environ.go
=== modified file 'provider/local/environ.go'
--- provider/local/environ.go 2013-10-04 08:45:50 +0000
+++ provider/local/environ.go 2013-10-16 21:28:54 +0000
@@ -335,6 +335,11 @@
   return httpstorage.Client(env.config.sharedStorageAddr())
  }

+// Implements environs.BootstrapStorager.
+func (env *localEnviron) EnableBootstrapStorage() error {
+ return env.setupLocalStorage()
+}
+
  // Destroy is specified in the Environ interface.
  func (env *localEnviron) Destroy() error {
   if !env.config.runningAsRoot {

« Back to merge proposal