Code review comment for lp:~axwalk/juju-core/lp1317267-manual-addmachine1dot18

Reviewers: mp+218746_code.launchpad.net,

Message:
Please take a look.

Description:
environs/manual: use old AddMAchines API

Manual provisioning was implicitly changed
to use the new AddMachinesV2 client API,
with no backwards compatibility failover.
I've changed it to explicitly use the old
API, since we do not require anything of
the new one.

Fixes lp:1317267

https://code.launchpad.net/~axwalk/juju-core/lp1317267-manual-addmachine1dot18/+merge/218746

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/95150043/

Affected files (+3, -1 lines):
   A [revision details]
   M environs/manual/provisioner.go

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-20140508054219-e6ylw5luw58d6zhx
+New revision: <email address hidden>

Index: environs/manual/provisioner.go
=== modified file 'environs/manual/provisioner.go'
--- environs/manual/provisioner.go 2014-04-07 10:05:04 +0000
+++ environs/manual/provisioner.go 2014-05-08 06:55:15 +0000
@@ -130,7 +130,7 @@

  func recordMachineInState(
   client *api.Client, machineParams params.AddMachineParams) (machineId
string, err error) {
- results, err :=
client.AddMachines([]params.AddMachineParams{machineParams})
+ results, err :=
client.AddMachines1dot18([]params.AddMachineParams{machineParams})
   if err != nil {
    return "", err
   }