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
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
+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
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
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
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
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
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
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
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
://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
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
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
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
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,
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:
>
> ---
>
+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
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
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
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
Team,
Please provide relevant permissions required. Currently, I am not able to
assign a bug to myself.
Thanks!
Santhosh
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.
> 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
,
Santhosh Edukulla
---
Tested changes on simulator, it created entities post the run so it seems tests
were working fine.
Thanks,
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
/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
--
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/
> --
-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
---
Thanks,
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
---
Thanks,
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
/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,
>
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
/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
-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
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
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
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
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
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
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
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
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
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
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:
>
> --
O] Apache CloudStack Console Proxy ...
> >>>> SUCCESS [0.045s]
> >>>>
> >>>> [INFO] Apache CloudStack Console Proxy - Server ..
> >>>> SUCCESS [3.691s]
> >>>>
> >>>> [INFO] Apache Cl
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
> 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
---
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
> 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
---
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
> 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
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
---
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
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
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
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
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
:
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
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.,
---
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
Diff: https://reviews.apache.org/r/15208/diff/
Testing
---
Thanks,
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
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
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
---
Thanks,
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
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
@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
Diffs
-
tools/marvin/pom.xml 0869248
Diff: https://reviews.apache.org/r/15304/diff/
Testing
---
Tested that installation works post packaging
Thanks,
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
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
---
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
--
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:
>
> -
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
)
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
-Utils.py-has-wrong-reference-for-ke_1.patch
Thanks,
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
thereafter
Thanks,
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
/diff/
Testing
---
Thanks,
Santhosh Edukulla
/marvin/marvin/sandbox/demo/simulator/testcase/libs/utils.py f26d2c0
Diff: https://reviews.apache.org/r/15392/diff/
Testing
---
Thanks,
Santhosh Edukulla
-asyncmgr.patch
Thanks,
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
./
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
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
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
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
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
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
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,
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
: 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
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
---
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
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 ...
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
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 - 100 of 557 matches
Mail list logo