RE: SSHException('Error reading SSH protocol banner',) while sshing to a machine behind a load balancing rule

2013-09-22 Thread Santhosh Edukulla
Girish, Please check whether you were able to telnet to ssh port on that machine? If it works and as well, if you were able to manually do an ssh to that machine, then it may not be a firewall configuration issue. If we are using either pexpect\paramiko library( assuming ), then open call of

RE: Virtual Router reachability

2013-09-29 Thread Santhosh Edukulla
Girish, 1. Using timeout will make it to wait for that many units always and again it may not be fool proof, we may succeed to find ssh daemon on remote machine is up and running and with this, we again run it few more times to check again if it is not running, so its not much predictable. I

RE: [DISCUSS] Breaking out Marvin from CloudStack

2013-10-02 Thread Santhosh Edukulla
+1. All libraries of marvin and tests using them will now be at single place. Though marvin and tests under it are used to test cloudstack, segregating it from main cloudstack product code is good. Going ahead, few more changes will be required to make tests run automatically based upon the cl

RE: [DISCUSS] Breaking out Marvin from CloudStack

2013-10-02 Thread Santhosh Edukulla
I believe both can exist independently. The framework should have more flexibility and facilitation to run features\test cases based upon a given release and version on a given test bed. It should be intelligent enough to run those cases and only those cases pertaining to that release with few

RE: [DISCUSS] Breaking out Marvin from CloudStack

2013-10-03 Thread Santhosh Edukulla
rom: Chip Childers [chip.child...@sungard.com] Sent: Thursday, October 03, 2013 11:07 AM To: dev@cloudstack.apache.org Subject: Re: [DISCUSS] Breaking out Marvin from CloudStack On Thu, Oct 03, 2013 at 05:55:14AM +0000, Santhosh Edukulla wrote: > I believe both can exist independently. The fra

RE: [DISCUSS] Components in JIRA and bug assignment

2013-10-04 Thread Santhosh Edukulla
Not Sure, if it falls under similar lines\requires a separate thread. I am trying to see if there is a way we can add a component field, primary owner fields for component available to be assigned for a given review under Http://reviews.apache.org. This way i will select component as "Automa

RE: [PROPOSAL] Service monitoring tool in virtual router

2013-10-05 Thread Santhosh Edukulla
A shell script can be used. Few thoughts below: 1. Collect the process id of all daemons you wanted to monitor using "pidof" of command and then use "kill" command to check if the pid you got is valid. Using kill we can send a signal 0, then check the status using echo $? . For sending a notifi

RE: [DISCUSS] Breaking out Marvin from CloudStack

2013-10-07 Thread Santhosh Edukulla
Team, Apart\Away from breaking out marvin from cloudstack, please check the other new details provided as part of the new proposal for marvin refactoring. Your inputs out of experience are invaluable. Any new feature tests for CS will be followed with the new approach, provided if we agree to a

RE: [DISCUSS] Breaking out Marvin from CloudStack

2013-10-08 Thread Santhosh Edukulla
creation as similar to the one provided by db connection as part of test client currently, the new data object therefore as part of testclient will then replace the current Service Class\object creation for a given test module. Loading the values from xml\csv in to data object for test

Review Request 14595: Adding https support to marvin

2013-10-11 Thread Santhosh Edukulla
tools/marvin/marvin/cloudstackTestClient.py 36f7f8d tools/marvin/marvin/configGenerator.py a966ae0 tools/marvin/marvin/deployDataCenter.py beed8c8 Diff: https://reviews.apache.org/r/14595/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 14595: Adding https support to marvin

2013-10-11 Thread Santhosh Edukulla
://reviews.apache.org/r/14595/diff/ Testing (updated) --- Tested against http. Setting up https environment for testing. Will appreciate a test if anyone has https enabled on their setup Thanks, Santhosh Edukulla

Error Codes\ Export Import Config

2013-10-15 Thread Santhosh Edukulla
Hello Team, 1/ Is there a document with information to know all the Error\Fault codes, their level ,subject and description info returned by different API's under cloudstack? EX: If we call a particular API, its returned code at various levels of API call, for failure\Info\success case in gene

RE: Review Request 14593: CLOUDSTACK-1833: Automation - Adding scale virtual machine test cases

2013-10-16 Thread Santhosh Edukulla
If ddt library is required for running tests, please add this dependency to marvin setup.py "install_requires" list with relevant version requirement information. This will be easier to get the required packages. Santhosh From: Gaurav Aradhye [nore...@r

RE: Http connection failure in marvin

2013-10-16 Thread Santhosh Edukulla
Girish, Thanks for the bug. Few notes below. 1/ Can you please add the server GET\POST calls with params received at the server information to the bug? This will help to know for which GET/POST call this issue has appeared? 2/ Under tools/marvin/marvin/cloudstackConnection.py, there is a memb

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-16 Thread Santhosh Edukulla
tps://reviews.apache.org/r/14459/#comment52693> Where is timeout declared and initialized? - Santhosh Edukulla On Oct. 10, 2013, 10:23 a.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-16 Thread Santhosh Edukulla
create a gtm. Also,please add the path to test plan for this feature along with feature owner information, i will invite him\her as well. santhosh - Santhosh Edukulla On Oct. 15, 2013, 11:30 a.m., Sowmya Krishnan wrote: > > --- >

RE: Cloudstack collab Hackathons

2013-10-16 Thread Santhosh Edukulla
+1 Santhosh From: Kelcey Jamison Damage [kel...@backbonetechnology.com] Sent: Wednesday, October 16, 2013 9:02 PM To: dev@cloudstack.apache.org Subject: Re: Cloudstack collab Hackathons +1 again for KVM in Python! - Original Message - From: "Dar

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-17 Thread Santhosh Edukulla
tps://reviews.apache.org/r/14459/#comment52694> empty list check - Santhosh Edukulla On Oct. 10, 2013, 10:23 a.m., Ashutosh Kelkar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

RE: Cloudstack collab Hackathons

2013-10-17 Thread Santhosh Edukulla
Marcus, 1/ For me, its not about the debate of Java VS python, or supporting anything to re-write existing already. Its an effort, learning,idea and thought of contribution being new. Having said that, we worked earlier on building an agent prototype where we had to deploy agents and push po

RE: Issues deploying with marvin

2013-10-18 Thread Santhosh Edukulla
Hello William, 1. After getting the latest master branch code, I just ran the command " mvn -P developer,systemvm clean install" and I could see that both "commands.xml" file was created under cs/ tools/apidoc/target/ and "addImageStore.py" under cs/tools/marvin/marvin/cloudstackAPI/ dire

Jira: Assign Permissions

2013-10-18 Thread Santhosh Edukulla
Team, Please provide relevant permissions required. Currently, I am not able to assign a bug to myself. Thanks! Santhosh

Re: Review Request 14595: Adding https support to marvin

2013-10-20 Thread Santhosh Edukulla
ps://reviews.apache.org/r/14595/#review27185 --- On Oct. 11, 2013, 12:51 p.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-21 Thread Santhosh Edukulla
> On Oct. 16, 2013, 4:43 p.m., Santhosh Edukulla wrote: > > Sowmya, > > > > > > Is it ok if we do a code walkthrough? Let me know i can create a gtm. > > Also,please add the path to test plan for this feature along with feature > > owner inf

Review Request 14826: Adding codes file to marvin.

2013-10-22 Thread Santhosh Edukulla
, Santhosh Edukulla

Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-22 Thread Santhosh Edukulla
--- Tested changes on simulator, it created entities post the run so it seems tests were working fine. Thanks, Santhosh Edukulla

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Santhosh Edukulla
visit: https://reviews.apache.org/r/14869/#review27366 --- On Oct. 23, 2013, 6:33 a.m., Santhosh Edukulla wrote: > > --- > This is an automatically generate

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Santhosh Edukulla
/deployDataCenter.py 6b5f64b Diff: https://reviews.apache.org/r/14869/diff/ Testing --- Tested changes on simulator, it created entities post the run so it seems tests were working fine. Thanks, Santhosh Edukulla

Re: Review Request 14648: Added few changes related to 4870

2013-10-23 Thread Santhosh Edukulla
-- On Oct. 15, 2013, 12:10 p.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14648/ > --

Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
-git Description --- Adding fix for the bug 4885. This will add timestamp and timetaken in seconds for a given testcase run. Diffs - tools/marvin/marvin/marvinPlugin.py aded17c Diff: https://reviews.apache.org/r/14874/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
--- Thanks, Santhosh Edukulla

Review Request 14875: Fixed few pep8 compliant issues

2013-10-23 Thread Santhosh Edukulla
-git Description --- There were few pep8 changes, fixed it. Added new patch on top of existing one. Diffs - tools/marvin/marvin/marvinPlugin.py aded17c Diff: https://reviews.apache.org/r/14875/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-10-23 Thread Santhosh Edukulla
--- Thanks, Santhosh Edukulla

RE: is marvin on master broken?

2013-10-24 Thread Santhosh Edukulla
Hello Darren, Some trace is still missing i believe. i could not see the last stack frame in the below trace as what lead to this trace? I just pulled the latest from master branch and used marvin to deploy few cloudstack entities and it worked. Can you please provide the command you are usin

RE: is marvin on master broken?

2013-10-24 Thread Santhosh Edukulla
/cloudstackConnection.py", line 269, in marvinRequest self.logging.debug("sending %s request: %s %s" % (method, cmdname, AttributeError: 'NoneType' object has no attribute 'debug' On Thu, Oct 24, 2013 at 1:21 AM, Santhosh Edukulla wrote: > Hello Darren, >

RE: jira permissions check

2013-10-25 Thread Santhosh Edukulla
If it is related to my mail asking for edit permissions, Its resolved for now with prasanna's help. Regards, Santhosh From: Daan Hoogland [daan.hoogl...@gmail.com] Sent: Friday, October 25, 2013 5:12 AM To: dev Subject: jira permissions check H, I was

Review Request 14925: Added few misc changes to marvin

2013-10-25 Thread Santhosh Edukulla
/deployDataCenter.py f2dccdb tools/marvin/marvin/integration/lib/utils.py d81e80d tools/marvin/marvin/marvinPlugin.py 3b282e4 Diff: https://reviews.apache.org/r/14925/diff/ Testing --- Thanks, Santhosh Edukulla

Review Request 14930: Added readable timestamp changes for start and end time to debug logs

2013-10-25 Thread Santhosh Edukulla
-git Description --- Added few timestamp changes for start and endtime. Diffs - tools/marvin/marvin/marvinPlugin.py 3b282e4 Diff: https://reviews.apache.org/r/14930/diff/ Testing --- Thanks, Santhosh Edukulla

RE: [DISCUSS] (CLOUDSTACK-4971) Verifying Test Beds : prerequisites for launching tests

2013-10-27 Thread Santhosh Edukulla
Here, we mean test setup, not marvin setup ( or setup.py ) . Each test has their own dependency, we are having setup,flow,teardown for test module. By below content of bug description, we mean verifying the test dependencies under setup class of a test feature or module not the marvin setup. T

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-28 Thread Santhosh Edukulla
53626> We verified the check inside the while loop, why do we need to check again here. Also, the scope of snapshots variable is inside the while loop, how we are getting the scope of that variable here outside of while loop - Santhosh Edukulla On Oct. 28, 2013, 6:02 a.m., Ashutosh

Re: Review Request 14925: Added few misc changes to marvin

2013-10-28 Thread Santhosh Edukulla
ly, visit: https://reviews.apache.org/r/14925/#review27589 --- On Oct. 25, 2013, 10:58 a.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-mail. To re

RE: Tiered Quality

2013-10-28 Thread Santhosh Edukulla
1.It seems we already have a code coverage numbers using sonar as below. It currently shows only the numbers for unit tests. https://analysis.apache.org/dashboard/index/100206 2. The below link has an explanation for using it for both integration and unit tests. http://docs.codehaus.org/displa

Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-29 Thread Santhosh Edukulla
ation/lib/utils.py 7d662af Diff: https://reviews.apache.org/r/15021/diff/ Testing --- Basic check to retrieve configuration values is done. Thanks, Santhosh Edukulla

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-29 Thread Santhosh Edukulla
n/config/setup.cfg PRE-CREATION tools/marvin/marvin/configGenerator.py 0cfad30 tools/marvin/marvin/integration/lib/utils.py 7d662af Diff: https://reviews.apache.org/r/15021/diff/ Testing --- Basic check to retrieve configuration values is done. Thanks, Santhosh Edukulla

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-29 Thread Santhosh Edukulla
en if exception happens, what to do with exception, here if we are sure that Snapshot call is clean, then it has to be a product bug? - Santhosh Edukulla On Oct. 30, 2013, 4:56 a.m., Ashutosh Kelkar wrote: > > --- > This i

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-30 Thread Santhosh Edukulla
ration provided by default, we are providing him the parsed dictionary by default by placing the configuration at a default location. Only if he wanted to change this configuration to a separate one he can still change by using getConfig method. - Santhosh Edukulla On Oct. 30, 2013, 10

RE: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-30 Thread Santhosh Edukulla
those with config dictionary values, which will be filled in dynamically. What else you see is missing? Let me know and I can enhance it to add. Santhosh -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, October 30, 2013 3:42 PM To: Santhosh Edukulla Cc

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-30 Thread Santhosh Edukulla
figuration handling. - Santhosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15021/#review27764 --- On Oct. 30, 2013, 10:12 a.m., Santhosh Edukulla wrote: > > --

RE: Issues deploying with marvin

2013-10-30 Thread Santhosh Edukulla
O] Apache CloudStack Console Proxy ... > >>>> SUCCESS [0.045s] > >>>> > >>>> [INFO] Apache CloudStack Console Proxy - Server .. > >>>> SUCCESS [3.691s] > >>>> > >>>> [INFO] Apache Cl

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-30 Thread Santhosh Edukulla
failing the test case here? I mean its better to fail the test case instead of raising exception. What does it is of significance when exception is raised? what is the outcome of raising exception? Will the test case be falling in fail\error state? - Santhosh Edukulla On Oct. 30, 20

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-30 Thread Santhosh Edukulla
> On Oct. 30, 2013, 10:22 a.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/cloudstackTestClient.py, line 22 > > <https://reviews.apache.org/r/15021/diff/1/?file=373112#file373112line22> > > > > It came from merge, i didnt used them in modules requir

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-31 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14426/#review27877 --- Ship it! Ship It! - Santhosh Edukulla On Oct. 28, 2013, 7:03

Re: Review Request 14426: Tests for Netscaler support as external LB Provider in VPC

2013-10-31 Thread Santhosh Edukulla
> On Oct. 16, 2013, 4:43 p.m., Santhosh Edukulla wrote: > > Sowmya, > > > > > > Is it ok if we do a code walkthrough? Let me know i can create a gtm. > > Also,please add the path to test plan for this feature along with feature > > owner inf

Re: Review Request 14459: CLOUDSTACK-2243: Add automation tests for VMs base image update faclity

2013-10-31 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14459/#review27880 --- Ship it! Ship It! - Santhosh Edukulla On Oct. 31, 2013, 6:39

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-31 Thread Santhosh Edukulla
> On Oct. 30, 2013, 10:22 a.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 337 > > <https://reviews.apache.org/r/15021/diff/1/?file=373114#file373114line337> > > > > Nopes, its right, user by default no need to mention any pa

Re: Review Request 15021: Fixed Bug: 4899 : Added Configuration Support to Marvin.

2013-10-31 Thread Santhosh Edukulla
ation/lib/utils.py 7d662af Diff: https://reviews.apache.org/r/15021/diff/ Testing --- Basic check to retrieve configuration values is done. File Attachments (updated) Added changes as discussed https://reviews.apache.org/media/uploaded/files/2013/10/31/0001-Fixed-Bug-4899_2.patch Thanks, Santhosh Edukulla

Re: Review Request 15115: CLOUDSTACK-4335: Delete test_deployVmSharedNetworkWithoutIpRange

2013-10-31 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15115/#review27888 --- Ship it! Ship It! - Santhosh Edukulla On Oct. 31, 2013, 8:32

[SOLVED]: RE: Issues deploying with marvin

2013-10-31 Thread Santhosh Edukulla
they have similar problems... Cheers... On Wed, Oct 30, 2013 at 8:56 AM, Will Stevens wrote: > Yes. I believe the linking is failing because the host os and the guest os > are different. > > Ws > > On Wednesday, October 30, 2013, Santhosh Edukulla wrote: > >> Based upo

RE: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console

2013-11-01 Thread Santhosh Edukulla
Santhanam Cc: Santhosh Edukulla; cloudstack; Girish Shilamkar Subject: Re: Review Request 14874: Marvin Plugin Changes: Adding timestamp and timetaken in seconds for a given testcase run to console This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14874/ On

Review Request 15175: Added a verifyElementInList utility function

2013-11-01 Thread Santhosh Edukulla
ls import verifyElementInList >>> a=[1,2,3] >>> verifyElementInList(a,2) [0, 'ELEMENT NOT FOUND IN THE INPUT'] >>> verifyElementInList(a,1) [1, None] >>> verifyElementInList(a,3) [0, 'ELEMENT NOT FOUND IN THE INPUT'] Thanks, Santhosh Edukulla

RE: Marvin deploy failed on master

2013-11-02 Thread Santhosh Edukulla
Hi Min, 1/ I just pulled the latest code from master and used marvin, it went fine. Are you still facing this issue? Make sure your local setup is clean and marvin installed with latest pull. 2/ It seems you don't have codes.py file available under python/site-packages/.../marvin folder? Ide

Re: Review Request 15175: Added a verifyElementInList utility function

2013-11-04 Thread Santhosh Edukulla
: https://reviews.apache.org/r/15175/#review28085 --- On Nov. 1, 2013, 7:44 a.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-m

Re: Review Request 15175: Added a verifyElementInList utility function

2013-11-04 Thread Santhosh Edukulla
ip it. - Santhosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15175/#review28089 --- On Nov. 1, 2013, 7:44 a.m.,

Re: Review Request 15175: Added a verifyElementInList utility function

2013-11-04 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15175/#review28095 --- Ship it! Ship It! - Santhosh Edukulla On Nov. 1, 2013, 7:44 a.m

Review Request 15208: Added bug fix for 5032

2013-11-04 Thread Santhosh Edukulla
Diff: https://reviews.apache.org/r/15208/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 15208: Added fix for bug 5032. Provides custom assert facility to test features. Added the new assert to cloudstackTestCase

2013-11-04 Thread Santhosh Edukulla
tools/marvin/marvin/cloudstackTestCase.py 85ef542 tools/marvin/marvin/integration/lib/utils.py d53c1ae Diff: https://reviews.apache.org/r/15208/diff/ Testing --- Thanks, Santhosh Edukulla

Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

2013-11-05 Thread Santhosh Edukulla
st_project_limits.py 63d1a98 test/integration/component/memory_limits/test_mem_project_limits.py PRE-CREATION test/integration/component/memory_limits/test_project_limits.py 1c0ed92 Diff: https://reviews.apache.org/r/15241/diff/ Testing --- Thanks, Santhosh Edukulla

RE: Ping to shared network gateway failing

2013-11-06 Thread Santhosh Edukulla
In shared network, we need to explicitly create the gateway. The ip available to vm as gateway need to be physically existing as gateway device. Santhosh From: Gaurav Aradhye [gaurav.arad...@clogeny.com] Sent: Wednesday, November 06, 2013 5:55 AM To: dev@c

Review Request 15263: Added fix for bug 5056.

2013-11-06 Thread Santhosh Edukulla
--- Thanks, Santhosh Edukulla

Re: Review Request 15263: Added fix for bug 5056.

2013-11-06 Thread Santhosh Edukulla
fea9b12 Diff: https://reviews.apache.org/r/15263/diff/ Testing --- File Attachments (updated) Added New File https://reviews.apache.org/media/uploaded/files/2013/11/06/0001-Added-fix-for-bug-5056.patch Thanks, Santhosh Edukulla

Re: Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

2013-11-06 Thread Santhosh Edukulla
ntegration/component/cpu_limits/test_project_limits.py 63d1a98 test/integration/component/memory_limits/test_mem_project_limits.py PRE-CREATION test/integration/component/memory_limits/test_project_limits.py 1c0ed92 Diff: https://reviews.apache.org/r/15241/diff/ Testing --- Thanks, Santhosh Edukulla

RE: Ping to shared network gateway failing

2013-11-06 Thread Santhosh Edukulla
@cloudstack.apache.org Cc: Girish Shilamkar Subject: Re: Ping to shared network gateway failing Hi Santosh, Do you mean creating gateway through API or is this a physical setup requirement to be served before running the test case? Regards, Gaurav On Wed, Nov 6, 2013 at 4:52 PM, Santhosh Edukulla

Review Request 15304: Added installation of marvin along with packaging to mvn build

2013-11-07 Thread Santhosh Edukulla
Diffs - tools/marvin/pom.xml 0869248 Diff: https://reviews.apache.org/r/15304/diff/ Testing --- Tested that installation works post packaging Thanks, Santhosh Edukulla

Review Request 15307: Increasing the timeout values to 600 seconds from 300 seconds

2013-11-07 Thread Santhosh Edukulla
- tools/marvin/marvin/integration/lib/utils.py 4d048f0 tools/marvin/marvin/remoteSSHClient.py fea9b12 Diff: https://reviews.apache.org/r/15307/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 15304: Added installation of marvin along with packaging to mvn build

2013-11-07 Thread Santhosh Edukulla
this support,post those mails raising issue with marvin. - Santhosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15304/#review28364 ---

Re: Review Request 15263: Added fix for bug 5056.

2013-11-07 Thread Santhosh Edukulla
views.apache.org/r/15263/#review28380 --- On Nov. 6, 2013, 1:44 p.m., Santhosh Edukulla wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 15241: Renamed two modules with same name to avoid confusion while reporting test results

2013-11-07 Thread Santhosh Edukulla
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15241/#review28495 --- On Nov. 6, 2013, 2:37 p.m., Santhosh Edukulla wrote: > > -

Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-07 Thread Santhosh Edukulla
remotesshClient.py to sshClient.py, it is more apt name ssh can be to local as well remote Diffs - Diff: https://reviews.apache.org/r/15341/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-07 Thread Santhosh Edukulla
) patch file https://reviews.apache.org/media/uploaded/files/2013/11/08/0001-Bug-fix-for-5099-Utils.py-has-wrong-reference-for-ke.patch patchfile https://reviews.apache.org/media/uploaded/files/2013/11/08/0001-Bug-fix-for-5099-Utils.py-has-wrong-reference-for-ke_1.patch Thanks, Santhosh

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Santhosh Edukulla
-Utils.py-has-wrong-reference-for-ke_1.patch Thanks, Santhosh Edukulla

Re: Review Request 15341: Fix-for-5099: Utils.py-has-wrong-reference, cleaned it. As well added Uniform naming convention

2013-11-08 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28531 --- Ship it! Ship It! - Santhosh Edukulla On Nov. 8, 2013, 10:43

Review Request 15393: Fix for bug CLOUDSTACK-5121

2013-11-10 Thread Santhosh Edukulla
thereafter Thanks, Santhosh Edukulla

Review Request 15395: Fixed few test case failures

2013-11-10 Thread Santhosh Edukulla
/integration/component/test_vpc_routers.py 8ed99ca test/integration/component/test_vpc_vm_life_cycle.py cc65eed test/integration/component/test_vpc_vms_deployment.py ebf3b31 Diff: https://reviews.apache.org/r/15395/diff/ Testing --- Thanks, Santhosh Edukulla

Review Request 15394: Added fix for CLOUDSTACK-5134

2013-11-11 Thread Santhosh Edukulla
/diff/ Testing --- Thanks, Santhosh Edukulla

Review Request 15392: Added fix for CLOUDSTACK-5134

2013-11-11 Thread Santhosh Edukulla
/marvin/marvin/sandbox/demo/simulator/testcase/libs/utils.py f26d2c0 Diff: https://reviews.apache.org/r/15392/diff/ Testing --- Thanks, Santhosh Edukulla

Re: Review Request 15392: Added fix for CLOUDSTACK-5134

2013-11-11 Thread Santhosh Edukulla
-asyncmgr.patch Thanks, Santhosh Edukulla

Coverage Analysis: Few Issues

2013-11-14 Thread Santhosh Edukulla
Team, While running code coverage analysis with sonar for integration tests, based upon the errors thrown, i could see the below issues\notes with CS project. Issue1: The coverage tool is complaining about duplicate sources for below files. These are available with same name under folders ./

Build with master fails

2013-11-18 Thread Santhosh Edukulla
Team, 1. I am getting below error when build with latest master. Skipping tests is proceeding to make the build success. [INFO] Apache CloudStack Framework - QuickCloud .. SKIPPED [INFO] [INFO] BUILD FAILURE [INFO

RE: Build with master fails

2013-11-18 Thread Santhosh Edukulla
nd it was working last time... Can you share the text file from /root/softwares/csmaster/ cloudstack/services/console-proxy-rdp/rdpconsole/target/surefire-reports? Thx, Laszlo On Mon, Nov 18, 2013 at 2:53 PM, Santhosh Edukulla < santhosh.eduku...@citrix.com> wrote: > Team, > > 1

Marvin Enhancement

2013-11-19 Thread Santhosh Edukulla
1. If tests are failing and it is already a known product issue, we should find a way to show it as other away from "failed". This can add some enhancements to test cases, map bugs logged to failures and only new failures away from those. If it is a known issue, the failure should report it as k

RE: Build with master fails

2013-11-19 Thread Santhosh Edukulla
al Message----- From: Santhosh Edukulla [mailto:santhosh.eduku...@citrix.com] Sent: Tuesday, November 19, 2013 12:50 PM To: dev@cloudstack.apache.org Subject: RE: Build with master fails - Test set: rdpclient.Mock

Re: Review Request 16867: Test Suite for Testing Remote Access VPN on VPC

2014-01-21 Thread Santhosh Edukulla
Totally we could see that 7 users are getting created, 6 through xrange loop and one thereafter? We mentioned a limit of 8, we want to touch 8+ to check the condition? test/integration/component/test_remotevpn_vpc.py <https://reviews.apache.org/r/16867/#comment61

Re: Review Request 16867: Test Suite for Testing Remote Access VPN on VPC

2014-01-21 Thread Santhosh Edukulla
tps://reviews.apache.org/r/16867/#comment61151> Not clearly understood this line of socket.setdefa...? What is the purpose of this? There are no explicit socket calls opened? - Santhosh Edukulla On Jan. 16, 2014, 7:12 p.m., Chandan Purushothama

Re: Review Request 16867: Test Suite for Testing Remote Access VPN on VPC

2014-01-21 Thread Santhosh Edukulla
tps://reviews.apache.org/r/16867/#comment61152> This is not added to clean up? - Santhosh Edukulla On Jan. 16, 2014, 7:12 p.m., Chandan Purushothama wrote: > > --- > This is an automatically generated e-mail. To reply,

RE: Error Codes\ Export Import Config

2014-01-22 Thread Santhosh Edukulla
appropriate action, before proceeding with next action. EX: INVALID_INPUT,SUCCESS etc. Some times, throwing exception may also could be because of an issue with code away from request handling. Regards, Santhosh From: Santhosh Edukulla Sent: Tuesday

Re: Review Request 17116: Phase1 Marvin Changes.Fix-for-CLOUDSTACK-5674, 5498, 5500( Fixed Few issues in patch 17113 )

2014-01-22 Thread Santhosh Edukulla
: https://reviews.apache.org/r/17116/diff/ Testing --- File Attachments (updated) New Patch File https://reviews.apache.org/media/uploaded/files/2014/01/22/1759bdd3-40f9-4462-81b5-e44df20774dc__new_diff.patch Thanks, Santhosh Edukulla

Re: Review Request 16867: Test Suite for Testing Remote Access VPN on VPC

2014-01-22 Thread Santhosh Edukulla
Also, will it work for all shells, dont we need to source anything before execution for this behavior to work? I believe its better to call based upon cmd line arguments for safer side. - Santhosh Edukulla On Jan. 16, 2014, 7:1

Re: Review Request 16178: CLOUDSTACK-2237: Automation - Adding new test cases for security group in advanced zone feature

2014-01-22 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16178/#review32598 --- Ship it! Ship It! - Santhosh Edukulla On Jan. 14, 2014, 8:51

Build on 4.3-forward Fails: Shows Failure for Framework-Clustering

2014-01-22 Thread Santhosh Edukulla
Team, 1. We are seeing a build failure for latest 4.3-forward branch code. Below is the error message: Apache CloudStack Cloud Engine API SUCCESS [2.351s] [INFO] Apache CloudStack Core SUCCESS [4.720s] [INFO] Apache CloudStack Agents ...

Review Request 17233: Fix for Issue CLOUDSTACK-5875

2014-01-23 Thread Santhosh Edukulla
rsion.check to false under global properties of CS post run using simulator. Otherwise you may see errors related to router upgrade etc. This i believe is a known issue and nothing to do with simulator. 3. Post the changes, tested only with simulator. Thanks, Santhosh Edukulla

RE: [ACS 4.3] RC update

2014-01-23 Thread Santhosh Edukulla
Simulator issue is fixed. It now works, please check. Santhosh From: sebgoa [run...@gmail.com] Sent: Wednesday, January 15, 2014 5:50 AM To: Santhosh Edukulla Cc: dev@cloudstack.apache.org; gir...@clogeny.com; Prasanna Santhanam Subject: Re: [ACS 4.3] RC

  1   2   3   4   5   6   >