← Back to branch summary

~rogpeppe/juju-core/azure

« back to all changes in this revision

Viewing changes to environs/maas/environ.go

  • Committer: Tarmac
  • Author(s): jtv at canonical
  • Date: 2013-07-17 12:10:27 UTC
  • mfrom: (1473.1.1 maas-loggo)
  • Revision ID: tarmac-20130717121027-xwqlhurg8sgq90gk
[r=jtv] Convert MAAS provider to loggo.

We've been told this was desired, so we might as well do it. Why keep the
obsolete approach around to set a bad example?

Show diffs side-by-side

added added

removed removed

Lines of Context:
environs/maas/environ.go
11
11
        "time"
12
12
 
13
13
        "launchpad.net/gomaasapi"
 
14
 
14
15
        "launchpad.net/juju-core/constraints"
15
16
        "launchpad.net/juju-core/environs"
16
17
        "launchpad.net/juju-core/environs/cloudinit"
17
18
        "launchpad.net/juju-core/environs/config"
18
19
        "launchpad.net/juju-core/environs/tools"
19
20
        "launchpad.net/juju-core/instance"
20
 
        "launchpad.net/juju-core/log"
21
21
        "launchpad.net/juju-core/state"
22
22
        "launchpad.net/juju-core/state/api"
23
23
        "launchpad.net/juju-core/utils"
74
74
        mcfg := environs.NewMachineConfig(machineID, state.BootstrapNonce, nil, nil)
75
75
        mcfg.StateServer = true
76
76
 
77
 
        log.Debugf("environs/maas: bootstrapping environment %q", env.Name())
 
77
        logger.Debugf("bootstrapping environment %q", env.Name())
78
78
        possibleTools, err := environs.FindBootstrapTools(env, cons)
79
79
        if err != nil {
80
80
                return nil, err
110
110
                if err2 != nil {
111
111
                        // Failure upon failure.  Log it, but return the
112
112
                        // original error.
113
 
                        log.Errorf("environs/maas: cannot release failed bootstrap instance: %v", err2)
 
113
                        logger.Errorf("cannot release failed bootstrap instance: %v", err2)
114
114
                }
115
115
                return fmt.Errorf("cannot save state: %v", err)
116
116
        }
196
196
                params.Add("mem", fmt.Sprintf("%d", *cons.Mem))
197
197
        }
198
198
        if cons.CpuPower != nil {
199
 
                log.Warningf("environs/maas: ignoring unsupported constraint 'cpu-power'")
 
199
                logger.Warningf("ignoring unsupported constraint 'cpu-power'")
200
200
        }
201
201
        return params
202
202
}
222
222
                return gomaasapi.MAASObject{}, nil, msg
223
223
        }
224
224
        tools := possibleTools[0]
225
 
        log.Warningf("environs/maas: picked arbitrary tools %q", tools)
 
225
        logger.Warningf("picked arbitrary tools %q", tools)
226
226
        return node, tools, nil
227
227
}
228
228
 
263
263
        defer func() {
264
264
                if err != nil {
265
265
                        if err := environ.releaseInstance(instance); err != nil {
266
 
                                log.Errorf("environs/maas: error releasing failed instance: %v", err)
 
266
                                logger.Errorf("error releasing failed instance: %v", err)
267
267
                        }
268
268
                }
269
269
        }()
288
288
        if err := environ.startNode(*instance.maasObject, series[0], userdata); err != nil {
289
289
                return nil, err
290
290
        }
291
 
        log.Debugf("environs/maas: started instance %q", instance.Id())
 
291
        logger.Debugf("started instance %q", instance.Id())
292
292
        return instance, nil
293
293
}
294
294
 
335
335
        maasObj := maasInst.maasObject
336
336
        _, err := maasObj.CallPost("release", nil)
337
337
        if err != nil {
338
 
                log.Debugf("environs/maas: error releasing instance %v", maasInst)
 
338
                logger.Debugf("error releasing instance %v", maasInst)
339
339
        }
340
340
        return err
341
341
}
411
411
}
412
412
 
413
413
func (environ *maasEnviron) Destroy(ensureInsts []instance.Instance) error {
414
 
        log.Debugf("environs/maas: destroying environment %q", environ.name)
 
414
        logger.Debugf("destroying environment %q", environ.name)
415
415
        insts, err := environ.AllInstances()
416
416
        if err != nil {
417
417
                return fmt.Errorf("cannot get instances: %v", err)
440
440
 
441
441
// MAAS does not do firewalling so these port methods do nothing.
442
442
func (*maasEnviron) OpenPorts([]instance.Port) error {
443
 
        log.Debugf("environs/maas: unimplemented OpenPorts() called")
 
443
        logger.Debugf("unimplemented OpenPorts() called")
444
444
        return nil
445
445
}
446
446
 
447
447
func (*maasEnviron) ClosePorts([]instance.Port) error {
448
 
        log.Debugf("environs/maas: unimplemented ClosePorts() called")
 
448
        logger.Debugf("unimplemented ClosePorts() called")
449
449
        return nil
450
450
}
451
451
 
452
452
func (*maasEnviron) Ports() ([]instance.Port, error) {
453
 
        log.Debugf("environs/maas: unimplemented Ports() called")
 
453
        logger.Debugf("unimplemented Ports() called")
454
454
        return []instance.Port{}, nil
455
455
}
456
456