You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by SrikanteswaraRao Talluri <sr...@citrix.com> on 2013/08/06 13:31:39 UTC

Review Request 13321: Fix test_vm_passwordenabled.py to properly reset guest VM password

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

Review request for cloudstack and Prasanna Santhanam.


Bugs: CLOUDSTACK-4098


Repository: cloudstack-git


Description
-------

Modify password enable vm script for the following: 

1. correct the services dictionary for the "small" virtual machine details. 
2. add the download of cloud-set-guest-password script and required egress rule(port 80). 
3. correct base.py for the following 
    a. get the reset password correctly. 
    b. add EgressFirewallRule command. 


Diffs
-----

  test/integration/component/test_vm_passwdenabled.py ebe32c8 
  tools/marvin/marvin/integration/lib/base.py 770ac3d 

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


Testing
-------

tests passed.


Thanks,

SrikanteswaraRao Talluri

Re: Review Request 13321: Fix test_vm_passwordenabled.py to properly reset guest VM password

Posted by Prasanna Santhanam <ts...@apache.org>.

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

Ship it!


- Prasanna Santhanam


On Aug. 6, 2013, 11:31 a.m., SrikanteswaraRao Talluri wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13321/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2013, 11:31 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4098
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Modify password enable vm script for the following: 
> 
> 1. correct the services dictionary for the "small" virtual machine details. 
> 2. add the download of cloud-set-guest-password script and required egress rule(port 80). 
> 3. correct base.py for the following 
>     a. get the reset password correctly. 
>     b. add EgressFirewallRule command. 
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_vm_passwdenabled.py ebe32c8 
>   tools/marvin/marvin/integration/lib/base.py 770ac3d 
> 
> Diff: https://reviews.apache.org/r/13321/diff/
> 
> 
> Testing
> -------
> 
> tests passed.
> 
> 
> Thanks,
> 
> SrikanteswaraRao Talluri
> 
>