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

« back to all changes in this revision

Viewing changes to provider/openstack/provider.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:
424
424
        return environs.EmptyStorage
425
425
}
426
426
 
427
 
func (e *environ) Bootstrap(cons constraints.Value, possibleTools tools.List, machineID string) error {
 
427
func (e *environ) Bootstrap(cons constraints.Value, possibleTools tools.List) error {
428
428
        // The client's authentication may have been reset when finding tools if the agent-version
429
429
        // attribute was updated so we need to re-authenticate. This will be a no-op if already authenticated.
430
430
        // An authenticated client is needed for the URL() call below.
432
432
        if err != nil {
433
433
                return err
434
434
        }
435
 
        return common.Bootstrap(e, cons, possibleTools, machineID)
 
435
        return common.Bootstrap(e, cons, possibleTools)
436
436
}
437
437
 
438
438
func (e *environ) StateInfo() (*state.Info, *api.Info, error) {
814
814
}
815
815
 
816
816
func (e *environ) Destroy() error {
817
 
        logger.Infof("destroying environment %q", e.name)
818
 
        insts, err := e.AllInstances()
819
 
        if err != nil {
820
 
                return fmt.Errorf("cannot get instances: %v", err)
821
 
        }
822
 
        var ids []instance.Id
823
 
        for _, inst := range insts {
824
 
                ids = append(ids, inst.Id())
825
 
        }
826
 
        err = e.terminateInstances(ids)
827
 
        if err != nil {
828
 
                return err
829
 
        }
830
 
 
831
 
        return e.Storage().RemoveAll()
 
817
        return common.Destroy(e)
832
818
}
833
819
 
834
820
func (e *environ) globalGroupName() string {