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

« back to all changes in this revision

Viewing changes to provider/ec2/ec2.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:
340
340
        return e.publicStorageUnlocked
341
341
}
342
342
 
343
 
func (e *environ) Bootstrap(cons constraints.Value, possibleTools tools.List, machineID string) error {
344
 
        return common.Bootstrap(e, cons, possibleTools, machineID)
 
343
func (e *environ) Bootstrap(cons constraints.Value, possibleTools tools.List) error {
 
344
        return common.Bootstrap(e, cons, possibleTools)
345
345
}
346
346
 
347
347
func (e *environ) StateInfo() (*state.Info, *api.Info, error) {
556
556
}
557
557
 
558
558
func (e *environ) Destroy() error {
559
 
        logger.Infof("destroying environment %q", e.name)
560
 
        insts, err := e.AllInstances()
561
 
        if err != nil {
562
 
                return fmt.Errorf("cannot get instances: %v", err)
563
 
        }
564
 
        var ids []instance.Id
565
 
        for _, inst := range insts {
566
 
                ids = append(ids, inst.Id())
567
 
        }
568
 
        err = e.terminateInstances(ids)
569
 
        if err != nil {
570
 
                return err
571
 
        }
572
 
 
573
 
        return e.Storage().RemoveAll()
 
559
        return common.Destroy(e)
574
560
}
575
561
 
576
562
func portsToIPPerms(ports []instance.Port) []ec2.IPPerm {