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

« back to all changes in this revision

Viewing changes to provider/maas/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:
77
77
}
78
78
 
79
79
// Bootstrap is specified in the Environ interface.
80
 
func (env *maasEnviron) Bootstrap(cons constraints.Value, possibleTools tools.List, machineID string) error {
81
 
        return common.Bootstrap(env, cons, possibleTools, machineID)
 
80
func (env *maasEnviron) Bootstrap(cons constraints.Value, possibleTools tools.List) error {
 
81
        return common.Bootstrap(env, cons, possibleTools)
82
82
}
83
83
 
84
84
// StateInfo is specified in the Environ interface.
387
387
}
388
388
 
389
389
func (environ *maasEnviron) Destroy() error {
390
 
        logger.Debugf("destroying environment %q", environ.name)
391
 
        insts, err := environ.AllInstances()
392
 
        if err != nil {
393
 
                return fmt.Errorf("cannot get instances: %v", err)
394
 
        }
395
 
        err = environ.StopInstances(insts)
396
 
        if err != nil {
397
 
                return err
398
 
        }
399
 
 
400
 
        return environ.Storage().RemoveAll()
 
390
        return common.Destroy(environ)
401
391
}
402
392
 
403
393
// MAAS does not do firewalling so these port methods do nothing.