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

« back to all changes in this revision

Viewing changes to provider/azure/environ.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:
245
245
}
246
246
 
247
247
// Bootstrap is specified in the Environ interface.
248
 
func (env *azureEnviron) Bootstrap(cons constraints.Value, possibleTools tools.List, machineID string) (err error) {
 
248
func (env *azureEnviron) Bootstrap(cons constraints.Value, possibleTools tools.List) (err error) {
249
249
        // The creation of the affinity group and the virtual network is specific to the Azure provider.
250
250
        err = env.createAffinityGroup()
251
251
        if err != nil {
267
267
                        env.deleteVirtualNetwork()
268
268
                }
269
269
        }()
270
 
        err = common.Bootstrap(env, cons, possibleTools, machineID)
 
270
        err = common.Bootstrap(env, cons, possibleTools)
271
271
        return err
272
272
}
273
273