← Back to branch summary

~rogpeppe/juju-core/azure

« back to all changes in this revision

Viewing changes to environs/azure/environ.go

[r=jtv] Remove unneeded internalStartInstance parameter.

This matters because it tightens up the signature of a function that is doing
the advance reconnaissance for an ongoing refactoring in the StartInstance()
and Bootstrap() methods.

Show diffs side-by-side

added added

removed removed

Lines of Context:
environs/azure/environ.go
133
133
        if err != nil {
134
134
                return nil, err
135
135
        }
136
 
        inst, err := env.internalStartInstance(machineID, cons, possibleTools, mcfg)
 
136
        inst, err := env.internalStartInstance(cons, possibleTools, mcfg)
137
137
        if err != nil {
138
138
                return nil, fmt.Errorf("cannot start bootstrap instance: %v", err)
139
139
        }
294
294
// internalStartInstance does the provider-specific work of starting an
295
295
// instance.  The code in StartInstance is actually largely agnostic across
296
296
// the EC2/OpenStack/MAAS/Azure providers.
297
 
// TODO(bug 1199847): Some of this work can be shared between providers.
298
297
// The instance will be set up for the same series for which you pass tools.
299
298
// All tools in possibleTools must be for the same series.
300
 
func (env *azureEnviron) internalStartInstance(machineID string, cons constraints.Value, possibleTools tools.List, mcfg *cloudinit.MachineConfig) (_ instance.Instance, err error) {
 
299
// TODO(bug 1199847): Some of this work can be shared between providers.
 
300
func (env *azureEnviron) internalStartInstance(cons constraints.Value, possibleTools tools.List, mcfg *cloudinit.MachineConfig) (_ instance.Instance, err error) {
301
301
        // Declaring "err" in the function signature so that we can "defer"
302
302
        // any cleanup that needs to run during error returns.
303
303
 
514
514
        }
515
515
        mcfg := env.makeMachineConfig(machineID, machineNonce, stateInfo, apiInfo)
516
516
        // TODO(bug 1193998) - return instance hardware characteristics as well.
517
 
        inst, err := env.internalStartInstance(machineID, cons, possibleTools, mcfg)
 
517
        inst, err := env.internalStartInstance(cons, possibleTools, mcfg)
518
518
        return inst, nil, err
519
519
}
520
520