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



test/integration/component/test_remotevpn_vpc.py
<https://reviews.apache.org/r/16867/#comment61421>

    I enabled EPEL repository on the CentOS/RHEL Server using this command. 
More information about it is present at 
    http://www.cyberciti.biz/faq/rhel-fedora-centos-linux-enable-epel-repo/
    EPEL mirror is needed to install vpn client related packages on the VM.
    
    The purpose of using this approach is to enable the VM deployed from 
default template to work as a VPN Client. As you are aware that we have 
different versions of CentOS Templates made available for different 
hypervisors. Packaging and installing them might result in dependency issues. 
Installation of all the rpms mentioned in the code happens within a minute. 
    
    Dropping the issue based on the above mentioned reason. Kindly reopen it in 
case if more explanation is required.
    



test/integration/component/test_remotevpn_vpc.py
<https://reviews.apache.org/r/16867/#comment61429>

    As you might have already noticed that the files constructed using the 
static methods are required for vpn client configuration on the VM. I did think 
about providing this scripts external to the script. But I noticed that it has 
a drawback. The configuration files are static. If the Test script writer wants 
to fine tune VPN Client installation in future in order to use the client to 
behave in a certain fashion, he/she needs to have the ability to modify the 
script to test the behavior of the feature(Ex: nat_traversal can be set to "no" 
in ipsec.conf script etc..).
    
    Using "w+" overwrites an existing file - 
http://stackoverflow.com/questions/10349781/how-to-open-read-write-or-create-a-file-with-truncation-possible
    
    Dropping the issue based on the above mentioned reason. Kindly reopen it in 
case if more explanation is required.
     



test/integration/component/test_remotevpn_vpc.py
<https://reviews.apache.org/r/16867/#comment61444>

    That's the syntax to run a function from a script in a command line.
    
    
http://stackoverflow.com/questions/8818119/linux-how-can-i-run-a-function-from-a-script-in-command-line
    
    Dropping the issue based on the above mentioned reason. Kindly reopen it in 
case if more explanation is required.
    
    


- Chandan Purushothama


On Jan. 16, 2014, 7:12 p.m., Chandan Purushothama wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16867/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2014, 7:12 p.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar, SrikanteswaraRao Talluri, 
> and Sheng Yang.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Test Suite for Testing Remote Access VPN on VPC. 
> 
> I successfully automated the following components:
> 
> 1.VPN Client Installation on Linux
> 2.Component that efficiently verifies Remote VPN Access between client and 
> the Server.
> 3.Developed a component that automates VPN Client Configuration and services.
> 4.Test Cases of the Feature
> 5.Tested the Code multiple times on XenServer and fixed the bugs.
> 6.Requirement on VMWare: Default Template should be CentOS 5.5 or higher for 
> the test suite to work. VPN Client cannot be installed on CentOS 5.3 Default 
> Template on VMWare.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_remotevpn_vpc.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/16867/diff/
> 
> 
> Testing
> -------
> 
> Test case no : Enable VPN for Public IP Address on the VPC ... ok
> Test case no : Remote a VPN User ... ok
> Test case no : Add a Different VPN User and Test Access with already existing 
> VPN User ... ok
> Test case no : Add a Previously Removed VPN User from the VPC and Test the 
> VPN Connectivity ... ok
> Test case no : Disable the VPN Service on the VPC ... ok
> Test case no : Enabled Previously Dsiabled VPN Access to VPC. ... ok
> Test case no : Create Nine VPN Users to test the remote.vpn.user.limit=8 
> Configuration parameter ... ok
> 
> ----------------------------------------------------------------------
> Ran 7 tests in 645.787s
> 
> OK
> 
> 
> File Attachments
> ----------------
> 
> 0001-Test-Suite-for-Remote-Access-VPN-on-VPC.patch
>   
> https://reviews.apache.org/media/uploaded/files/2014/01/14/c6d3f593-d0eb-407c-aad2-574ebf9ca0f8__0001-Test-Suite-for-Remote-Access-VPN-on-VPC.patch
> 
> 
> Thanks,
> 
> Chandan Purushothama
> 
>

Reply via email to