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

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

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

2014-03-11 Thread Koushik Das
that isn't hurtful but an empty ids when no id is given is not checked! Koushik Das wrote: I haven't removed 'id' due to back-compat. As I have mentioned 'id' and 'ids' are mutually exclusive. Also it is a valid scenario to not pass any of id or ids. daan Hoogland wrote: ok, I

DeployVM failing on master with NPE

2014-03-10 Thread Koushik Das
I am seeing the following exception ERROR [c.c.a.ApiServer] (38419149@qtp-1917357200-11:ctx-116b8207 ctx-9a54f764) unhandled exception executing api command: [Ljava.lang.String;@36d30b1d java.lang.NullPointerException at

Re: DeployVM failing on master with NPE

2014-03-10 Thread Koushik Das
This is a new install using master. So this is not about db changes but mostly a bug. On 10-Mar-2014, at 9:58 PM, Marcus shadow...@gmail.com wrote: Mike has sent out several emails about the DB changes he made. On Mon, Mar 10, 2014 at 5:28 AM, Koushik Das koushik@citrix.com wrote: I am

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

2014-03-05 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18352/#review36215 --- Ship it! Ship It! - Koushik Das On March 5, 2014, 8:50 a.m

Re: [DISCUSS] Checking in code that will break others' environments

2014-03-05 Thread Koushik Das
Before doing a git pull, I generally check the sql schema changes and run the delta manually on my existing setup. In most of the cases that works for me without having to redeploy the db. -Koushik On 06-Mar-2014, at 11:43 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Yeah, I

Re: Review Request 18616: added delete iso api to the simulator

2014-03-04 Thread Koushik Das
- Koushik Das On March 4, 2014, 5:55 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18616/ --- (Updated

Re: Review Request 18680: added mock handlers for VpnUsersCfgCommand and RemoteAccessVpnCfgCommand in the simulator

2014-03-04 Thread Koushik Das
- Koushik Das On March 4, 2014, 5:56 a.m., Rajani Karuturi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18680/ --- (Updated

Re: ALARM - ACS reboots host servers!!!

2014-03-02 Thread Koushik Das
The primary storage needs to be put in maintenance before doing any upgrade/reboot as mentioned in the previous mails. -Koushik On 03-Mar-2014, at 6:07 AM, Marcus shadow...@gmail.com wrote: Also, please note that in the bug you referenced it doesn't have a problem with the reboot being

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

2014-03-02 Thread Koushik Das
I am facing a build failure but for a different reason on 4.3-forward. Looks like due to commit 065ac54bd7755cb44667c71311293c8f2110b3f3 Author: tuna ng.t...@gmail.com Date: Sat Mar 1 11:36:27 2014 +0700 CLOUDSTACK-6162: add UI for OVS plugin [INFO]

Re: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

2014-02-28 Thread Koushik Das
as in orchestrateStart() - Koushik Das On Feb. 27, 2014, 12:35 p.m., Sanjay Tripathi wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17889

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

2014-02-23 Thread Koushik Das
for null host id? Make it long instead of Long. - Koushik Das On Feb. 21, 2014, 11:36 a.m., Saksham Srivastava wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18352

Re: Primary storage maintenance doesn't trigger VM shutdown in 4.3

2014-02-17 Thread Koushik Das
For XS this works as expected. For system VMs (SSVM, CPVM) there is a monitoring thread to check if they are up and running. If not then it tries to bring them up. Now if there is no PS available then the deployment fails and it is tried again. This is expected. -Koushik On 15-Feb-2014, at

Re: [PROPOSAL] Storage OverProvisioning as Per Primary Basis

2014-02-14 Thread Koushik Das
Please add more details for upgrade scenario. Also can you give some examples as to how the capacity computation is done for operations where the volume gets migrated from one PS to another having different over provisioning factors. Is there any value add if you give this also at cluster

RE: [PROPOSAL] List VM API enhancement

2014-02-12 Thread Koushik Das
, as this is the most used parameter :) I can see that several existing commands - archive/deleteAlerts are using ApiConstants.IDs parameter. We can mark IDs as deprecated, so its no longer used by new commands. -Alena. On 2/7/14, 11:03 AM, Koushik Das koushik@citrix.com wrote: Good point Min

Re: [PROPOSAL] List VM API enhancement

2014-02-07 Thread Koushik Das
parameter can be one or more. Thanks -min On 2/6/14 3:24 AM, Koushik Das koushik@citrix.com wrote: Yes it will be like a findByIds() and the one id case is just a special case for this. On 06-Feb-2014, at 4:24 PM, Daan Hoogland daan.hoogl...@gmail.com wrote: looks

Re: [PROPOSAL] List VM API enhancement

2014-02-07 Thread Koushik Das
the similar convention, id parameter can be one or more. Thanks -min On 2/6/14 3:24 AM, Koushik Das koushik@citrix.commailto:koushik@citrix.com wrote: Yes it will be like a findByIds() and the one id case is just a special case for this. On 06-Feb-2014, at 4:24 PM, Daan Hoogland daan.hoogl

[PROPOSAL] List VM API enhancement

2014-02-06 Thread Koushik Das
Currently list VM can only be called using a single VM ID. So if there is a need to query a set of VMs using ID then either multiple list VM calls need to be made or all VMs needs to be fetched and then do a client side filtering. Both approaches are sub-optimal - the former results in multiple

Re: [PROPOSAL] List VM API enhancement

2014-02-06 Thread Koushik Das
, Abhinandan Prateek abhinandan.prat...@citrix.com wrote: +1, The listVM call is one of the most resource intensive call. Any step to optimise it are welcome. On 06/02/14 2:01 pm, Koushik Das koushik@citrix.com wrote: Currently list VM can only be called using a single VM ID. So

Re: [PROPOSAL] Introduce API returning you an answer from CloudStack storage/host allocators whethere there is enough resources for vm deployment

2014-02-03 Thread Koushik Das
As I understand this is a best-effort approach. Even if the proposed API returns YES, deployment may still fail. I personally feel that it is better to improve the deployVM API itself to return accurate error message rather than having multiple API calls. If the deployvm call call fails, it

VM instance view UI is broken in 4.3-forward

2014-01-27 Thread Koushik Das
I noticed that VM instance view is broken in UI. It works fine in 4.3 branch. It worked till commit d31fa09c7e5bb04fcc4594c0352d9b91171e9d1a. -Koushik

Re: VM instance view UI is broken in 4.3-forward

2014-01-27 Thread Koushik Das
The only UI related commit I see is 81a83b5b31f0a237ae56449c60350160915f28f8. List VM API is working fine. On 27-Jan-2014, at 2:26 PM, Koushik Das koushik@citrix.com wrote: I noticed that VM instance view is broken in UI. It works fine in 4.3 branch. It worked till commit

4.3 cherry-pick request

2014-01-27 Thread Koushik Das
Animesh, Please cherry-pick the following commit to 4.3 Commit 3fae917ebefb56ba280c2e6c5b6ff0f1720d46ed in branch refs/heads/4.3-forward from Koushik Dashttps://issues.apache.org/jira/secure/ViewProfile.jspa?name=koushikd [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=3fae917 ]

Re: Review Request 17233: Fix for Issue CLOUDSTACK-5875

2014-01-23 Thread Koushik Das
master - f999a01837e60f0c51ef0eb4ad19c29e43ca3037 - Koushik Das On Jan. 23, 2014, 10:27 a.m., Santhosh Edukulla wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17233

Re: [Proposal]CloudStack IAM plugin feature (CLOUDSTACK-5920)

2014-01-22 Thread Koushik Das
Some questions: - Is there a concept of generic permission (any action, any resource etc.)? There shouldn't be a need to define hundreds of explicit permissions for admin account. - I think it would be good to have a notion of parent policy. This will avoid duplication of permissions. - Can

Re: Review Request 16732: CLOUDSTACK-5827: [Automation] Destroy VM failed, while deleting account

2014-01-08 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16732/#review31367 --- Ship it! Ship It! - Koushik Das On Jan. 8, 2014, 9:43 a.m

Re: Review Request 16566: CLOUDSTACK-5551: Search not working for Configuration parameters in (Account/zone/cluster/storage) settings page

2014-01-02 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16566/#review31029 --- Ship it! Ship It! - Koushik Das On Jan. 2, 2014, 10:20 a.m

Re: Review Request 16566: CLOUDSTACK-5551: Search not working for Configuration parameters in (Account/zone/cluster/storage) settings page

2014-01-02 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16566/#review31030 --- Patch doesn't apply on master. - Koushik Das On Jan. 2, 2014, 10

Re: Distributed locking mechanism?

2013-12-22 Thread Koushik Das
What is the account creation logic on Solidfire SAN if you try to create the same simultaneously? I am assuming there should be some unique constraint like account name/id etc. Try to see if this can be utilised so that one of the account creation call fails with appropriate error/exception.

Re: Question about resource ownership

2013-12-22 Thread Koushik Das
Yes that is the current behaviour. All hosts in a cluster should be owned by a single MS. In case when agent load balancing is enabled (agent.lb is set to true), this constraint no longer holds and the hosts in a cluster may get split across multiple MS. If you look at the 'host' table there is

Re: Review Request 16350: Fixed it in ClusterManagerImpl while resetting the heart beat connection

2013-12-19 Thread Koushik Das
/ --- (Updated Dec. 19, 2013, 10:43 a.m.) Review request for cloudstack, Abhinandan Prateek and Koushik Das. Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-5457 https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-5457 Repository: cloudstack-git

Re: VMEntityVO and VMInstanceVO

2013-12-18 Thread Koushik Das
For now I have removed cpu, ram and cpu speed fields from vm_instance table. On 18-Dec-2013, at 6:22 AM, Prachi Damle prachi.da...@citrix.com wrote: Koushik, My comments inline. Prachi -Original Message- From: Koushik Das Sent: Tuesday, December 17, 2013 4:06 AM To: Prachi

Re: Review Request 16315: CLOUDSTACK-5495 Unable to stop VM

2013-12-17 Thread Koushik Das
- Koushik Das On Dec. 17, 2013, 9:38 a.m., bharat kumar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16315/ --- (Updated Dec. 17

Re: Review Request 16315: CLOUDSTACK-5495 Unable to stop VM

2013-12-17 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16315/#review30529 --- Ship it! Ship It! - Koushik Das On Dec. 17, 2013, 11:58 a.m

Re: VMEntityVO and VMInstanceVO

2013-12-15 Thread Koushik Das
Prachi, From the history I see that you added VMEntityVO. Any specific reason for having both? On 12-Dec-2013, at 2:36 PM, Koushik Das koushik@citrix.com wrote: I see that both the VOs uses the same vm_instance table. What is the need for having 2 VOs for the same table? Should one

VMEntityVO and VMInstanceVO

2013-12-12 Thread Koushik Das
I see that both the VOs uses the same vm_instance table. What is the need for having 2 VOs for the same table? Should one be removed? Also VMEntityVO I see a field called speed which is not there in VMInstanceVO. Why this discrepancy? Thanks, Koushik

Re: Review Request 16129: CLOUDSTACK-5419: missing parameters in configuration table and to remove unused parameters

2013-12-10 Thread Koushik Das
/#comment57658 In an existing 4.2/master setup I see that the component of both vmsnapshot.create.wait and vmsnapshot.max is set to none. Handle this scenario as well. - Koushik Das On Dec. 9, 2013, 1:17 p.m., Harikrishna Patnala wrote

Re: Review Request 16129: CLOUDSTACK-5419: missing parameters in configuration table and to remove unused parameters

2013-12-10 Thread Koushik Das
Dec. 10, 2013, 10:55 a.m.) Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-5419 https://issues.apache.org/jira/browse/CLOUDSTACK-5419 Repository: cloudstack-git Description --- CLOUDSTACK-5419: missing parameters in configuration table and to remove

Re: Blocker bug: can't create the volumes on Xen 6.2.0 on the latest 4.3

2013-12-09 Thread Koushik Das
I see that for XS the location of script files in the host has changed from /opt/xensource/bin to /opt/cloud/bin. On latest master branch I see that the scripts are still getting placed in the old location while adding a host. Is there any commit missed out on master? -Koushik On 10-Dec-2013,

Re: Review Request 15763: Changed 'MigrateCommand' to use the global setting value of 'execute.in.sequence.hypervisor.commands'

2013-12-05 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15763/#review29870 --- Ship it! Ship It! - Koushik Das On Dec. 5, 2013, 3:28 p.m

Re: Review Request 15763: Changed 'MigrateCommand' to use the global setting value of 'execute.in.sequence.hypervisor.commands'

2013-12-05 Thread Koushik Das
On Dec. 6, 2013, 4:19 a.m., Koushik Das wrote: Ship It! master - 88f64b2b93a3898e7868e9e55c1feca57b2ed48e - Koushik --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15763/#review29870

Re: Review Request 15763: Changed 'MigrateCommand' to use the global setting value of 'execute.in.sequence.hypervisor.commands'

2013-12-03 Thread Koushik Das
in VirtualMachineManager.java where ExecuteInSequence is defined. - Koushik Das On Nov. 21, 2013, 6:39 p.m., Alex Ough wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15763

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-12-03 Thread Koushik Das
I have posted my review comments. Except for a few minor comments changes look good to me. -Koushik On 02-Dec-2013, at 9:15 PM, Alex Ough alex.o...@sungard.com wrote: All, It's been a while since this review was requested, so can anyone review this to move on? Thanks in advance. Alex

Re: Review Request 15763: Changed 'MigrateCommand' to use the global setting value of 'execute.in.sequence.hypervisor.commands'

2013-12-03 Thread Koushik Das
On Dec. 3, 2013, 8:44 a.m., Koushik Das wrote: core/src/com/cloud/agent/api/MigrateCommand.java, line 32 https://reviews.apache.org/r/15763/diff/1/?file=389559#file389559line32 I see that you have tested for XS. In the mail thread people have mentioned that it should work

Re: Review Request 15763: Changed 'MigrateCommand' to use the global setting value of 'execute.in.sequence.hypervisor.commands'

2013-12-03 Thread Koushik Das
is not showing up. Also there are 2 commits in the patch file. Can you resubmit the patch with only a single commit? - Koushik Das On Dec. 3, 2013, 5:07 p.m., Alex Ough wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 15832: enable custom offering support for scalevm

2013-11-29 Thread Koushik Das
- Koushik Das On Nov. 29, 2013, 7:46 a.m., bharat kumar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15832/ --- (Updated Nov. 29

Re: Review Request 15832: enable custom offering support for scalevm

2013-11-29 Thread Koushik Das
, 2013, 12:35 p.m.) Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-5161 https://issues.apache.org/jira/browse/CLOUDSTACK-5161 Repository: cloudstack-git Description --- enable scaling of a vm using custom offering CLOUDSTACK-5161 Diffs

Re: Cloudstack local storage

2013-11-29 Thread Koushik Das
For using system VMs on local storage there is a global setting system.vm.use.local.storage which needs to be set to true. -Koushik On 29-Nov-2013, at 7:47 PM, Antonio Petrocelli a.petroce...@netsons.com wrote: Hello, we want use cloudstack 4.2 with local storage + VMware 5.1.

Re: Review Request 15832: enable custom offering support for scalevm

2013-11-27 Thread Koushik Das
/#comment56781 Put some comment to explain the logic server/src/com/cloud/vm/UserVmManagerImpl.java https://reviews.apache.org/r/15832/#comment56784 I see the usage event publishing code getting repeated. Move them to a separate method. - Koushik Das On Nov. 27, 2013, 9:33 a.m., bharat

Re: [PROPOSAL] User VM HA using native XS HA capabilities

2013-11-27 Thread Koushik Das
don't think VM HA can be done with hypervisor HA because of this. --Alex -Original Message- From: Koushik Das [mailto:koushik@citrix.comhttp://citrix.com] Sent: Tuesday, November 26, 2013 10:51 PM To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org Subject: Re: [PROPOSAL] User

Re: Review Request 15505: Usage details are not getting populated when using dynamic offerings.

2013-11-26 Thread Koushik Das
- Koushik Das On Nov. 26, 2013, 8:42 a.m., bharat kumar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15505/ --- (Updated Nov. 26

Re: Review Request 15834: CLOUDSTACK-4737: handling usage events for dynamic compute offering

2013-11-26 Thread Koushik Das
- Koushik Das On Nov. 26, 2013, 11:28 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15834/ --- (Updated

Re: Review Request 15505: Usage details are not getting populated when using dynamic offerings.

2013-11-26 Thread Koushik Das
On Nov. 26, 2013, 9:12 a.m., Koushik Das wrote: master - ef40e156ed376968fa9fd15078034d54b97deecc 4.3 - 6af8021ef7188b8f2d92bc891b5108f0e3503284 - Koushik --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 15834: CLOUDSTACK-4737: handling usage events for dynamic compute offering

2013-11-26 Thread Koushik Das
On Nov. 26, 2013, 11:58 a.m., Koushik Das wrote: master - b2f0a0bce26e342f18da42a51ae19d51c238a762 4.3 - 8970c9515441c50a0249b9624b120d4e5348b5af - Koushik --- This is an automatically generated e-mail. To reply, visit: https

Re: [PROPOSAL] User VM HA using native XS HA capabilities

2013-11-26 Thread Koushik Das
. Other folks: please read the entire thread and the links from Koushik; there's a planned deprecation here. --David On Mon, Nov 25, 2013 at 2:38 AM, Koushik Das koushik@citrix.com wrote: Thanks for the comments David. See inline. -Koushik On 22-Nov-2013, at 7:31 PM, David Nalley da

Re: [PROPOSAL] User VM HA using native XS HA capabilities

2013-11-25 Thread Koushik Das
going forward. For the next release both options will be there. Maybe post that the custom HA solution can be removed for XS 6.2 and above. --David On Fri, Nov 22, 2013 at 7:27 AM, Koushik Das koushik@citrix.com wrote: Initial draft of the FS https://cwiki.apache.org/confluence

Re: [PROPOSAL] User VM HA using native XS HA capabilities

2013-11-22 Thread Koushik Das
Initial draft of the FS https://cwiki.apache.org/confluence/display/CLOUDSTACK/User+VM+HA+using+native+XS+HA+capabilities -Koushik On 21-Nov-2013, at 9:59 AM, Koushik Das koushik@citrix.com wrote: Cloudstack relies on custom HA logic for user VMs running on Xenserver. The reason

[PROPOSAL] User VM HA using native XS HA capabilities

2013-11-20 Thread Koushik Das
Cloudstack relies on custom HA logic for user VMs running on Xenserver. The reason for doing it like this may be due the fact that native HA capabilities in XS was not mature enough during the initial days. Also in the custom HA logic, Cloudstack has to correctly determine the state of a VM

Re: bf22f6dfe reverts commit 1762dbbb

2013-11-15 Thread Koushik Das
I have reverted the same in 4.3 as well commit f84b729eb01ffc7cae1e210b58f80ff2033ad785 Author: Koushik Das kous...@apache.org Date: Fri Nov 15 12:59:48 2013 +0530 Revert CLOUDSTACK-5176: This reverts commit f29c7188ba03f1997238d50f8a11857e94fe4b55. On 15-Nov-2013, at 10:35 AM

Re: Review Request 15507: CLOUDSTACK-4835: Update global configuration test cases failed in master

2013-11-14 Thread Koushik Das
://reviews.apache.org/r/15507/#comment55923 Is there any other changes in this file? If not remove it. framework/config/src/org/apache/cloudstack/framework/config/impl/ConfigDepotImpl.java https://reviews.apache.org/r/15507/#comment55924 this is not required - Koushik Das On Nov. 14, 2013, 7:36

Re: Review Request 15507: CLOUDSTACK-4835: Update global configuration test cases failed in master

2013-11-14 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15507/#review28947 --- Ship it! Ship It! - Koushik Das On Nov. 14, 2013, 9:25 a.m

Re: New Committer: Sanjeev Neelarapu

2013-11-13 Thread Koushik Das
Congrats Sanjeev On 14-Nov-2013, at 11:29 AM, Prasanna Santhanam t...@apache.org wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Sanjeev Neelarapu to become a committer and we are pleased to announce that they have accepted. Being a committer allows many

Re: SSL and JCE

2013-11-11 Thread Koushik Das
I see the JCE extensions in jdk 1.7 as well. They are present under java_home/jre/lib/security. But still I see a test failure. Is there any other configuration that is required? Running org.apache.cloudstack.network.lb.CertServiceTest Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time

Re: SSL and JCE

2013-11-11 Thread Koushik Das
(); On 12-Nov-2013, at 11:35 AM, Koushik Das koushik@citrix.com wrote: I see the JCE extensions in jdk 1.7 as well. They are present under java_home/jre/lib/security. But still I see a test failure. Is there any other configuration that is required? Running

Re: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-10 Thread Koushik Das
- 6a8ec786ad2a843768c26099e17f89782cd77f75 - Koushik Das On Nov. 8, 2013, 12:24 p.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348

Re: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-08 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15348/#review28538 --- Ship it! Ship It! - Koushik Das On Nov. 8, 2013, 12:02 p.m

Re: Cloudstack 4.2.0 migration of local data volume is not allowed

2013-11-07 Thread Koushik Das
Is your data volume and VM on the same host? If not then the volume cannot be attached to the VM in case of a local volume. -Koushik On 07-Nov-2013, at 4:03 PM, Antonio Petrocelli a.petroce...@netsons.com wrote: Hello, i've this error when i try to attach a volume to vm: Failed to

Re: Review Request 15173: Dynamic compute offering.

2013-11-06 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15173/#review28355 --- Ship it! Ship It! - Koushik Das On Nov. 7, 2013, 6:24 a.m

Re: Review Request 15173: Dynamic compute offering.

2013-11-06 Thread Koushik Das
On Nov. 7, 2013, 7:17 a.m., Koushik Das wrote: Ship It! commit 7095ea2b5e497620c8c879517565046e3b46e493 - Koushik --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15173/#review28355

Re: Review Request 15018: CLOUDSTACK-4908: CPU socket count of host

2013-11-05 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15018/#review28183 --- Ship it! Ship It! - Koushik Das On Nov. 5, 2013, 10:34 a.m

Re: Review Request 15173: Dynamic compute offering.

2013-11-04 Thread Koushik Das
Again this is not required engine/schema/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java https://reviews.apache.org/r/15173/#comment54674 ordering of parameter is not correct. optional parameters should be at the end - Koushik Das On Nov. 1, 2013, 5:40 a.m., bharat kumar

Re: Review Request 15018: CLOUDSTACK-4908: CPU socket count of host

2013-11-04 Thread Koushik Das
://reviews.apache.org/r/15018/#comment54681 Simply drop and recreate the view - Koushik Das On Oct. 31, 2013, 10:14 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Command sequence logic in agent code

2013-10-24 Thread Koushik Das
Created https://issues.apache.org/jira/browse/CLOUDSTACK-4944 to track this issue. Alex, Any reason for adding requests based on sequence and not doing FIFO? Do you see any issues if request always gets added to the end of the queue? On 23-Oct-2013, at 6:26 PM, Koushik Das koushik

Re: Command sequence logic in agent code

2013-10-24 Thread Koushik Das
look here. I think when commands originate from the management server, it doesn't go through this code. This is if the command came from the agent which doesn't matter if there's multiple management servers. --Alex -Original Message- From: Koushik Das Sent: Thursday, October

Re: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Koushik Das
Congrats Talluri! On 23-Oct-2013, at 4:07 PM, Prasanna Santhanam t...@apache.org wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Srikanteswara Talluri to become a committer and we are pleased to announce that they have accepted. Being a committer allows many

Re: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Koushik Das
Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam t...@apache.org wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Sowmya Krishnan to become a committer and we are pleased to announce that they have accepted. Being a committer allows many

Re: Review Request 14305: CLOUDSTACK-4493: registerSSHKeyPair API doc contains wrong API response (private key)

2013-10-21 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14305/#review27250 --- Ship it! Ship It! - Koushik Das On Oct. 21, 2013, 5:14 a.m

Re: Review Request 14305: CLOUDSTACK-4493: registerSSHKeyPair API doc contains wrong API response (private key)

2013-10-19 Thread Koushik Das
https://reviews.apache.org/r/14305/#comment52933 Why this is changed to protected? api/src/org/apache/cloudstack/api/response/SSHKeyPairResponse.java https://reviews.apache.org/r/14305/#comment52934 Protected? This is not required - Koushik Das On Oct. 18, 2013, 12:43 p.m

Re: Cloudstack collab Hackathons

2013-10-17 Thread Koushik Das
Agree with Marcus here. If we are planning to rewrite, I think that there should be some discussion around the possibility of making it a direct agent like XS/Vmware. If this is possible then installing the KVM agent on individual hosts will be eliminated. On 17-Oct-2013, at 8:49 PM, Marcus

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
, at 3:21 AM, Prachi Damle prachi.da...@citrix.commailto:prachi.da...@citrix.com wrote: -Original Message- From: Koushik Das [mailto:koushik@citrix.comhttp://citrix.com] Sent: Wednesday, October 16, 2013 5:21 AM To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org Subject: Re

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
is the latter where there is no pools in the allocator avoid set. So as you suggested in this case as well the cluster needs to be in avoid set. Prachi -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Thursday, October 17, 2013 9:23 AM To: dev@cloudstack.apache.org

Re: Possible bug in DeploymentPlanner?

2013-10-16 Thread Koushik Das
On 16-Oct-2013, at 3:12 AM, Prachi Damle prachi.da...@citrix.com wrote: -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: Tuesday, October 15, 2013 11:43 AM To: dev@cloudstack.apache.org Subject: Re: Possible bug in DeploymentPlanner? Thanks

Re: Possible bug in DeploymentPlanner?

2013-10-15 Thread Koushik Das
this, that needs to be fixed. Also, I do think in general it will be safe to add a configurable retry limit on this loop to have control in case any new logic in allocators don't follow this reasoning. I will add that limit. Thanks, Prachi -Original Message- From: Koushik

Re: [PROPOSAL] Remove Setters from *JoinVO

2013-10-09 Thread Koushik Das
Views are meant to be read only. So +1 for removing setters. On 04-Oct-2013, at 10:59 PM, SuichII, Christopher chris.su...@netapp.com wrote: *JoinVOs are used to store entries from MySQL views, which are not editable. I think removing setters from the *JoinVOs may help avoid some potential

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-03 Thread Koushik Das
for your reply, but unfortunately, this setting does NOT cover 'MigrateCommand'. As you specified, it seems to be effective only in Start/Stop/CopyCommand. So can we include 'MigrateCommand' in that setting? Thanks Alex Ough On Wed, Oct 2, 2013 at 11:44 PM, Koushik Das koushik

Re: A question on vm migrations when hosts are set into a maintenance mode.

2013-10-02 Thread Koushik Das
It is already a global setting in 4.2. The name of the setting is execute.in.sequence.hypervisor.commands. Check out Config.java ExecuteInSequence(Advanced, ManagementServer.class, Boolean.class, execute.in.sequence.hypervisor.commands, true, If set to true, StartCommand, StopCommand,

Re: Hi

2013-10-02 Thread Koushik Das
Hi Steve, Welcome to CloudStack. Looking forward to working with you. On 02-Oct-2013, at 6:10 PM, Chip Childers chip.child...@sungard.com wrote: Welcome Steve! On Wed, Oct 2, 2013 at 12:38 AM, Steve Wilson steve.wil...@citrix.comwrote: Hi Everyone, I just wanted to take a second

Re: my two cents on RBAC/authorization

2013-09-29 Thread Koushik Das
I think it is important to identify what is exclusively for Cloudstack and what all can be reused across multiple services. 1. Authentication - Typically enterprises/service providers would like to reuse their existing authentication systems. So an easy mechanism needs to be provided for

Re: [MAJOR][BUG] ACS powers off some VMs in vSphere - when MS service is restarted

2013-09-29 Thread Koushik Das
Ilya, Can you check in the MS logs if more than one thread tries to process connect for the same host simultaneously? In case of XS I had seen that there is a possibility of the vm states getting incorrectly determined (during vm sync) in case more than one thread tries to process connect to

Primary store (PS) failure/outage

2013-09-16 Thread Koushik Das
Currently the way Cloudstack deals with PS failure is to reboot all hosts associated with the cluster. Selectively cleaning up the affected VMs would have been the best option, but since issues were seen with stopping VMs on the hypervisors (at least in Xenserver 5.6 [1]) reboot was the next

Re: [ANNOUNCE] New Committer: Dave Cahill

2013-09-07 Thread Koushik Das
Congrats Dave! On 07-Sep-2013, at 7:50 AM, Marcus Sorensen shadow...@gmail.com wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Dave Cahill to become a committer and we are pleased to announce that he has accepted. Dave has been an active member of the

Re: New Committer: Daan Hoogland

2013-08-28 Thread Koushik Das
Congrats Daan! On 28-Aug-2013, at 7:28 PM, Chip Childers chip.child...@sungard.com wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Daan Hoogland to become a committer and we are pleased to announce that he has accepted! Daan's been quite active on the dev

Re: [ANNOUNCE] New Committer: Ian Duffy

2013-08-28 Thread Koushik Das
Congrats Ian! On 28-Aug-2013, at 12:42 AM, Sebastien Goasguen run...@gmail.com wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Ian Duffy to become a committer and we are pleased to announce that he has accepted. Being a committer enables easier contribution to

Optimize code for scale

2013-08-28 Thread Koushik Das
Recently I worked on some performance related issues and noticed a pattern in the code that lead to increased latency for some APIs in a scaled up environment ( 10K user VMs, 10K hosts). The pattern is like this: ListHost hosts = listHosts(); // based on some filter for (Host h : hosts) {

Re: [BUG] vSphere - VMs are force powered off by ACS

2013-08-27 Thread Koushik Das
During MS restart as part of host connect, vm sync happens. This may be stopping VMs if there is a mismatch in actual state and what is present in the database. Do you see anything useful in the MS logs? -Koushik On 27-Aug-2013, at 4:20 AM, Musayev, Ilya imusa...@webmd.net wrote: I've been

Re: Review Request 13842: CLOUDSTACK-4482: getVMPassword() API call does not return password for Vms that are deployed with password enabled templates.

2013-08-27 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13842/#review25612 --- Ship it! Ship It! - Koushik Das On Aug. 27, 2013, 10:53 a.m

RE: Unable to create volume - volume size xxxx, but the maximum size allowed is 0 Gb.

2013-08-21 Thread Koushik Das
In 4.2 branch it gets initialized from config property storage.max.volume.size. Storage DEFAULT management-server storage.max.volume.size 2000The maximum size for a volume (in GB). Which branch are you in? Master. Maybe some missing commit in master. -Original Message-

Re: [ANNOUNCE] New Committer: Toshiaki Hatano

2013-08-20 Thread Koushik Das
Congrats Toshiaki! On 20-Aug-2013, at 4:26 AM, Marcus Sorensen shadow...@gmail.commailto:shadow...@gmail.com wrote: Toshiaki

Review Request 13627: Fix for CLOUDSTACK-3441

2013-08-17 Thread Koushik Das
that is checking if the network is already implemented or shutdowned outside the lock. Diffs - server/src/com/cloud/network/NetworkManagerImpl.java 68b1b4f Diff: https://reviews.apache.org/r/13627/diff/ Testing --- Thanks, Koushik Das

Re: Review Request 13627: Fix for CLOUDSTACK-3441

2013-08-17 Thread Koushik Das
. As part of the fix moved out code that is checking if the network is already implemented or shutdowned outside the lock. Diffs (updated) - server/src/com/cloud/network/NetworkManagerImpl.java 68b1b4f Diff: https://reviews.apache.org/r/13627/diff/ Testing --- Thanks, Koushik Das

<    3   4   5   6   7   8   9   >