Re: Adding git commit ids to our packaging...

2013-07-17 Thread Prasanna Santhanam
chive before tar'ing, great! > > > > The only other minor concern would be the release verification process will > > have to be updated to account for the fact that we'll have one file (or some > > lines) that are different between the actual tagged commit in the repo and > > what's in the tarball. As far as I know, that's just a process change for > > us. > > Easy... > > > > -chip -- Prasanna., Powered by BigRock.com

Re: Review Request 12700: Fix for CLOUDSTACK-3596 to include domain id

2013-07-17 Thread Prasanna Santhanam
e you are a committer. please commit this. - Prasanna Santhanam On July 17, 2013, 6:10 p.m., sailaja mada wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

deleteAffinityGroup API

2013-07-17 Thread Prasanna Santhanam
@Override @ActionEvent(eventType = EventTypes.EVENT_AFFINITY_GROUP_DELETE, eventDescription = "Deleting affinity group") public boolean deleteAffinityGroup(Long affinityGroupId, String account, Long domainId, String affinityGroupName) { Thanks, -- Prasanna.,

[rant] stupid test cases

2013-07-17 Thread Prasanna Santhanam
#x27;s a pointless effort. I'm sure there are others in the automated tests that do this kind of wasteful testing. So please please pleaes &*()#@()# please review test plans before automating them! I'm not going to be looking at this forever to fix these issues wh

Re: Is it possible for reviewer to add other reviewers in the reviewboard?

2013-07-17 Thread Prasanna Santhanam
est damn cloud orchestrator out there. On Wed, Jul 17, 2013 at 03:02:23PM -0700, Sheng Yang wrote: > Or only submitter can modify it? > > I find it would be useful if we identify the people who need to review it > after it's submitted. > > --Sheng -- Prasanna., Powered by BigRock.com

CallContexts?

2013-07-17 Thread Prasanna Santhanam
-4ddf-ab47-9307a600d31e] * CallContext records information about the environment the call is made. This * class must be always be available in all CloudStack code. Does this mean we have to do something special when creating new APIs? I'm not quite clear from this description. Thanks,

Re: CallContexts?

2013-07-18 Thread Prasanna Santhanam
On Thu, Jul 18, 2013 at 11:58:30AM +0530, Prasanna Santhanam wrote: > I see the following repeated lines with API calls on master code > lately: What's the call context? and what's the role? > > 2013-07-18 11:52:24,372 DEBUG [cloudstack.context.CallContext] > (RouterSta

Re: deleteAffinityGroup API

2013-07-18 Thread Prasanna Santhanam
n for a regular user's > group. > Thanks Prachi - I think it is related to the recent changes in CallContext that is making the user system for the API call preventing it from deleteing the aff.group with just an id. Filed a bug for it. -- Prasanna., Powered by BigRock.com

Re: Review Request 12700: Fix for CLOUDSTACK-3596 to include domain id

2013-07-18 Thread Prasanna Santhanam
DSTACK-3596: Domain ID missing to deploy VM request . Included Domain id > > thanks. submitted this - Prasanna --- This is an automatically generated e-mail. To reply, visit: https://

Re: Review Request 12717: CLOUDSTACK-3618: [Automation] API call listprojectaccounts failed, and test case test_projects.py:test_08_cleanup_after_project_delete failed due to this.

2013-07-18 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12717/#review23359 --- Ship it! Ship It! - Prasanna Santhanam On July 18, 2013, 9:14

Re: Review Request 12717: CLOUDSTACK-3618: [Automation] API call listprojectaccounts failed, and test case test_projects.py:test_08_cleanup_after_project_delete failed due to this.

2013-07-18 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12717/#review23364 --- Ship it! Ship It! - Prasanna Santhanam On July 18, 2013, 9:14

Re: Review Request 12546: CLOUDSTACK-3168: test_network.py - Changed try_ssh function defn. Resolved Object NoneType issues

2013-07-18 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12546/#review23365 --- Ship it! Ship It! - Prasanna Santhanam On July 18, 2013, 8:45

Re: Review Request 12720: CLOUDSTACK: 3382 Unable to Migrate VM's If the hosts are implicitly or explicitly dedicated.

2013-07-18 Thread Prasanna Santhanam
test_explicit_dedication.py for this failure? That would be really useful to catch this part of tricky code - Prasanna Santhanam On July 18, 2013, 11:06 a.m., Saksham Srivastava wrote: > > --- > This is an automatically generated e-mail. To rep

Re: [jira] [Closed] (CLOUDSTACK-3636) [Automation]Fix iintegration.component.test_accounts.TestServiceOfferingHierarchy.test_01_service_offering_hierarchy script

2013-07-18 Thread Prasanna Santhanam
second, msg=msg) > > File "/usr/local/lib/python2.7/unittest/case.py", line 487, in > > _baseAssertEqual > > raise self.failureException(msg) > > AssertionError: Check List Service Offerings for a valid response > > -- > This message is automatically generated by JIRA. > If you think it was sent incorrectly, please contact your JIRA administrators > For more information on JIRA, see: http://www.atlassian.com/software/jira -- Prasanna., Powered by BigRock.com

Re: deleteAffinityGroup API

2013-07-18 Thread Prasanna Santhanam
On Thu, Jul 18, 2013 at 02:17:46PM +0530, Prasanna Santhanam wrote: > On Thu, Jul 18, 2013 at 07:14:42AM +, Prachi Damle wrote: > > Account and domainId are not required parameters of this API. It > > works fine with just an id too. > > > > Account and domain will

Re: Is it possible for reviewer to add other reviewers in the reviewboard?

2013-07-18 Thread Prasanna Santhanam
ed this is me volunteering to keep track of > unanswered questions for a few weeks. As we go along good ideas on how to > improve our way might spring up. /me is an optimist at rare occasions. I rant and go do the postive thing hoping for it to be picked up. Guess that makes me an optimist :) -- Prasanna., Powered by BigRock.com

Re: Why are these code in utils?

2013-07-18 Thread Prasanna Santhanam
e service layer expects as Map but wasn't sure if it should go here. I guess it should now that you mention it? -- Prasanna., Powered by BigRock.com

Re: deleteAffinityGroup API

2013-07-18 Thread Prasanna Santhanam
ow I repro-ed the issue. On Thu, Jul 18, 2013 at 11:14:19PM +, Prachi Damle wrote: > Prasanna does the regression test scripts call using 8096 port? > > If yes then that's the reason why the API is failing. > > From: Alena Prokharchyk > Sent: Thursday

Re: Review Request 12755: Fix ostype CLOUDSTACK-3645

2013-07-18 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12755/#review23481 --- Ship it! Ship It! - Prasanna Santhanam On July 19, 2013, 5:25

Re: New user - Introduction

2013-07-19 Thread Prasanna Santhanam
x27;d > like to contribute more to this great project in the future. > > Best regards, > > Rene Diepstraten -- Prasanna., Powered by BigRock.com

Re: Does the default Xen SystemVM Template understand S3?

2013-07-19 Thread Prasanna Santhanam
rote: > > > AFAIK, the database deploy script in Master sets the Xen SystemVM URL > > > to > > > http://download.cloud.com/templates/acton/acton-systemvm- > > 02062012.vhd. > > > bz2 > > > > > > Is this SystemVM image aware of S3 object store? > > >

Re: [rant] stupid test cases

2013-07-19 Thread Prasanna Santhanam
won't clean up because the > > account is gone and we throw null pointers because a bunch of code looks up > > account when deleting. However, to your point, VMs can be created in a > > "stopped" state and that wouldn't incur the overhead of deployment. &g

Re: Reviewboard and jenkins

2013-07-19 Thread Prasanna Santhanam
be seen here: > https://reviews.apache.org/r/12762/ > > Cheers, > > Hugo YES! YES! YES! and link it to the checkin tests coming up in the `simulator` view on jenkins.cloudstack.org. -- Prasanna., Powered by BigRock.com

Re: Noisy ASFBot

2013-07-19 Thread Prasanna Santhanam
hink? > > Cheers, > /ignore ASFBot or Cato_Fong? -- Prasanna., Powered by BigRock.com

master unstable with CLOUDSTACK-3682/3683

2013-07-19 Thread Prasanna Santhanam
Both bugs are blocking the run of tests on master. -- Prasanna., Powered by BigRock.com

Re: Review Request 12762: CLOUDSTACK-3620: [Automation] API removevpnuser failed, in test case test_project_usage.TestVpnUsage.test_01_vpn_usage.

2013-07-19 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12762/#review23568 --- Ship it! - Prasanna Santhanam On July 19, 2013, 3:44 p.m

Re: Review Request 12758: Cloustack - 3452: Added check if security groups feature is enabled in basic zone before creating any sec group

2013-07-20 Thread Prasanna Santhanam
) 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

Re: Reviewboard and jenkins

2013-07-20 Thread Prasanna Santhanam
> There are probably some bug still in the script, but hey we got to > start somewhere. ;-) > Absolutely, could you put it up on cso-infra/github so we can hack it and customize? -- Prasanna., Powered by BigRock.com

Re: Noisy ASFBot

2013-07-20 Thread Prasanna Santhanam
eedoh in the #asfinfra channel is your man for tuning the bot. On Sat, Jul 20, 2013 at 12:15:33PM +0200, Daan Hoogland wrote: > I agree, > > @Prasanna, can we make ignore default and /activate ASFBot? That would have > my preference. > > Daan > > > On Sat, Jul 20, 201

Re: Noisy ASFBot

2013-07-20 Thread Prasanna Santhanam
I've turned off the noisy jobs from Cato_Fong. I suspect it isn't useful to others but only to those who are debugging the tests. Will enable them at a more appropriate time later. On Sat, Jul 20, 2013 at 04:23:36PM +0530, Prasanna Santhanam wrote: > What I like particularly about A

Re: Reviewboard and jenkins

2013-07-20 Thread Prasanna Santhanam
On Sat, Jul 20, 2013 at 09:46:59AM -0400, David Nalley wrote: > On Sat, Jul 20, 2013 at 5:53 AM, Hugo Trippaers wrote: > > > > > > Sent from my iPhone > > > > On 20 jul. 2013, at 10:09, Prasanna Santhanam wrote: > > > >> On Sat, Jul 20

Re: Using the ACS simulator for jclouds "live" tests - some questions. =)

2013-07-21 Thread Prasanna Santhanam
s > wrote: > > > On Tue, Jul 09, 2013 at 05:24:53PM +, Edison Su wrote: > > > I think it's the issue > > https://issues.apache.org/jira/browse/CLOUDSTACK-3139, that your mgt > > server can't access the url of template. > > > > Yeah, it appears so. Thanks for searching for me. Apparently I was too > > lazy to actually check myself. ;-) > > -- Prasanna., Powered by BigRock.com

Re: Review Request 12805: CLOUDSTACK-3695: test_project_configs.TestProjectInviteRequired.test_add_user_to_project failed with missing attribute "email"

2013-07-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12805/#review23618 --- Ship it! Ship It! - Prasanna Santhanam On July 22, 2013, 9:09

Re: Review Request 12804: CLOUDSTACK-3679: Resolved ssh issue for load balance test cases

2013-07-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12804/#review23619 --- Ship it! Ship It! - Prasanna Santhanam On July 22, 2013, 8:24

Fwd: Issues with LDAP (git will be down for pushes/reverts)

2013-07-22 Thread Prasanna Santhanam
What's broken is broken for now. -- Prasanna., Powered by BigRock.com --- Begin Message --- Good morning, We are currently investigating an issue with LDAP as we have seen a few problem reports this morning with ability to use credentials. You may find yo

Re: Reviewboard and jenkins

2013-07-22 Thread Prasanna Santhanam
#x27; the patch based on the jenkins 'Ship it'. Only that the patch compiles successfully and is good to be reviewed by a human reviewer. I know that's common sense, but just wanted to spell it out. Thanks, -- Prasanna., Powered by BigRock.com

Re: Review Request 12812: CLOUDSTACK-3610: Fix regression test "test_accounts.TestUserLogin"

2013-07-22 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12812/#review23630 --- Ship it! Ship It! - Prasanna Santhanam On July 22, 2013, 1:52

Re: Tracing registerTemplate API, is it possible to connect Eclipse debugger to System VMs?

2013-07-22 Thread Prasanna Santhanam
> > --- > > Yu-Heng (Ryan) Lei, Associate Reasearcher Chunghwa Telecom Laboratories / > > Cloud Computing Laboratory > > ryan...@cht.com.tw<https://email.cht.com.tw/owa/redir.aspx?C=- > > wE1FEC3G0SWYpVkiWo8SsDdf3ZqO9AIuAPTzRnFYCUi- > > z4YljtI_hyVKkNHfn9F1Bn-vUWJnQ4.&URL=mailto%3aryanlei%40cht.com.tw> > > or > > ryanlei750...@gmail.com -- Prasanna., Powered by BigRock.com

Re: Regression failures

2013-07-22 Thread Prasanna Santhanam
s, On Wed, Jul 17, 2013 at 04:34:26PM +0530, Prasanna Santhanam wrote: > Reviving this effort starting today: > > I'm going to switch the test infra to run regression tests only for > the next couple of weeks. So far the BVTs have been running and we > have good success rate on the

Re: Regression failures

2013-07-22 Thread Prasanna Santhanam
(status = Open or status = > Reopened or status = "Ready To Review") and (summary ~ Automation OR > component = Automation) order by key desc """ > > > > -Original Message- > > From: Prasanna Santhanam [mailto:t...@apache.org] > >

Re: [Discuss] Apply rules on Virtual Router

2013-07-23 Thread Prasanna Santhanam
done for any configuration within the VR not just iptable rules? -- Prasanna., Powered by BigRock.com

Re: Review Request 12817: CLOUDSTACK-3689: Test case test_accounts.TesttemplateHierarchy.test_01_template_hierarchy failed during ListTemplate call

2013-07-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12817/#review23689 --- Ship it! Ship It! - Prasanna Santhanam On July 22, 2013, 3:25

Re: Review Request 12853: CLOUDSTACK-3688: Test cases test_accounts.TestDomainForceRemove.test_forceDeleteDomain, not found the domain to delete and failed.

2013-07-23 Thread Prasanna Santhanam
Prokharchyk, Devdeep Singh, and Prasanna Santhanam. Changes --- Change to test is fine. Not sure about the DomainManager change. Adding Alena as reviewer. Bugs: CLOUDSTACK-3688 Repository: cloudstack-git Description --- CLOUDSTACK-3688: Test cases

Re: Review Request 12854: CLOUDSTACK-3727: Resource Count calculation happens for deleted accounts on restart of managment server.

2013-07-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12854/#review23695 --- Ship it! Ship It! - Prasanna Santhanam On July 23, 2013, 12:54

Re: Review Request 12859: fixed account objects CLOUDSTACK-3726

2013-07-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12859/#review23700 --- Ship it! Ship It! - Prasanna Santhanam On July 23, 2013, 3:11

Re: Review Request 12858: CLOUDSTACK-3724: Test case "test_vpc_vms_deployment.py" failed to find network offering.

2013-07-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12858/#review23701 --- Ship it! Ship It! - Prasanna Santhanam On July 23, 2013, 2:53

Re: [rant] stupid test cases

2013-07-23 Thread Prasanna Santhanam
fixed. And the entire suite (atleast 8 to 10 tests) are such scenarios. We need to start working on critically reviewing the tests that come through. Of course I'd like to hear others thoughts on this. On Fri, Jul 19, 2013 at 06:06:18PM +0530, Prasanna Santhanam wrote: > My problem is tha

Re: Regression failures

2013-07-23 Thread Prasanna Santhanam
On Tue, Jul 23, 2013 at 11:56:02AM +0530, Prasanna Santhanam wrote: > Thanks Sowmya, I see more issues in the list now when the component is > Automation. Filtered this further down to : """ project = CloudStack AND (status = Open OR status = Reopened OR status = "Ready

New Components on JIRA

2013-07-23 Thread Prasanna Santhanam
Router - Anything related to the Virtual Router appliance * XCP - Xen Cloud Platform --- Anyone see any problems? -- Prasanna., Powered by BigRock.com

Re: New Components on JIRA

2013-07-23 Thread Prasanna Santhanam
On Tue, Jul 23, 2013 at 05:23:03PM +, Ram Ganesh wrote: > > -Original Message- > > From: Prasanna Santhanam [mailto:t...@apache.org] > > Sent: 23 July 2013 22:36 > > To: CloudStack Dev > > Subject: New Components on JIRA > > > > The following

Re: Review Request 11942: ProcessUtil cleanup

2013-07-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11942/ --- (Updated July 24, 2013, 4:42 a.m.) Review request for cloudstack, Frank Zhang a

Automated Tests: Account Cleanup in test modules

2013-07-23 Thread Prasanna Santhanam
plugin in nose that will abort the test if it takes longer than 1/2hr. I think this should help weeding out test that do arbitrary 'sleeps' or wait for very long cleanup operations. That way we should be able to optimize the test as well Thoughts? -- Prasanna., Powered by BigRock.com

Re: Review Request 12893: CLOUDSTACK-3696: Test case test_project_limits.TestResourceLimitsProject.test_06_volumes_per_project failed due to volume usage exceeded for project.

2013-07-23 Thread Prasanna Santhanam
5-a459-648f01c833c1 testclient.testcase.TestResourceLimitsProject: DEBUG: Deploying VM for project: 2ca8db1a-a97a-4cf5-a459-648f01c833c1 I will attach the logs to the bug report - Prasanna Santhanam On July 24, 2013, 5 a.m., Sanjay Tripathi wrote: > > ---

Re: Automated Tests: Account Cleanup in test modules

2013-07-23 Thread Prasanna Santhanam
r (I got questions from others working on the tests) so it was better to spell them out. It's time to write up these guidelines in our wiki which has some basic guidelines already: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+-+Testing+with+Python#Marvin-TestingwithPython-Guid

Re: Review Request 12886: Fix test case issue test_vm_life_cycle.py:test_09_expunge_vm : CLOUDSTACK-2899

2013-07-24 Thread Prasanna Santhanam
deployment. The magic timeout here will again fail when the number of tests you are running in parallel increase exhausting the workers. Also we are already waiting for 2 cycles of the expunge thread to ensure that we don't miss because of expunge thread exhaustion - Prasanna Santhanam On

Re: Review Request 12897: CLOUDSTACK-3594: Fix regression in Affinity Groups tests

2013-07-24 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12897/#review23765 --- Ship it! Ship It! - Prasanna Santhanam On July 24, 2013, 10:51

Re: Review Request 12874: Fix reboot failures with wait time

2013-07-24 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12874/#review23766 --- I think you forgot to attach the diff here. - Prasanna Santhanam

Re: [rant] stupid test cases

2013-07-24 Thread Prasanna Santhanam
On Tue, Jul 23, 2013 at 06:12:31PM +, Mathias Mullins wrote: > Prasanna, > > If we broke these tests down into smaller test specific test cases, could > we automate them into one specific end-to-end test together? I think this is called 'sequencing'. You have

Re: Review Request 12874: Fix reboot failures with wait time

2013-07-24 Thread Prasanna Santhanam
> > From: Sowmya Krishnan [mailto:sowmya.krish...@citrix.com] > > Sent: Wednesday, July 24, 2013 10:37 PM > > To: dev@cloudstack.apache.org; Prasanna Santhanam > > Subject: RE: Review Request 12874: Fix reboot failures with wait time > > > > RB doesn???t ac

Re: Review Request 12934: Tests for egress firewall rules for advance zone

2013-07-24 Thread Prasanna Santhanam
Shilamkar, Jayapal Reddy, and Prasanna Santhanam. Changes --- Jayapal, need your help reviewing this buddy. Repository: cloudstack-git Description --- Tests for egress firewall rules for advance zone. Diffs - test/integration/component/test_egress_fw_rules.py PRE-CREATION Diff

Re: Review Request 12913: Fix map based APIs in CloudMonkey

2013-07-24 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12913/ --- (Updated July 25, 2013, 4:52 a.m.) Review request for cloudstack and Rohit Yada

Re: Automated Tests: Account Cleanup in test modules

2013-07-24 Thread Prasanna Santhanam
On Wed, Jul 24, 2013 at 01:45:13PM -0700, Sheng Yang wrote: > On Tue, Jul 23, 2013 at 11:54 PM, Prasanna Santhanam wrote: > > > On Tue, Jul 23, 2013 at 11:20:22PM -0700, Sheng Yang wrote: > > > About the patch I committed, in fact it's not moving. It's fixin

Re: Review Request 12723: test for Script

2013-07-25 Thread Prasanna Santhanam
4.2 commit 76e283687f8a9014d85e766c2543f3c98dcb6062 -> master - Prasanna Santhanam On July 24, 2013, 5:42 p.m., Laszlo Hornyak wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 12874: Fix reboot failures with wait time

2013-07-25 Thread Prasanna Santhanam
> On July 24, 2013, 4:25 p.m., Prasanna Santhanam wrote: > > I think you forgot to attach the diff here. Sent via mailing list and applied - Prasanna --- This is an automatically generated e-mail. To reply, vis

Re: Build failure on master...anyone know what this is?

2013-07-25 Thread Prasanna Santhanam
dCommand=1 > > Traceback (most recent call last): > > File "/cyg/d/src/acs/master/tools/apidoc/gen_toc.py", line 189, in > > > > category = choose_category(fn) > > File "/cyg/d/src/acs/master/tools/apidoc/gen_toc.py", line 169, in > > choose_category > > (fn, __file__)) > > Exception: Need to add a category for deleteCacheStore.xml to > > /cyg/d/src/acs/master/tools/apidoc/gen_toc.py:known_categories > > > > > > Thanks. > > --Alex > > -- Prasanna., Powered by BigRock.com

Re: Build failure on master...anyone know what this is?

2013-07-25 Thread Prasanna Santhanam
tegory = choose_category(fn) > File "/cyg/d/src/acs/master/tools/apidoc/gen_toc.py", line 169, in > choose_category > (fn, __file__)) > Exception: Need to add a category for deleteCacheStore.xml to > /cyg/d/src/acs/master/tools/apidoc/gen_toc.py:known_categories > > > Thanks. > --Alex -- Prasanna., Powered by BigRock.com

Re: Review Request 12948: Affinity Groups: Adding new test cases

2013-07-26 Thread Prasanna Santhanam
rebase against the latest changes. I saw your change only after I had made changes. - Prasanna Santhanam On July 25, 2013, 3:32 p.m., Gaurav Aradhye wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Prasanna Santhanam
last night. Can you reuse that? We need to keep the delete operations consistent across the integration library. If a custom delete is required, then that can be handled within the test. So this test will fail with the latest marvin - Prasanna Santhanam On July 26, 2013, 7:42 a.m., Girish Shil

Re: unanswerred questions 15-21 juli

2013-07-26 Thread Prasanna Santhanam
impression > that this number is rather stable. The week after ccc it was 6. > > greeting from earth, > Daan Daan, Thanks for keeping track. Can you also include those from markmail so anyone who has perhaps missed an opportunity to answer can reply back? -- Prasanna., Powered by BigRock.com

xs-tools

2013-07-26 Thread Prasanna Santhanam
b.com/mcclurmc/xs-tools/blob/master/get-xs-tools.sh -- Prasanna., Powered by BigRock.com

Re: xs-tools

2013-07-26 Thread Prasanna Santhanam
On Fri, Jul 26, 2013 at 03:43:57PM +0530, Prasanna Santhanam wrote: > Does anybody know where and how the xs-tools.iso comes from on the > ISOs page? It is downloaded and ready after the SSVM shows up. But I > don't see it anywhere in the urls in the template table or in the > cod

Re: Review Request 12962: CLOUDSTACK-3594: Fix params of AffinityGroup.delete API

2013-07-26 Thread Prasanna Santhanam
> On July 26, 2013, 8:21 a.m., Prasanna Santhanam wrote: > > I've changed the library method to only accept delete by id last night. Can > > you reuse that? We need to keep the delete operations consistent across the > > integration library. > > If a custom dele

Re: unanswerred questions 15-21 juli

2013-07-26 Thread Prasanna Santhanam
On Fri, Jul 26, 2013 at 01:02:49PM +0200, Daan Hoogland wrote: > Do you want me to include links to those four messages or do you > state that there are more, that I have missed? Sorry - I mean the links/subjects to those emails so I can find them in my mail client. -- Pr

Re: [DISCUSS] Upgrade path to ACS 4.2 from CCP

2013-07-26 Thread Prasanna Santhanam
5to306-cleanup.sql setup/db/db/schema-305to306.sql setup/db/db/schema-306to307.sql setup/db/db/vmwaretmplinsert-304to305.sql -- Prasanna., Powered by BigRock.com

BuiltIn appliance for use in integration tests

2013-07-26 Thread Prasanna Santhanam
ill testing the template at the moment but wanted to bring this up in case anyone sees problems with this approach? -- Prasanna., Powered by BigRock.com

Re: Commit broke vm_data for Xen

2013-07-26 Thread Prasanna Santhanam
ters. > > This does it all in one go, processed on the router. > > > > BUG-ID: CLOUDSTACK-3163 > > Tested-by: Wido > > Signed-off-by: Marcus Sorensen 1374695897 > > -0600 > > > > I would bring the file back now. > > > > Thanks. > > > > --Sheng -- Prasanna., Powered by BigRock.com

Re: Review Request 12948: Affinity Groups: Adding new test cases

2013-07-26 Thread Prasanna Santhanam
where the test requires multiple hosts to be executed successfully. Can you please remove that tag from the tests? test/integration/component/test_affinity_groups.py <https://reviews.apache.org/r/12948/#comment47793> wait should not be required. - Prasanna

Re: [DISCUSS] vlan uri format change

2013-07-26 Thread Prasanna Santhanam
easier to fix the code to use > >> Nicira NVP and other sdn networks whereever vlans are used now. Doing this > >> will result in a patch that is a subset of my earlier patch. It touches a > >> lot of core code and tests but has been tested thoroughly for VPC > >> gateways > >> with both vlans and Nicira NVP. > >> > >> regards, > >> Daan > >> > >> > > -- Prasanna., Powered by BigRock.com

Re: Review Request 11900: Fix-all-occurrences-of-account.account-from-test_netscaler_configs.py

2013-07-26 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11900/#review24011 --- Discarding in favour of CLOUDSTACK-3075 - Prasanna Santhanam On

Re: Review Request 12972: Update CitrixResourceBase to use new vmdata programming method

2013-07-27 Thread Prasanna Santhanam
o this probably is an existing bug not a result of the python plugin. The python script also looks fine save for the print statements in the plugin which could be put into SMlog in the Xen host using utils.SMlog for logging. Could you rename it to vmdata.py since this works for both xen

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-07-27 Thread Prasanna Santhanam
, Sanjay Tripathi, and Prasanna Santhanam. Changes --- including dev and QA for review Repository: cloudstack-git Description --- Adding resource limit tests related to Memory. Changes suggested in CPU resource limit tests which are applicable here are incorporated too. Updated

Re: Do we delete an account after all users are deleted?

2013-07-27 Thread Prasanna Santhanam
have clarity at the time it was written. I removed all skips in the tests so we'll find those invalid ones now. -- Prasanna., Powered by BigRock.com

Re: Do we delete an account after all users are deleted?

2013-07-27 Thread Prasanna Santhanam
On Sat, Jul 27, 2013 at 01:34:25PM +, Sowmya Krishnan wrote: > > > > -Original Message- > > From: Prasanna Santhanam [mailto:t...@apache.org] > > Sent: Saturday, July 27, 2013 6:55 PM > > To: dev@cloudstack.apache.org > > Subject: Re: Do we d

Re: BuiltIn appliance for use in integration tests

2013-07-28 Thread Prasanna Santhanam
On Fri, Jul 26, 2013 at 09:24:18AM -0400, Chip Childers wrote: > On Fri, Jul 26, 2013 at 06:02:04PM +0530, Prasanna Santhanam wrote: > > Almost all the integration tests use the built-in CentOS template in > > the deployment for their test scenarios. Sometimes this is less > >

Re: Do we delete an account after all users are deleted?

2013-07-28 Thread Prasanna Santhanam
RE: Do we delete an account after all users are deleted? > > > > > > > > > > > > > -Original Message- > > > > From: Sowmya Krishnan [mailto:sowmya.krish...@citrix.com] > > > > Sent: Saturday, July 27, 2013 7:05 PM > > >

Re: New Components on JIRA

2013-07-28 Thread Prasanna Santhanam
On Wed, Jul 24, 2013 at 10:15:12AM +, Ram Ganesh wrote: > > > > > > Prasanna, > > > > > > How about - automation-product and automation-script components? > > > Automation-product for all product bugs discovered by the automation > > >

Re: Review Request 13009: CLOUDSTACK-3744 - Accounts not cleaned up

2013-07-28 Thread Prasanna Santhanam
do not consume physical/virtual resources, we should defer them from the cleanup process. - Prasanna Santhanam On July 29, 2013, 4:12 a.m., Sowmya Krishnan wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 13010: rb-jenkins-integration test patch

2013-07-28 Thread Prasanna Santhanam
Trippaers and Prasanna Santhanam. Repository: cloudstack-git Description (updated) --- Test patch request for rb-jenkins integration. Do not merge https://github.com/CloudStack-extras/reviewboard-tools/issues/2 Diffs - INSTALL.md b0e1a761 Diff: https://reviews.apache.org/r/13010/diff

Review Request 13010: rb-jenkins-integration test patch

2013-07-28 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13010/ --- Review request for cloudstack, Hugo Trippaers and Prasanna Santhanam

Re: Review Request 13009: CLOUDSTACK-3744 - Accounts not cleaned up

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13009/#review24108 --- Ship it! Ship It! - Prasanna Santhanam On July 29, 2013, 9:31

Re: Review Request 13014: CLOUDSTACK-3841: Fixed TestDeleteSecurityGroup.test_01_delete_security_grp_running_vm

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13014/#review24126 --- Ship it! Ship It! - Prasanna Santhanam On July 29, 2013, 12:39

Re: Review Request 12942: CLOUDSTACK-3454: Fix test_portable_publicip

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12942/#review24129 --- Ship it! - Prasanna Santhanam On July 25, 2013, 12:50 p.m

Re: Review Request 13025: Bugfix CLOUDSTACK-3915: Creating is mis-spelled in CreateFirewallRuleCmd

2013-07-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13025/#review24208 --- Ship it! - Prasanna Santhanam On July 29, 2013, 7:20 p.m

Re: Review Request 12963: Fix some simple typos in ConfigurationManager

2013-07-29 Thread Prasanna Santhanam
) commit c78ed39b2f53a3544b93515e6bb4889eef8c5888 (4.2) - Prasanna Santhanam On July 26, 2013, 9:07 a.m., Thomas O'Dowd wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Open up ports beyond 80/443/8080 for downloading templates

2013-07-29 Thread Prasanna Santhanam
3/cloudian and one can get direct http access to the image on the object store. The code in validateUrl assumes that the url check goes always to the SSVM on which only 80/443/8080 are open by default. -- Prasanna., Powered by BigRock.com

Re: S3 Object Store testing

2013-07-30 Thread Prasanna Santhanam
al Message- > From: Thomas O'Dowd [mailto:tpod...@cloudian.com] > Sent: Tuesday, July 30, 2013 2:25 PM > To: Sanjeev Neelarapu > Cc: Cloudstack DEV > Subject: S3 Object Store testing > > Sanjeev, > > I was talking to Prasanna earlier regarding testing various par

Re: Noisy ASFBot

2013-07-30 Thread Prasanna Santhanam
On Sat, Jul 20, 2013 at 12:15:33PM +0200, Daan Hoogland wrote: > I agree, > > @Prasanna, can we make ignore default and /activate ASFBot? That would have > my preference. > I got annoyed about this today myself. infrabot: Prasanna Santhanam [Created] (INFRA-6605) Turn off g

Re: Review Request 13001: Automation: Adding test cases for CPU limits from Limit Resources.

2013-07-30 Thread Prasanna Santhanam
Shilamkar, sailaja mada, Sanjay Tripathi, and Prasanna Santhanam. Changes --- including dev/qa owners for first review. Repository: cloudstack-git Description --- Adding test cases for CPU limits from Limit Resources. Marvin changes have to be picked up from Memory Limits patch (https

Re: Review Request 13055: Fix to select default vpc offering with internal lb

2013-07-30 Thread Prasanna Santhanam
t. How about list the vpcoffering based on the name? and then use the returned uuid. - Prasanna Santhanam On July 30, 2013, 6:51 a.m., SrikanteswaraRao Talluri wrote: > > --- > This is an automatically generated e-mail. To

Re: Review Request 13056: CLOUDSTACK-3168: Fix test_reboot_router.py to ssh using public IP

2013-07-30 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13056/#review24237 --- Ship it! - Prasanna Santhanam On July 30, 2013, 7:17 a.m

<    2   3   4   5   6   7   8   9   10   11   >