~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): William Reade
  • Date: 2013-10-02 10:38:04 UTC
  • mfrom: (1915.1.3 juju-core)
  • Revision ID: tarmac-20131002103804-zxlnv7ik17unxc3p
[r=fwereade] provider/common: Destroy, and tests

Added a common implementation of Destroy, and tests for Bootstrap which had
somehow never been written. Also drive-by removed the machineId arg from
consideration when bootstrapping, because, WTF... a non-0 bootstrap machine
would break *everything*.

https://codereview.appspot.com/14254043/

Show diffs side-by-side

added added

removed removed

Lines of Context:
50
50
                return err
51
51
        }
52
52
 
53
 
        // The bootstrap instance gets machine id "0".  This is not related to
54
 
        // instance ids.  Juju assigns the machine ID.
55
 
        const machineID = "0"
56
53
        logger.Infof("bootstrapping environment %q", environ.Name())
57
54
        var vers *version.Number
58
55
        if agentVersion, ok := cfg.AgentVersion(); ok {
89
86
        if len(newestTools) == 0 {
90
87
                return fmt.Errorf("No bootstrap tools found")
91
88
        }
92
 
        return environ.Bootstrap(cons, newestTools, machineID)
 
89
        return environ.Bootstrap(cons, newestTools)
93
90
}
94
91
 
95
92
// verifyBootstrapInit does the common initial check before bootstrapping, to