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 integr

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 the

Re: [PROPOSAL] Service monitoring tool in virtual router

2013-09-30 Thread Koushik Das
This is a very useful feature. Can this be extended to the other system VMs? SSVM and CPVM Based on the discussion I see that there is an assumption that restarting services/rebooting should fix the issues. Is that always true? What if the service fails to restart after repeated attempts? What

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, StopComm

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 wrote: > Welcome Steve! > > > On Wed, Oct 2, 2013 at 12:38 AM, Steve Wilson wrote: > >> Hi Everyone, >> >> I just wanted to take a second and introduce myself. I recently joined

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

2013-10-03 Thread Koushik Das
our 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 We

Re: Back end volume names in KVM

2013-10-09 Thread Koushik Das
On 10-Oct-2013, at 9:27 AM, Marcus Sorensen wrote: > Does anyone have any reservations about changing the volume identifier in > KVM's volume creation command to be the uuid of the volume? Currently for > new volumes it generates a random uuid and passes that back to be stored in > the database.

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" 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 > confusion as sett

Possible bug in DeploymentPlanner?

2013-10-15 Thread Koushik Das
I was making some changes in the storage pool allocators related to some bug fix and came across this code snippet in planDeplyment() method of DeploymentPlanningManagerImpl.java. In this if the checkClustersforDestination() returns null and the 'avoids' parameter is not correctly updated (one s

Re: Possible bug in DeploymentPlanner?

2013-10-15 Thread Koushik Das
is reasoning. I will add that limit. > > Thanks, > Prachi > > > > -Original Message- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: Tuesday, October 15, 2013 9:19 AM > To: > Subject: Possible bug in DeploymentPlanner? > > I was making som

Re: Possible bug in DeploymentPlanner?

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

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 Sor

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
21 AM, Prachi Damle mailto:prachi.da...@citrix.com>> wrote: -Original Message- From: Koushik Das [mailto:koushik@citrix.com<http://citrix.com>] Sent: Wednesday, October 16, 2013 5:21 AM To: mailto:dev@cloudstack.apache.org>> Subject: Re: Possible bug in DeploymentPlanne

Re: Possible bug in DeploymentPlanner?

2013-10-17 Thread Koushik Das
mption in canAvoidCluster(). The scenario I am seeing 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@c

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, 201

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: [ANNOUNCE] New PMC member: Animesh Chaturvedi

2013-10-21 Thread Koushik Das
Congrats Animesh! On 22-Oct-2013, at 12:31 AM, Chip Childers wrote: > The Project Management Committee (PMC) for Apache CloudStack has asked > Animesh Chaturvedi to join the PMC and we are pleased to announce that they > have accepted. > > Join me in congratulating Animesh! > > -The CloudStack

Command sequence logic in agent code

2013-10-23 Thread Koushik Das
I was looking at the command sequencing logic in the agent code. Each agent maintains a sequence that gets initialised based on following logic private static final Random s_rand = new Random(System.currentTimeMillis()); _nextSequence = s_rand.nextInt(Short.MAX_VALUE) << 48; For every co

Re: [ANNOUNCE] New Committer: Talluri, Srikanteswara

2013-10-23 Thread Koushik Das
Congrats Talluri! On 23-Oct-2013, at 4:07 PM, Prasanna Santhanam 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 contribut

Re: [ANNOUNCE] New Committer: Krishnan, Sowmya

2013-10-23 Thread Koushik Das
Congrats Sowmya! On 23-Oct-2013, at 4:16 PM, Prasanna Santhanam 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 contributors to con

Re: [ANNOUNCE] New Committer: Shilamkar, Girish

2013-10-23 Thread Koushik Das
Congrats Girish! On 23-Oct-2013, at 4:08 PM, Prasanna Santhanam wrote: > The Project Management Committee (PMC) for Apache CloudStack has asked Girish > Shilamkar to become a committer and we are pleased to announce that they have > accepted. > > Being a committer allows many contributors to co

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 wrote: > I

Re: Command sequence logic in agent code

2013-10-24 Thread Koushik Das
ink 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- >>

Review Request 15080: CLOUDSTACK-4855: Throttle based on the # of outstanding requests to the directly managed HV host (direct agents)

2013-10-30 Thread Koushik Das
7d3f765 Diff: https://reviews.apache.org/r/15080/diff/ Testing --- Verified by tweaking the per agent upper cap to a value of 1 and checked that the requests are getting scheduled but the executor thread simply bails out. Thanks, Koushik Das

Re: Review Request 15173: Dynamic compute offering.

2013-11-04 Thread Koushik Das
chema/src/com/cloud/service/ServiceOfferingVO.java <https://reviews.apache.org/r/15173/#comment54675> 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 n

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

2013-11-04 Thread Koushik Das
setup/db/db/schema-421to430.sql <https://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 aut

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-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/#re

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 wrote: > Hello, > > i've this error when i try to attach a volume to vm: > > "Failed to attach local data volu

Re: pool.storage.capacity.disablethreshold global config

2013-11-07 Thread Koushik Das
>From what I understand, now there is a new interface Configurable that exposes >the config parameters. In this case if you look at the >CapacityManagerImpl.java, there is a method getCongigKeys() that return the >config parameters. During MS startup all these are automatically populated in >th

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: Review Request 15348: CLOUDSTACK-5105 Template/ISO download fails cause the object to disappear from UI

2013-11-10 Thread Koushik Das
4.3 -> 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.

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 /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 elapsed: 1.456

Re: SSL and JCE

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

Re: New Committer: Sanjeev Neelarapu

2013-11-13 Thread Koushik Das
Congrats Sanjeev On 14-Nov-2013, at 11:29 AM, Prasanna Santhanam 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 contributors to

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

2013-11-14 Thread Koushik Das
tps://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.

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: 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 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: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-17 Thread Koushik Das
/xenserver/xcposs. This also needs to be fixed for devcloud to work - Koushik Das On May 17, 2013, 1:56 p.m., sanjeev n wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

RE: CLOUDSTACK-2554: Yet another blocker?

2013-05-17 Thread Koushik Das
The problem is that there is no default ctor in XcpServerResource.java. Now the default ctor was changed to a non-default one by this commit. Adding a default ctor should fix the issue but need to check if there will be any other side effects due to the version field introduced. commit 63630a41

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread Koushik Das
tps://reviews.apache.org/r/11221/#comment42836> For other entries I see different target paths. What should be the correct one in these cases? - Koushik Das On May 20, 2013, 6:44 a.m., sanjeev n wrote: > > --- > This is an automatica

Re: Review Request: CLOUDSTACK-2543: createipAlias.sh/deleteipAlias.sh won't be copied to XenServer host

2013-05-20 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11221/#review20768 --- Ship it! Ship It! - Koushik Das On May 20, 2013, 1:14 p.m

RE: [ANNOUNCE] New committer: Wei Zhou

2013-05-23 Thread Koushik Das
Congrats Wei! > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, May 24, 2013 2:53 AM > To: > Subject: [ANNOUNCE] New committer: Wei Zhou > > The Project Management Committee (PMC) for Apache CloudStack has > asked Wei Zhou to become a committer

RE: [ANNOUNCE] New committer: Venkata Swamy

2013-05-23 Thread Koushik Das
Congrats Swamy! > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, May 24, 2013 2:52 AM > To: > Subject: [ANNOUNCE] New committer: Venkata Swamy > > The Project Management Committee (PMC) for Apache CloudStack has > asked Venkata Swamy to become

RE: [ANNOUNCE] New committer: Sailaja Mada

2013-05-23 Thread Koushik Das
Congrats Sailaja! > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, May 24, 2013 2:52 AM > To: > Subject: [ANNOUNCE] New committer: Sailaja Mada > > The Project Management Committee (PMC) for Apache CloudStack has > asked Sailaja Mada to become

RE: Convention on UUID column

2013-05-23 Thread Koushik Das
It is better to add constraints in the db for all uuid fields. That way uuid field will never get missed out. I see that for some tables there is a NOT NULL constraint. -Koushik > -Original Message- > From: Min Chen [mailto:min.c...@citrix.com] > Sent: Friday, May 24, 2013 4:09 AM > To

RE: [ANNOUNCE] New committer: Sangeetha Hariharan

2013-05-23 Thread Koushik Das
Congrats Sangeetha! > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Friday, May 24, 2013 11:50 AM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Sangeetha Hariharan > > The Project Management Committee (PMC) for Apache CloudStack has >

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-24 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/#review20991 --- Ship it! Ship It! - Koushik Das On May 24, 2013, 10:19 a.m

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-05-27 Thread Koushik Das
NetworkApplianceManagerImpl.java <https://reviews.apache.org/r/11435/#comment43575> message says apply but this for revoke. right? also dhcp has become "dhco" server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java <https://reviews.apache.org/r/114

Re: Review Request: CLOUDSTACK-2648 [Multiple_IP_Ranges] Reboot or start/stop router vm deletes the ip alises created on VR in case of multiple subnets

2013-05-27 Thread Koushik Das
er/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java <https://reviews.apache.org/r/11437/#comment43593> The message says apply, should be revoke. - Koushik Das On May 27, 2013, 3:23 p.m., bharat kumar wrote: > > --- >

Re: Review Request: CLOUDSTACK-2620 [Multiple_IP_Ranges] Guest vm's nameserver is not set to VRs guest IP address in case of multiple subnets

2013-05-28 Thread Koushik Das
om/cloud/network/router/VirtualNetworkApplianceManagerImpl.java <https://reviews.apache.org/r/11436/#comment43619> You have the cidr which can be used as well in dhcp-range parameter. - Koushik Das On May 27, 2013, 3:05 p.m., bharat kumar wrote: > > -

RE: [PROPOSAL] Pushback 4.2.0 Feature Freeze

2013-05-30 Thread Koushik Das
> -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Thursday, May 30, 2013 12:36 PM > To: dev@cloudstack.apache.org > Subject: Re: [PROPOSAL] Pushback 4.2.0 Feature Freeze > > On Thu, May 30, 2013 at 3:02 AM, murali reddy > wrote: > > We should do a health-check of

Re: Review Request: CLOUDSTACK-2648 [Multiple_IP_Ranges] Reboot or start/stop router vm deletes the ip alises created on VR in case of multiple subnets

2013-06-03 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11437/#review21311 --- Ship it! Ship It! - Koushik Das On June 3, 2013, 10:10 a.m

Re: Review Request: CLOUDSTACK-2620 [Multiple_IP_Ranges] Guest vm's nameserver is not set to VRs guest IP address in case of multiple subnets

2013-06-03 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11436/#review21314 --- Ship it! Ship It! - Koushik Das On June 3, 2013, 1:08 p.m

StaticNatRule vs StaticNat

2013-06-03 Thread Koushik Das
What is the difference between these interfaces? I see that StaticNat is used in network elements. And StaticNatRule used elsewhere including APIs. Given that PF and FW rules uses a single interface everywhere, should a similar thing be there for static nat rules as well? -Koushik

RE: [VOTE] Pushback 4.2.0 Feature Freeze

2013-06-03 Thread Koushik Das
+0 If proposed features are completed earlier then we shouldn't wait for 4 weeks. -Koushik > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, May 31, 2013 8:30 PM > To: dev@cloudstack.apache.org > Subject: [VOTE] Pushback 4.2.0 Feature Freeze >

Re: Review Request: Cloudstack-2511 Multiple_Ip_Ranges: Adding guest ip range in subset/superset to existing CIDR is allowed https://issues.apache.org/jira/browse/CLOUDSTACK-2511, Cloudstack-2651 [Sha

2013-06-10 Thread Koushik Das
imple -> 1 commit = 1 bug fix - Koushik Das On June 4, 2013, 2:51 p.m., bharat kumar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://review

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread Koushik Das
set to false, but the code is still moving ahead. Previously there was a return So the remaining code should only be executed if (result == true). - Koushik Das On June 3, 2013, 10:27 a.m., bharat kumar wrote: > > --- > This is an

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-10 Thread Koushik Das
<https://reviews.apache.org/r/11435/#comment44842> if result_final == false, why is txn committed? - Koushik Das On June 3, 2013, 10:27 a.m., bharat kumar wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-11 Thread Koushik Das
<https://reviews.apache.org/r/11435/#comment44863> why not pass the 'caller' parameter to handleIpAliasDelete? server/src/com/cloud/configuration/ConfigurationManagerImpl.java <https://reviews.apache.org/r/11435/#comment44862> why is both rollback and commit done

Re: Review Request: Cloudstack-2621 [Multiple_IP_Ranges] Failed to delete guest IP range from a new subnet/C

2013-06-12 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11435/#review21791 --- Ship it! Ship It! - Koushik Das On June 12, 2013, 10:59 a.m

RE: Using "In Progress" status in JIRA

2013-06-14 Thread Koushik Das
> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Friday, June 14, 2013 6:32 AM > To: dev@cloudstack.apache.org > Subject: Using "In Progress" status in JIRA > > Folks > > It seems that we do not use "In Progress" status in JIRA as often as we

RE: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Koushik Das
Congrats Jayapal > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Monday, June 17, 2013 9:02 PM > To: dev@cloudstack.apache.org > Subject: [ANNOUNCE] New committer: Jayapal Reddy Uradi > > The Project Management Committee (PMC) for Apache CloudStack ha

RE: Which file to update for schema change required for 4.1.1?

2013-06-19 Thread Koushik Das
Ideally if the fix involves schema changes then it should go to 4.2. If there is no alternative then 410to411.sql looks like the option. But then the upgrade path becomes complicated. The following needs to be handled 4.1.0 -> 4.2.0, 4.1.1 -> 4.2.0. Or else the upgrade sql script needs to be gua

RE: [ANNOUNCE] New committer: Mike Tutkowski

2013-06-19 Thread Koushik Das
Congrats Mike! > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Wednesday, June 19, 2013 7:10 PM > To: dev@cloudstack.apache.org > Subject: FW: [ANNOUNCE] New committer: Mike Tutkowski > > > The Project Management Committee (PMC) for Apache CloudStack has >

(CLOUDSTACK-3248) XenServer Host got removed successfully inspite of running VMs on the host

2013-07-10 Thread Koushik Das
This looks like a serious issue. Cloudstack goes into a bad state due to this. From the UI the host first needs to be put into maintenance and only after that the host can be deleted. But if deleteHost API is directly invoked there is no such restriction (there is no check to see the ResourceSta

RE: HA for VMWare

2013-07-10 Thread Koushik Das
That's right. Cloudstack relies on native HA capabilities provided by vSphere. > -Original Message- > From: Kirk Kosinski [mailto:kirkkosin...@gmail.com] > Sent: Thursday, July 11, 2013 6:30 AM > To: us...@cloudstack.apache.org > Cc: Chip Childers; dev@cloudstack.apache.org > Subject: Re:

RE: Localization

2013-07-10 Thread Koushik Das
That's correct. Currently all messages are hard-coded in the Java code. First all these strings needs to be moved out to something like a 'resource file' and only then they can be localized. > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Thursda

RE: CLOUDSTACK-531: FIXED ? OPENED ?

2013-07-11 Thread Koushik Das
Hi Nicolas, This issue got fixed as part of CLOUDSTACK-595. The fix is present in 4.0 and onwards. I will resolve 531 as a duplicate of 595. -Koushik > -Original Message- > From: nicolas.lamira...@orange.com [mailto:nicolas.lamira...@orange.com] > Sent: Thursday, July 11, 2013 5:59 PM >

RE: Another ISO Question

2013-07-12 Thread Koushik Das
Should it fall back on the compute offering tags if disk offering tags cannot be satisfied? This will make the behavior compatible with that of deploying VM from template. -Koushik > -Original Message- > From: Prachi Damle [mailto:prachi.da...@citrix.com] > Sent: Friday, July 12, 2013 2

RE: Is VMware working in 4.2?

2013-07-15 Thread Koushik Das
This is fixed as part of CLOUDSTACK-3445 in master and 4.2. The issue was with local storage pool used bytes getting wrongly stored as you noted. -Koushik > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Tuesday, July 16, 2013 6:46 AM > To: dev@cl

RE: Template Question

2013-07-18 Thread Koushik Das
I am also seeing it. -Koushik > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Wednesday, July 17, 2013 9:22 PM > To: dev@cloudstack.apache.org > Subject: Template Question > > Hi, > > I've noticed recently on XenServer when SSVM and CPVM are dep

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-19 Thread Koushik Das
. updateCluster needs to be fixed as updateConfiguration is a generic approach and shouldn't be broken for handling a special scenario. - Koushik Das On July 17, 2013, 7:37 a.m., Harikrishna Patnala wrote: > > --- > This is an automatic

Re: Review Request 12427: Fix for CLOUDSTACK-3365: cluster level parameters cluster.(cpu/memory).allocated.capacity.notificationthreshold is not considering overcommit value

2013-07-20 Thread Koushik Das
in the code everywhere based on over provisioning factor. Please open a new bug for future to store used/total capacity values in db. based on same scale factor. - Koushik Das On July 19, 2013, 10:14 a.m., Harikrishna Patnala wrote: > > --

Re: Review Request 12427: Fix for CLOUDSTACK-3365: cluster level parameters cluster.(cpu/memory).allocated.capacity.notificationthreshold is not considering overcommit value

2013-07-20 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23577 --- Ship it! Ship It! - Koushik Das On July 19, 2013, 10:14 a.m

Re: Error to start MS on 4.2

2013-07-20 Thread Koushik Das
The code needs to handle the case when the db. entry is not present. It should fall back on some default value. -Koushik On 21-Jul-2013, at 12:38 AM, Min Chen wrote: > Thanks Vijay. I just figured out that trick myself:) I will check in a fix > to 4.2, seems that he forgot to seed this config

System VMs not coming up due to https://reviews.apache.org/r/12685/

2013-07-22 Thread Koushik Das
Commit id: 2d4464d2badc9aff842fd180bafc4c384a83a91d -return new URI(scheme + "://" + value); +// do we need to check that value does not contain a scheme +// part? +if (value.toString().contains(":")) +return new U

RE: System VMs not coming up due to https://reviews.apache.org/r/12685/

2013-07-22 Thread Koushik Das
RE: System VMs not coming up due to https://reviews.apache.org/r/12685/ Where are you getting your system VM from? DL > -Original Message----- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: 22 July 2013 11:09 AM > To: cloudstack-...@incubator.apache.org > Subject: S

Re: System VMs not coming up due to https://reviews.apache.org/r/12685/

2013-07-22 Thread Koushik Das
ld be the correct code for the function that is your culprit, but master > doesn't work due to database mismatches between the code and the databse > schema. I will test this as soon as possible. > > regards, > Daan > > > On Mon, Jul 22, 2013 at 5:18 PM,

Re: Review Request 12849: added backwards compatibility code to Networks enums

2013-07-23 Thread Koushik Das
e that upgraded setups work fine as well. - Koushik Das On July 23, 2013, 10:02 a.m., daan Hoogland wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Koushik Das
For now I think cloud-utils is the best option. > -Original Message- > From: Amogh Vasekar [mailto:amogh.vase...@citrix.com] > Sent: Wednesday, July 24, 2013 10:48 PM > To: dev@cloudstack.apache.org > Subject: Re: Inclusion of commons-httpclient-contrib jar > > Hi Alex, > > Thanks! I had

Re: Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-25 Thread Koushik Das
views.apache.org/r/12808/ > --- > > (Updated July 23, 2013, 11:41 a.m.) > > > Review request for cloudstack, Koushik Das and Nitin Mehta. > > > Bugs: CLOUDSTACK-1528 and CLOUDSTACK-2536 > > > Repository: cloudstack-git > > > Description > --- > >

Re: Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-25 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12808/#review23823 --- Ship it! Ship It! - Koushik Das On July 25, 2013, 7:19 a.m

RE: [DISCUSS] vlan uri format change

2013-07-27 Thread Koushik Das
I raised this in a separate thread. Daan created a new patch https://reviews.apache.org/r/12849/ to address backward compat. > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Saturday, July 27, 2013 11:12 AM > To: dev@cloudstack.apache.org > Subject: Re: [DI

RE: Request to inform which DB command to run on existing setup if any db schema changes

2013-07-30 Thread Koushik Das
Rajesh, You can look at the check-in history for schema-410to420.sql for the changes > -Original Message- > From: Rajesh Battala [mailto:rajesh.batt...@citrix.com] > Sent: Tuesday, July 30, 2013 5:03 PM > To: dev@cloudstack.apache.org > Subject: Request to inform which DB command to run on

RE: [ACS42][Request] CLOUDSTACK-4001

2013-08-01 Thread Koushik Das
I see this in the review request Review 13189 failed the build test : FAILURE The url of build cloudstack-master-with-patch #99 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/99/ Can you address this? -Koushik > -Original Message- > From: Sanjay Tripathi [mailto:sa

Re: Review Request 13072: Print agent host name in logging of agent commands

2013-08-01 Thread Koushik Das
nd of correlation would be much more useful. - Koushik Das On July 30, 2013, 5:37 p.m., Marcus Sorensen wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

RE: Review Request 13072: Print agent host name in logging of agent commands

2013-08-01 Thread Koushik Das
Cmd , MgmtId: 2546680725505, via: 1, Ver: v1, Flags: 100011, [{"com.cloud.agent.api.StopCommand":{"isProxy":false,"executeInSequence":false,"vmName":"i-2-45-VM","wait":0}}] } The changes you have made would definitely improve the readability. -Koushik From: Mar

RE: [DISCUSS] [jira] make affectedVersion field mandatory.....

2013-08-02 Thread Koushik Das
+1 > -Original Message- > From: Ram Ganesh [mailto:ram.gan...@citrix.com] > Sent: Friday, August 02, 2013 4:38 PM > To: dev@cloudstack.apache.org > Subject: [DISCUSS] [jira] make affectedVersion field mandatory. > > Hi, > > While triaging bugs I noticed that many bugs had affectedVer

RE: Reverting 17267794adb2bab923fb20515a7b943780d61921

2013-08-05 Thread Koushik Das
+1. Provided git allows to make a specific file as read-only. > -Original Message- > From: Nitin Mehta [mailto:nitin.me...@citrix.com] > Sent: Monday, August 05, 2013 12:01 PM > To: dev@cloudstack.apache.org; Saksham Srivastava > Subject: Re: Reverting 17267794adb2bab923fb20515a7b943780d61

RE: [jira] [Commented] (CLOUDSTACK-4179) [Performance Testing] Time taken for Deploy VM async job to complete is considerably higher

2013-08-08 Thread Koushik Das
The increase in deployVM time is due to the dedicated resources feature. I have updated the issue with my analysis. Need some inputs from the original devs. -Koushik > -Original Message- > From: Koushik Das (JIRA) [mailto:j...@apache.org] > Sent: Thursday, August 08, 2013 6:3

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Koushik Das
or the same. Also what is logic for multiplying the speed with 0.99? - Koushik Das On Aug. 9, 2013, 1 p.m., Harikrishna Patnala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://rev

RE: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-10 Thread Koushik Das
Congrats Vijay! > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Saturday, August 10, 2013 10:31 AM > To: CloudStack Dev > Subject: [ANNOUNCE] New Committer: Vijay Bhamidipati > > The Project Management Committee (PMC) for Apache CloudStack has > asked Vija

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-11 Thread Koushik Das
info. Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.884 sec <<< FAILURE! Results : Failed tests: testScaleVMF3(com.cloud.hypervisor.xen.resource.CitrixResourceBaseTest): (..) Tests run: 3, Failures: 1, Errors: 0, Skip

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-11 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24978 --- Ship it! Ship It! - Koushik Das On Aug. 11, 2013, 5:28 p.m

Change history lost for upgrade sql script schema-412to420.sql

2013-08-12 Thread Koushik Das
I noticed that the upgrade sql script (earlier used to be schema-410to420.sql) has been renamed and in the process all the change history is lost. Can we ensure as a process to do a proper git file rename instead of creating a new file so that the changes history is not lost? In this case is the

RE: Change history lost for upgrade sql script schema-412to420.sql

2013-08-12 Thread Koushik Das
b8b456e652d47f2a6;hb=e61a38087 > d158da51f7b945d16e4749a83512ab5 > > Regards, > Devdeep > > > -Original Message- > > From: Koushik Das [mailto:koushik@citrix.com] > > Sent: Monday, August 12, 2013 2:50 PM > > To: cloudstack-...@incubator.apache.org > >

RE: Could not find corresponding resource manager for VMware

2013-08-13 Thread Koushik Das
Also refer https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+to+build+CloudStack for the list of nonoss jars that needs to be installed before doing the build. -Koushik > -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Wednesday, August 14,

RE: [DISCUSS] git commit messages

2013-08-15 Thread Koushik Das
Agree. The message should indicate the root cause of the problem and the solution. -Koushik > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Friday, August 16, 2013 2:58 AM > To: dev@cloudstack.apache.org > Subject: RE: [DISCUSS] git commit messages > > +1

  1   2   3   4   5   6   7   8   9   10   >