Thanks for the patch @storgashov! Code looks good, but a couple of tests should 
also be added:

* Add the new options to the 
[AWSInstanceApiTest#testRunInstancesOptions](https://github.com/jclouds/jclouds/blob/master/providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/features/AWSInstanceApiTest.java#L119-L143),
 to verify the requests are properly generated.
* Include the new options in the 
[AWSEC2ComputeServiceApiMockTest#launchVPCSpotInstanceSubnetId](https://github.com/jclouds/jclouds/blob/master/providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/compute/AWSEC2ComputeServiceApiMockTest.java#L49-L87),
 or just copy the test and create a new one, to verify that options are 
properly populated from the compute service options to the generated http 
request.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/954#issuecomment-219988320

Reply via email to