Merge lp:~axwalk/goamz/ec2test-runinstances-availzone into lp:goamz

Proposed by Andrew Wilkins
Status: Merged
Approved by: Ian Booth
Approved revision: 47
Merged at revision: 47
Proposed branch: lp:~axwalk/goamz/ec2test-runinstances-availzone
Merge into: lp:goamz
Diff against target: 11 lines (+1/-1)
1 file modified
ec2/ec2test/server.go (+1/-1)
To merge this branch: bzr merge lp:~axwalk/goamz/ec2test-runinstances-availzone
Reviewer Review Type Date Requested Status
goamz maintainers Pending
Review via email: mp+221982@code.launchpad.net

Commit message

ec2/ec2test: fix AvailZone handling in RunInstances

The RunInstances method in ec2test was expecting
a form parameter of "AvailZone", whereas the API
specifies it as "Placement.AvailabilityZone". I have
changed it to match.

Description of the change

ec2/ec2test: fix AvailZone handling in RunInstances

The RunInstances method in ec2test was expecting
a form parameter of "AvailZone", whereas the API
specifies it as "Placement.AvailabilityZone". I have
changed it to match.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ec2/ec2test/server.go'
--- ec2/ec2test/server.go 2014-05-30 09:43:41 +0000
+++ ec2/ec2test/server.go 2014-06-04 05:47:25 +0000
@@ -877,7 +877,7 @@
877 // make sure that form fields are correct before creating the reservation.877 // make sure that form fields are correct before creating the reservation.
878 instType := req.Form.Get("InstanceType")878 instType := req.Form.Get("InstanceType")
879 imageId := req.Form.Get("ImageId")879 imageId := req.Form.Get("ImageId")
880 availZone := req.Form.Get("AvailZone")880 availZone := req.Form.Get("Placement.AvailabilityZone")
881881
882 r := srv.newReservation(srv.formToGroups(req.Form))882 r := srv.newReservation(srv.formToGroups(req.Form))
883883

Subscribers

People subscribed via source and target branches