You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Gaurav Aradhye <ga...@clogeny.com> on 2013/07/19 11:53:34 UTC
Review Request 12758: Cloustack - 3452: Resolved ssh issue for basic zone.
Added security group for vms created in basic zone to allow ssh
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12758/
-----------------------------------------------------------
Review request for cloudstack and Prasanna Santhanam.
Repository: cloudstack-git
Description
-------
Resolved issue CLOUDSTACK - 3452: https://issues.apache.org/jira/browse/CLOUDSTACK-3452
Added basic security group if not present, each vm created in basic zone will be added to this security group.
This group has been authorized ingress traffic on port 22 (TCP connection).
Now SSH connection error is removed for basic zone.
For advanced zone, working remains the same as before.
Diffs
-----
tools/marvin/marvin/integration/lib/base.py 161d03c
Diff: https://reviews.apache.org/r/12758/diff/
Testing
-------
Tested below test cases:
integration.smoke.test_vm_life_cycle.TestVMLifeCycle.test_10_attachAndDetach_iso
integration.smoke.test_volumes.TestVolumes.test_02_attach_volume
integration.smoke.test_volumes.TestCreateVolume.test_01_create_volume
Thanks,
Gaurav Aradhye
Re: Review Request 12758: Cloustack - 3452: Added check if security groups
feature is enabled in basic zone before creating any sec group
Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12758/#review23571
-----------------------------------------------------------
Ship it!
commit 4f8c43fc2089e455aaad8203c01d44ed409d0c32 (master)
commit 3c360ce02807240586d471ab4c0cbd24d76ed7e6 (4.2)
- Prasanna Santhanam
On July 19, 2013, 1:07 p.m., Gaurav Aradhye wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12758/
> -----------------------------------------------------------
>
> (Updated July 19, 2013, 1:07 p.m.)
>
>
> Review request for cloudstack and Prasanna Santhanam.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Resolved issue CLOUDSTACK - 3452: https://issues.apache.org/jira/browse/CLOUDSTACK-3452
>
> Addded check if security groups feature is enabled for basic zone before creating security group while creating VM.
>
>
> Diffs
> -----
>
> tools/marvin/marvin/integration/lib/base.py 13507bc
>
> Diff: https://reviews.apache.org/r/12758/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Gaurav Aradhye
>
>
Re: Review Request 12758: Cloustack - 3452: Added check if security groups
feature is enabled in basic zone before creating any sec group
Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12758/
-----------------------------------------------------------
(Updated July 19, 2013, 1:07 p.m.)
Review request for cloudstack and Prasanna Santhanam.
Summary (updated)
-----------------
Cloustack - 3452: Added check if security groups feature is enabled in basic zone before creating any sec group
Repository: cloudstack-git
Description (updated)
-------
Resolved issue CLOUDSTACK - 3452: https://issues.apache.org/jira/browse/CLOUDSTACK-3452
Addded check if security groups feature is enabled for basic zone before creating security group while creating VM.
Diffs (updated)
-----
tools/marvin/marvin/integration/lib/base.py 13507bc
Diff: https://reviews.apache.org/r/12758/diff/
Testing (updated)
-------
Thanks,
Gaurav Aradhye