-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6203/
-----------------------------------------------------------

(Updated Aug. 16, 2012, 8:53 a.m.)


Review request for cloudstack and Prachi Damle.


Description
-------

In CloudStack, changing the api response of deployVirtualMachine and 
listVirtualMachine to include ssh keypair information.


This addresses bug CS-14770.


Diffs (updated)
-----

  api/src/com/cloud/api/response/UserVmResponse.java 652e026 
  awsapi/src/com/cloud/bridge/service/EC2SoapServiceImpl.java 38f80c8 
  awsapi/src/com/cloud/bridge/service/core/ec2/EC2Engine.java 1363d0d 
  awsapi/src/com/cloud/bridge/service/core/ec2/EC2Instance.java 6e2d696 
  awsapi/src/com/cloud/stack/models/CloudStackUserVm.java e1db2d9 
  server/src/com/cloud/api/ApiDBUtils.java e1e8865 
  server/src/com/cloud/api/ApiResponseHelper.java 0340a94 
  server/src/com/cloud/user/dao/SSHKeyPairDao.java 84295da 
  server/src/com/cloud/user/dao/SSHKeyPairDaoImpl.java 58999e0 
  server/src/com/cloud/vm/dao/UserVmData.java 4a0d10d 
  server/src/com/cloud/vm/dao/UserVmDetailsDaoImpl.java 6975c8c 

Diff: https://reviews.apache.org/r/6203/diff/


Testing
-------

Verified that the output of ec2-run-instances and ec2-describe-instances 
contained the keypair-name that was used to launch the virtual-machine.


Thanks,

Likitha Shetty

Reply via email to