RE: [ANNOUNCE] New committer: Harikrishna Patnala

2015-03-10 Thread Koushik Das
Congrats Hari! -Original Message- From: Rajani Karuturi [mailto:raj...@apache.org] Sent: Tuesday, 10 March 2015 11:00 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] New committer: Harikrishna Patnala The Project Management Committee (PMC) for Apache CloudStack has asked Harikrishna

[PROPOSAL] Allow configuring system VMs to use local storage at zone level

2015-03-05 Thread Koushik Das
Based on the discussion [1], I have created a FS [2] and tracking bug [3]. Please review and provide your comments. -Koushik [1] http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201502.mbox/%3cad8a3382-cfc6-4fd3-92de-df61f5a8e...@citrix.com%3e [2]

Re: [DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-03-02 Thread Koushik Das
:49, Koushik Das koushik@citrix.commailto:koushik@citrix.com wrote: On 01-Mar-2015, at 12:59 AM, Paul Angus paul.an...@shapeblue.commailto:paul.an...@shapeblue.com wrote: Hi Wilder, The guys are talking about enabling system VMs to use local storage on a zone by zone basis

Re: [DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-03-01 Thread Koushik Das
. Cheers, Wilder Sent from my iPhone On 27 Feb 2015, at 14:21, Rohit Yadav rohit.ya...@shapeblue.com wrote: Sounds good, go for it! On 27-Feb-2015, at 4:55 pm, Koushik Das koushik@citrix.com wrote: Currently it is not possible to configure system VMs to use local storage

[DISCUSS] Allow configuring system VMs to use local storage at zone level

2015-02-27 Thread Koushik Das
Currently it is not possible to configure system VMs to use local storage on a per zone basis. Since it is a global config (system.vm.use.local.storage), either it is enabled for all zones or none. The idea is to make it a zone level config. The current zone level setting of 'enabling local

RE: long running async job

2015-02-05 Thread Koushik Das
There is another config 'job.cancel.threshold.minutes' (defaulted to 60 mins) based on which async job is forcefully cancelled. Can you check if this is cancelling the job? Also if you want to debug - checkout the GC code present in AsyncJobManagerImpl.java -Koushik -Original

Re: merge from 4.5 to master

2014-12-08 Thread Koushik Das
I tried a merge just now and it failed for tools/appliance/build.sh On 09-Dec-2014, at 11:23 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Sounds good - thanks On Mon, Dec 8, 2014 at 10:52 PM, Rajani Karuturi raj...@apache.org wrote: yes ~Rajani On Tue, Dec 9, 2014 at

Re: Review Request 28700: Attach volume events are not correctly shown in events.

2014-12-04 Thread Koushik Das
- 7ef0f5507d3b0debe4cfc7fb3bcada39c069e33f - Koushik Das On Dec. 4, 2014, 10:02 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28700

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread Koushik Das
://reviews.apache.org/r/27868/#comment102368 What operation, which hypervisor and VM? Also the info log is not required as the exception message will be present in logs. - Koushik Das On Nov. 11, 2014, 2:19 p.m., bharat kumar wrote

Re: Review Request 27904: cloudstackoperations like deployvm, deleteNW are failing if CS fail to contact rabbit mq server

2014-11-11 Thread Koushik Das
- f1361796bf1fc73a65c47583dd0a5d6ef2dc3c27 - Koushik Das On Nov. 12, 2014, 4:13 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27904

Re: [ANNOUNCE] New PMC Member: Rajani Karuturi

2014-10-17 Thread Koushik Das
Congrats Rajani! On 17-Oct-2014, at 3:51 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: The Project Management Committee (PMC) for Apache CloudStack are pleased to announce that Rajani Karuturi has accepted our invitation to join the PMC. Please join me in congratulating her. On behalf

Re: Review Request 26168: There are new VM State Machine changes introduced which were missed to capture the usage events

2014-10-09 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26168/#review55964 --- - Koushik Das On Oct. 9, 2014, 6:26 a.m., Damodar Reddy Talakanti

RE: Review Request 26168: There are new VM State Machine changes introduced which were missed to capture the usage events

2014-10-09 Thread Koushik Das
Somehow the review comments are not showing? 1. Would it make sense to make the pre/post method signature identical? 2. In StateEntry.addTransition(), do you need E and S as separate parameters? From: Koushik Das [mailto:nore...@reviews.apache.org] On Behalf Of Koushik Das Sent

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread Koushik Das
://reviews.apache.org/r/25430/#comment91419 In the prvious line, service offering is getting set, why set it again? Since there is a way to pass offering in ctor, is an explicit setter really required? - Koushik Das On Sept. 8, 2014, 9:16 a.m., bharat kumar wrote

RE: simulator run

2014-09-04 Thread Koushik Das
Are the tests present under test/integration/smoke/misc also getting executed? If so please exclude them for now. The tests under /misc folder is based on the new simulator changes to test failures from agent layer. These tests can cause interference in other tests. There are 2 options to make

RE: simulator run

2014-09-04 Thread Koushik Das
-2.7 -v --with-marvin --marvin-config=setup/dev/advanced.cfg --with-xunit --xunit-file=xunit.xml -a tags=advanced,required_hardware=false --zone=Sandbox-simulator --hypervisor=simulator -w test/integration/smoke Cheers, Hugo On 4 sep. 2014, at 13:32, Koushik Das koushik@citrix.com wrote

RE: simulator run

2014-09-04 Thread Koushik Das
of getting a list of tests that are valid for execution against the simulator for 4.3, 4.4 and master? Thanks, Ian On 4 September 2014 13:03, Koushik Das koushik@citrix.com wrote: Ok, in that case the tests under /misc won't get executed. Looks like there are some other issues

RE: Unable to connect to management server on recent builds

2014-09-02 Thread Koushik Das
This is the erring commit commit c730bc3491f8b684c5ae51e0bff54cf7577cc246 Author: Rohit Yadav rohit.ya...@shapeblue.com Date: Mon Sep 1 21:14:13 2014 +0200 server: Add event bus bean as commented xml in META-INF core This adds a spring bean xml to have EventBus for ACS, but the

RE: 4.5 RM

2014-08-20 Thread Koushik Das
I see a lot of discussion on enabling CI. Agree that it is important but CI in itself is not going to solve the quality issue. The other important aspect is the effectiveness of tests that get executed as part of CI. Unless there is a proper set of automated tests (which is kept up-to-date) for

Re: Review Request 24646: CLOUDSTACK-7337: Volume state remains in allocated after volume creation faliure from snapshot

2014-08-20 Thread Koushik Das
- Koushik Das On Aug. 20, 2014, 11:38 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24646/ --- (Updated

Re: Review Request 24152: CLOUDSTACK-7182: NPE while trying to deploy VMs in parallel in isolated network

2014-08-17 Thread Koushik Das
- Koushik Das On Aug. 5, 2014, 5:16 a.m., Koushik Das wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24152/ --- (Updated Aug. 5

Re: Review Request 24152: CLOUDSTACK-7182: NPE while trying to deploy VMs in parallel in isolated network

2014-08-17 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24152/#review50653 --- On Aug. 5, 2014, 5:16 a.m., Koushik Das wrote

Re: Review Request 24549: CLOUDSTACK-7305: hypervisor type parameter is mandatory when deploying VM using ISO

2014-08-12 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24549/#review50302 --- Ship it! Ship It! - Koushik Das On Aug. 12, 2014, 11:03 a.m

RE: [DB-CHANGE] Infrastructure tab fails to load with db exception

2014-08-06 Thread Koushik Das
Thanks Saksham. This fixed the initial issue. But I noticed a new one, after destroying the last VR if you select the infra view it again results in exception. Not sure if anything else needs to be fixed. -Original Message- From: Saksham Srivastava [mailto:saksham.srivast...@citrix.com]

RE: NO commits to master

2014-08-05 Thread Koushik Das
Why not cut out a new develop branch after the 24/48 hrs window and discard the current develop branch. That is much more simpler. -Original Message- From: Rohit Yadav [mailto:rohit.ya...@shapeblue.com] Sent: Tuesday, 5 August 2014 3:20 PM To: dev@cloudstack.apache.org Subject: Re: NO

Re: Review Request 24152: CLOUDSTACK-7182: NPE while trying to deploy VMs in parallel in isolated network

2014-08-04 Thread Koushik Das
simulator: 1. created vpc, VR gets started as part of this 2. created network in the vpc 3. deployed vm in network created in step#2 Thanks, Koushik Das

Re: Review Request 24152: CLOUDSTACK-7182: NPE while trying to deploy VMs in parallel in isolated network

2014-08-04 Thread Koushik Das
sometime VR gets stopped and network goes to allocated state. At this point started the VR. Also repeated same for VPC. 4. Ran #3 using XS setup as well Thanks, Koushik Das

Re: [JENKINS] simulator build

2014-08-01 Thread Koushik Das
Thanks Hugo for setting it up. On 01-Aug-2014, at 7:21 PM, Hugo Trippaers h...@trippaers.nl wrote: hey all, I’ve managed to get the simulator build running again to jenkins.b.o. Its running as a single job called simulator-singlerun and builds, runs and tests cloudstack using the

Review Request 24152: CLOUDSTACK-7182: NPE while trying to deploy VMs in parallel in isolated network

2014-07-31 Thread Koushik Das
. deployed vm in network created in step#2 Thanks, Koushik Das

RE: Simulator on master is not building

2014-07-30 Thread Koushik Das
The simulator code should be kept up to date with any changes done in the core product. If there are any interface changes, necessary changes should be made in simulator plugin as well along with other hypervisor plugins. To build simulator code use refer to [1]. -Koushik [1]

RE: Simulator on master is not building

2014-07-30 Thread Koushik Das
this is fixed it needs to be tested again. Also there needs to be a consensus to include it in default profile. -Original Message- From: Edison Su Sent: Wednesday, 30 July 2014 11:30 PM To: dev@cloudstack.apache.org; Koushik Das Subject: RE: Simulator on master is not building +1, include

RE: Simulator on master is not building

2014-07-30 Thread Koushik Das
if there are any issues. -Koushik -Original Message- From: Anthony Xu Sent: Wednesday, 30 July 2014 11:37 PM To: Koushik Das; dev@cloudstack.apache.org Subject: RE: Simulator on master is not building Hi Koushik, fixed the build for simulator. I just found out Simulator use its own VOes for some VOes

RE: [ANNOUNCE] Rajani Karuturi as committer

2014-07-22 Thread Koushik Das
Congrats Rajani! -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Tuesday, 22 July 2014 11:53 AM To: CloudStack Dev Subject: [ANNOUNCE] Rajani Karuturi as committer The Project Management Committee (PMC) for Apache CloudStack has asked Rajani Karuturi to

Re: Question about Global Settings

2014-07-13 Thread Koushik Das
I don't see any issues using vendor/plugin specific settings as long as its use is limited to the plugin code. But there is limitations in the framework related to the way plugin specific stuff is handled. Ideally you would want to create db schema/configurations related to a plugin only if it

RE: Coverity Scan Report: July 11 2014

2014-07-13 Thread Koushik Das
Should commits be reverted if they are not Findbugs compliant? Otherwise defect density would never come down. -Original Message- From: Santhosh Edukulla [mailto:santhosh.eduku...@citrix.com] Sent: Friday, 11 July 2014 8:59 PM To: dev@cloudstack.apache.org Subject: Coverity Scan Report:

RE: [ANNOUNCE] Will Stevens as committer

2014-07-04 Thread Koushik Das
Congrats Will! -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Friday, 4 July 2014 2:13 AM To: cloudstack Subject: [ANNOUNCE] Will Stevens as committer The Project Management Committee (PMC) for Apache CloudStack has asked Will Stevens to become a committer and we are

RE: [ANNOUNCE] New PMC Member: Mark Hinkle

2014-07-04 Thread Koushik Das
Congrats Mark! -Original Message- From: John Burwell [mailto:jburw...@basho.com] Sent: Thursday, 3 July 2014 11:21 PM To: dev@cloudstack.apache.org Cc: market...@cloudstack.apache.org Subject: [ANNOUNCE] New PMC Member: Mark Hinkle The Project Management Committee (PMC) for Apache

Re: Review Request 23192: Adding Readme and run checkbox at the end of the installation. Also installing mysql connector

2014-07-03 Thread Koushik Das
On July 3, 2014, 2:18 p.m., Chip Childers wrote: -1 to this commit until legal policy issues are resolved Reverted the commit commit 4607c26949b6b925fdc0eb2a5253d536dd333439 Author: Koushik Das kous...@apache.org Date: Fri Jul 4 10:04:04 2014 +0530 Revert CLOUDSTACK-6834 : 3. Some

Re: Review Request 23195: CLOUDSTACK-6873: Moving the tests using SimulatorMock to different folder so that they can be run separately and serially

2014-07-03 Thread Koushik Das
On July 1, 2014, 10:36 a.m., Koushik Das wrote: Why are tests using simulator mock moved to 'maint' folder? If you remove test_deploy_vm.py make sure that the following test is moved to another file. This is meant to test list VM with multiple 'ids'. @attr(tags = ['advanced

Re: Review Request 23192: Adding Readme and run checkbox at the end of the installation. Also installing mysql connector

2014-07-03 Thread Koushik Das
://reviews.apache.org/r/23192/ --- (Updated July 3, 2014, 11:58 a.m.) Review request for cloudstack, Abhinandan Prateek, Koushik Das, and Hugo Trippaers. Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-6834 https

Re: Review Request 23192: Adding Readme and run checkbox at the end of the installation. Also installing mysql connector

2014-07-03 Thread Koushik Das
- Koushik Das On July 3, 2014, 11:58 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23192

Re: Review Request 23192: Adding Readme and run checkbox at the end of the installation. Also installing mysql connector

2014-07-03 Thread Koushik Das
. Also a lot of folks adopted the common practice to put the commit id in a comment after committing a review. That makes it easier to trace. Cheers, Hugo On 3 jul. 2014, at 14:29, Koushik Das koushik@citrix.com wrote: On July 3, 2014, 11:58 a.m., Hugo Trippaers wrote: I'm

Re: Review Request 23192: Adding Readme and run checkbox at the end of the installation. Also installing mysql connector

2014-07-02 Thread Koushik Das
/23192/#comment82951 License header? - Koushik Das On July 2, 2014, 12:29 p.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23192

Re: Review Request 23195: CLOUDSTACK-6873: Moving the tests using SimulatorMock to different folder so that they can be run separately and serially

2014-07-01 Thread Koushik Das
remove test_deploy_vm.py make sure that the following test is moved to another file. This is meant to test list VM with multiple 'ids'. @attr(tags = ['advanced', 'basic', 'sg'], required_hardware=false) def test_deploy_vm_multiple(self): - Koushik Das On July 1, 2014, 10:12 a.m

Re: Review Request 23195: CLOUDSTACK-6873: Moving the tests using SimulatorMock to different folder so that they can be run separately and serially

2014-07-01 Thread Koushik Das
On July 1, 2014, 10:36 a.m., Koushik Das wrote: Why are tests using simulator mock moved to 'maint' folder? If you remove test_deploy_vm.py make sure that the following test is moved to another file. This is meant to test list VM with multiple 'ids'. @attr(tags = ['advanced

RE: [ANNOUNCE] Santhosh Edukulla as a committer...

2014-06-30 Thread Koushik Das
Congrats Santhosh! -Original Message- From: Alex Huang [mailto:alex.hu...@citrix.com] Sent: Monday, 30 June 2014 9:55 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Santhosh Edukulla as a committer... Hi All, The Project Management Committee (PMC) for Apache CloudStack has asked

RE: What to modify to increase instance password length?

2014-06-25 Thread Koushik Das
Shouldn't the code handle the scenario if vm.password.length is missing from db? In this case the upgrade script will put it but if someone accidentally removes it. -Original Message- From: Nux! [mailto:n...@li.nux.ro] Sent: Thursday, 26 June 2014 2:55 AM To: dev@cloudstack.apache.org

RE: What to modify to increase instance password length?

2014-06-25 Thread Koushik Das
This is a cleaner approach to handle new config parameters https://cwiki.apache.org/confluence/display/CLOUDSTACK/Configuration -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Thursday, 26 June 2014 10:40 AM To: dev@cloudstack.apache.org Subject: RE: What

RE: S3 use with simulator

2014-06-23 Thread Koushik Das
store has to be associated with a data center On Jun 20, 2014, at 7:07 AM, Koushik Das koushik@citrix.com wrote: As per the code the following are the supported image store provider types // constants for provider names static final String NFS_IMAGE = NFS; static final String

Re: S3 use with simulator

2014-06-23 Thread Koushik Das
| +---+--+ protocol = http providername = S3 scope = REGION url = http://localhost:9081/riak-cs Apparently successfully, even though the removal of the NFS store entirely is a bit worrisome. On Jun 23, 2014, at 4:32 AM, Koushik Das koushik@citrix.com wrote: The error message

RE: S3 use with simulator

2014-06-23 Thread Koushik Das
: Monday, 23 June 2014 4:40 PM To: dev@cloudstack.apache.org Subject: Re: S3 use with simulator On Jun 23, 2014, at 12:51 PM, Koushik Das koushik@citrix.com wrote: inline On 23-Jun-2014, at 4:05 PM, Sebastien Goasguen run...@gmail.com wrote: Hi Koushik, thanks for the hint, I checked

RE: S3 use with simulator

2014-06-23 Thread Koushik Das
, 23 June 2014 5:47 PM To: dev@cloudstack.apache.org Subject: Re: S3 use with simulator On Jun 23, 2014, at 7:14 AM, Koushik Das koushik@citrix.com wrote: This is the request http://localhost:8096/api?command=updateCloudToUseObjectStoreurl=http ://localhost:1234/testname=testprovider

Re: Review Request 22865: [Automation] test_01_primary_storage_iscsi failed on test_primary_storage.py - Wrong iscsi path format - it should be /targetIQN/LUN

2014-06-23 Thread Koushik Das
master - d3ffb7a5659bf988d00f607ee14cc07741b76c4a - Koushik Das On June 23, 2014, 10:54 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22865

RE: S3 use with simulator

2014-06-20 Thread Koushik Das
As per the code the following are the supported image store provider types // constants for provider names static final String NFS_IMAGE = NFS; static final String S3_IMAGE = S3; static final String SWIFT_IMAGE = Swift; static final String SAMPLE_IMAGE = Sample; static

Re: Is VM snapshot supported in KVM ?

2014-06-19 Thread Koushik Das
VM snapshot is disabled for KVM. Check hypervisor_capabilities table. Not sure what was the reason for keeping it disabled. On 20-Jun-2014, at 10:16 AM, Rayees Namathponnan rayees.namathpon...@citrix.com wrote: Hi All, Is VM snapshot supported in KVM, below doc says its supported in KVM,

Re: Review Request 22510: [Windows] Can not create Template from ROOT snapshot. Fixed the same by adding pathSeparator to the DataStoreTO.java

2014-06-18 Thread Koushik Das
- Koushik Das On June 18, 2014, 9:41 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22510

RE: [ACS431][QUESTION] db upgrace

2014-06-17 Thread Koushik Das
4.3.1 - 4.4.0 upgrade also needs to be handled. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, 17 June 2014 9:16 PM To: dev Subject: [ACS431][QUESTION] db upgrace In my simple mind it seems we only need to create a Upgrade430to431.java file and

Re: Review Request 22510: [Windows] Can not create Template from ROOT snapshot. Fixed the same by adding pathSeparator to the DataStoreTO.java

2014-06-16 Thread Koushik Das
/src/com/cloud/hypervisor/xenserver/resource/XenServerStorageProcessor.java https://reviews.apache.org/r/22510/#comment80695 Is the intent to replace all occurrences of File.Separator? I still see this one left out. Also please open a bug to track changes to other HVs. - Koushik Das

Re: Review Request 22453: [windows]Adding prot fields to the database creation wizard

2014-06-12 Thread Koushik Das
- Koushik Das On June 12, 2014, 6:58 a.m., Damodar Reddy Talakanti wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22453

Re: Review Request 22454: Fixed few coverity reported issues

2014-06-12 Thread Koushik Das
-forward - 32a7d5d387804374c38ed568e2dcd1e9c543873c - Koushik Das On June 11, 2014, 9:18 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22454

Re: Review Request 22454: Fixed few coverity reported issues

2014-06-11 Thread Koushik Das
/#comment80324 Method signature is listByPodIdDcId(long, long). Parameter podId is of type Long, so it needs to be passed correctly. - Koushik Das On June 11, 2014, 9:18 a.m., Rajani Karuturi wrote: --- This is an automatically generated e

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Koushik Das
/VirtualMachineManagerImpl.java https://reviews.apache.org/r/22364/#comment79921 These methods are getting used by the job framework. Check handleVmWorkJob() method in the same java file. - Koushik Das On June 10, 2014, 4:06 a.m., Rajani Karuturi wrote

Re: Review Request 22364: Fixed 26 issues reported by coverity

2014-06-10 Thread Koushik Das
-forward - 390e498dc58a89f0b060c50e5b796061bc97342e - Koushik Das On June 10, 2014, 8:24 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22364

Re: [ACS44] 112 unpicked cherries in 4.4-forward. why?

2014-06-09 Thread Koushik Das
and decide what to do about them: blame 617826d16b4d5220bb3b51ed511b3c065d0e8926 'Koushik Das kous...@apache.org' - 'Daan Hoogland d...@onecht.net' - 'CLOUDSTACK-6445: Simulator enhancements Refer FS - https://cwiki.apache.org/confluence/display/CLOUDSTACK/Simulator+enha ncements ' blame

Cherry-pick request

2014-06-05 Thread Koushik Das
Hi Daan, Please cherry-pick the following commit to 4.4. Commit 025c143ac792905557794751118cf5319592 in cloudstack's branch refs/heads/4.4-forward from [~koushikd] [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=025c143 ] Thanks, Koushik

RE: Cherry-pick request

2014-06-05 Thread Koushik Das
Thanks, will add [ACS44] for any such request in future. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thursday, 5 June 2014 7:23 PM To: dev Subject: Re: Cherry-pick request On Thu, Jun 5, 2014 at 10:37 AM, Koushik Das koushik@citrix.com wrote

Re: Review Request 22194: Fixed Resource leak reported by coverity

2014-06-03 Thread Koushik Das
-forward - d511847cfedad5478d1b4087c8f97be2c5bf3cc8 - Koushik Das On June 3, 2014, 10:42 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22194

Re: Review Request 22093: VPC's VR missing public NIC eth1

2014-06-02 Thread Koushik Das
this go to 4.3-4.4 upgrade? - Koushik Das On May 30, 2014, 9 p.m., daan Hoogland wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22093

RE: [ANNOUNCE] Amogh Vasekar as committer

2014-06-02 Thread Koushik Das
Congrats Amogh! -Original Message- From: John Kinsella [mailto:j...@stratosec.co] Sent: Monday, 2 June 2014 11:44 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Amogh Vasekar as committer The Project Management Committee (PMC) for Apache CloudStack has asked Amogh Vasekar to

Re: Regarding local storage pool allocator

2014-05-28 Thread Koushik Das
Earlier there was no scenarios where deployment plan had host id specified for a ROOT disk. If new scenarios like storage migration needs them, the condition can be relaxed to remove data disk type check. On 28-May-2014, at 2:18 PM, Wido den Hollander w...@widodh.nl wrote: On 05/28/2014

Re: Problem with mailing list....

2014-05-14 Thread Koushik Das
https://blogs.apache.org/infra/entry/mail_outage On 14-May-2014, at 5:58 AM, Min Chen min.c...@citrix.com wrote: I had the same issue, my email sent a while back is still not sent to community yet. -min On 5/13/14 4:57 PM, Alex Huang alex.hu...@citrix.com wrote: I've find some of my

Re: Review Request 21198: Added Fix for CLOUDSTACK-6648

2014-05-13 Thread Koushik Das
in master, please create a new patch - Koushik Das On May 13, 2014, 5:19 a.m., Santhosh Edukulla wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21198

Re: seeing Unknown parameters : ctxdetails for addResourceDetail/removeResourceDetail

2014-05-13 Thread Koushik Das
I am seeing the same for many commands. So this is not specific to add/removeResourceDetail 2014-05-09 18:17:39,477 WARN [c.c.a.d.ParamGenericValidationWorker] (API-Job-Executor-24:job-28 ctx-6eaab5d7) Received unknown parameters for command updateNetworkServiceProvider. Unknown parameters :

Re: Review Request 21197: Added port issue and user apiclient fix issue

2014-05-13 Thread Koushik Das
master - e69a94b61ee160532d660165e8b715f0f3f83ea9 - Koushik Das On May 8, 2014, 9:49 a.m., Santhosh Edukulla wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21197

Re: How is this working?

2014-05-05 Thread Koushik Das
the storage_pool table's available_bytes column to be used_bytes. It looks like this code you reference was missed. On Mon, May 5, 2014 at 4:35 AM, Koushik Das koushik@citrix.com wrote: I came across this code snippet in hostConnect() method in DefaultHostListener.java. Look at the line where

Re: listVirtualMachine API With Name Parameter Set

2014-05-02 Thread Koushik Das
The name parameter is used in the sql query filter as where name like name, so more than one can be returned. On 01-May-2014, at 8:16 PM, Matt Spurlin matt.spur...@appcore.com wrote: Hi all, I am looking at CLOUDSTACK-6555https://issues.apache.org/jira/browse/CLOUDSTACK-6555. I am wondering

Re: Review Request 20703: Fixed CLOUDSTACK-6509 Cannot import multiple LDAP/AD users into a cloudstack account

2014-04-29 Thread Koushik Das
, please create a new one. - Koushik Das On April 28, 2014, 10:23 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20703

Re: HypervisorType - possible feature idea

2014-04-24 Thread Koushik Das
Ideally the API and orchestration layers shouldn't have any HV specific code. You may have already seen that there are places in the code which checks if an operation is supported based on HV type like below. if (Vmware) throw ex All these checks should rely on hypervisor_capabilities table

Re: [PROPOSAL] Simulator enhancements

2014-04-23 Thread Koushik Das
This is a document to help write tests using the new simulator mock framework. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Writing+tests+leveraging+the+simulator+enhancements On 18-Apr-2014, at 5:09 PM, Koushik Das koushik@citrix.com wrote: I have added this to master as part

Re: [ACS44] 4.4-forward branch creation accident

2014-04-22 Thread Koushik Das
Daan, I am not seeing the commits that went to the previously created 4.4-forward branch in the new 4.4-forward branch. For e.g. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c8f2bf2 Are you going to merge all of them into 4.4-forward? -Koushik On 23-Apr-2014, at 2:53 AM, Mike

Re: [PROPOSAL] Simulator enhancements

2014-04-18 Thread Koushik Das
using simulator. On 01-Apr-2014, at 5:47 PM, Koushik Das koushik@citrix.com wrote: The current simulator provides a way to test various Cloudstack features without using actual resources. But its current capabilities are limited since it only allows success response for all agent

Re: Anybody experienced test failure on master?

2014-04-18 Thread Koushik Das
'mvm clean install' worked for me. Are you trying noredist? On 18-Apr-2014, at 6:03 AM, Min Chen min.c...@citrix.com wrote: Hi there, Did anybody experience test failure on master branch? I got this: T E S T S --- Running

Re: Review Request 18768: CLOUDSTACK-6151: Local data disk with tag goes to the wrong local storage pool

2014-04-16 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18768/#review40542 --- Ship it! Ship It! - Koushik Das On April 16, 2014, 10:14 a.m

Re: Review Request 20250: Fixed errors with simulator VM snapshot delete and revert

2014-04-15 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20250/#review40378 --- Ship it! Ship It! - Koushik Das On April 11, 2014, 2:37 a.m

RE: Unable to start instance on 4.3 using Simulator

2014-04-15 Thread Koushik Das
In 4.3 you need to set global config router.version.check to false to workaround this issue in simulator. This is fixed in 4.4/master by commit 38b8677e01fe1ec89df43abe9fac26295597080f. -Koushik -Original Message- From: David Grizzanti [mailto:david.grizza...@sungard.com] Sent:

Re: Coding Standards Questions

2014-04-13 Thread Koushik Das
On 12-Apr-2014, at 2:14 AM, Alex Hitchins a...@alexhitchins.com wrote: Daan, Are you referring to keeping line lengths up to 80 characters? Sorry - tired eyes. My thoughts were more that in a function there should only be one return statement rather than many, all nested in layers of

Re: Remove Mgmt Server

2014-04-10 Thread Koushik Das
+1 to the feature. On 11-Apr-2014, at 6:05 AM, Michael Phillips mphilli7...@hotmail.com wrote: I can create a JIRA request for this, but how many +1's do we need before it gets added to the To Do list? From: alex.hu...@citrix.com To: dev@cloudstack.apache.org Subject: RE: Remove Mgmt

Re: Validating check-ins for your local changes, using Simulator

2014-04-03 Thread Koushik Das
Thanks for updating the wiki Santhosh. Also for any new feature adding agent commands, the simulator needs to be fixed as well. The integration tests added for new features should be run against simulator as well to ensure it is not broken. -Koushik On 03-Apr-2014, at 2:16 AM, Santhosh

Re: [PROPOSAL] Simulator enhancements

2014-04-02 Thread Koushik Das
control the agent state as well? Say I want to test VM HA, Can I be able to set the host status to be disconnected and then verify that VM's on the host gets migrated? On 01/04/14 5:47 PM, Koushik Das koushik@citrix.com wrote: The current simulator provides a way to test various

[PROPOSAL] Simulator enhancements

2014-04-01 Thread Koushik Das
The current simulator provides a way to test various Cloudstack features without using actual resources. But its current capabilities are limited since it only allows success response for all agent commands and some ability to introduce delays. I am proposing some enhancements to the simulator

Re: [PROPOSAL] Simulator enhancements

2014-04-01 Thread Koushik Das
for a specific storage/host? For example, I have 5 storages in my cluster but I want to simulate failure for 3 of the storage pools. Is that possible? [Koushik] Good point. I will add storage pool also into the scope. -Original Message- From: Koushik Das [mailto:koushik@citrix.com

Re: Review Request 19272: simulator manager impl changes to handle SetMonitorServiceCommand

2014-03-20 Thread Koushik Das
- fc8cb754f0c6645bac5f35d8ad3d16e5e17a2d50 - Koushik Das On March 17, 2014, 5:33 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19272

Re: Review Request 18768: CLOUDSTACK-6151: Local data disk with tag goes to the wrong local storage pool

2014-03-20 Thread Koushik Das
? - Koushik Das On March 5, 2014, 11:17 a.m., Saksham Srivastava wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18768

Re: Review Request 19038: CLOUDSTACK-6090: Virtual Router Service Failure Alerting

2014-03-14 Thread Koushik Das
, f293c94bc0198f162c80ff807ca81459150354a5 - Koushik Das On March 14, 2014, 9:36 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19038

Re: Review Request 19022: List VM enhancement to support querying with multiple VM IDs

2014-03-13 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19022/#review37045 --- Ship it! Ship It! - Koushik Das On March 11, 2014, 6:29 a.m

Re: Review Request 19022: List VM enhancement to support querying with multiple VM IDs

2014-03-13 Thread Koushik Das
On March 13, 2014, 12:19 p.m., Koushik Das wrote: Ship It! Commit 5779292e93fe52a7fb5ec0abffe1e3f6b9c10121 in cloudstack's branch refs/heads/master from [~koushikd] [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=5779292 ] CLOUDSTACK-6052: List VM enhancement to support

Re: [20/31] git commit: updated refs/heads/distributedrouter to a8d43ba

2014-03-13 Thread Koushik Das
://git-wip-us.apache.org/repos/asf/cloudstack/diff/5779292e Branch: refs/heads/distributedrouter Commit: 5779292e93fe52a7fb5ec0abffe1e3f6b9c10121 Parents: bc4c805 Author: Koushik Das kous...@apache.org Authored: Tue Mar 11 11:49:13 2014 +0530 Committer: Koushik Das kous...@apache.org Committed

Re: Review Request 18902: CLOUDSTACK-6211: Xenserver - HA - SSVM fails to start due to running out of management Ip ranges when testing host down scenarios

2014-03-12 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18902/#review37031 --- Ship it! Ship It! - Koushik Das On March 7, 2014, 11:04 a.m

Review Request 19022: List VM enhancement to support querying with multiple VM IDs

2014-03-11 Thread Koushik Das
425aeb7 Diff: https://reviews.apache.org/r/19022/diff/ Testing --- Added integration test, also verified manually. Thanks, Koushik Das

Re: [PROPOSAL] List VM API enhancement

2014-03-11 Thread Koushik Das
of these little 'English' rules implied in using a programming language, like the use of adjectives. They matter in Westerns languages and unless we define a system wide set of different rules let's stick with English. kind regards, Daan Hoogland Op 13 feb. 2014 08:32 schreef Koushik Das koushik

<    2   3   4   5   6   7   8   9   >