~rogpeppe/juju-core/438-local-instance-Addresses

« back to all changes in this revision

Viewing changes to environs/bootstrap/bootstrap.go

  • Committer: Tarmac
  • Author(s): Roger Peppe, John Arbash Meinel
  • Date: 2013-10-02 12:38:13 UTC
  • mfrom: (1717.1.42 375-dummy-prepare-state-id)
  • Revision ID: tarmac-20131002123813-3tidl1lj00d3a29c
[r=rogpeppe] environs/dummy: add state-id

This makes the dummy provider more like the other
providers in that the Environ does not refer
directly to the bootstrapped state, but is looked
up when operations are performed on it.

This is also a test case for adding attributes at Prepare
time - many tests needed changing to correctly
deal with this.

https://codereview.appspot.com/14207046/

Show diffs side-by-side

added added

removed removed

Lines of Context:
93
93
// confirm that the environment isn't already running, and that the storage
94
94
// works.
95
95
func verifyBootstrapInit(env environs.Environ) error {
 
96
        // TODO(rog) this feels like a layering violation - providers
 
97
        // should not necessarily be required to store their bootstrap
 
98
        // state in a file. This verification should probably
 
99
        // be moved into provider and called by the providers themselves.
96
100
        stor := env.Storage()
97
101
        _, err := common.LoadState(stor)
98
102
        if err == nil {