RE: [VOTE] Accept the donation of RDP client code into Apache CloudStack

2013-10-21 Thread Devdeep Singh
+1 -Original Message- From: Donal Lafferty [mailto:donal.laffe...@citrix.com] Sent: Monday, October 21, 2013 11:42 PM To: dev@cloudstack.apache.org Subject: [VOTE] Accept the donation of RDP client code into Apache CloudStack As stated in a previous thread [1], Citrix is proposing the do

RE: [ANNOUNCE] New PMC member: Animesh Chaturvedi

2013-10-21 Thread Devdeep Singh
Congrats Animesh. Regards, Devdeep -Original Message- From: chip.child...@sungard.com [mailto:chip.child...@sungard.com] On Behalf Of Chip Childers Sent: Tuesday, October 22, 2013 12:31 AM To: Subject: [ANNOUNCE] New PMC member: Animesh Chaturvedi The Project Management Committee (PMC)

Re: Review Request 14827: hyperv unit tests

2013-10-27 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14827/#review27588 --- Ship it! Ship It! - Devdeep Singh On Oct. 23, 2013, 11:05 a.m

Re: Review Request 13735: CLOUDSTACK-4450: Possibility of /tmp/xapilog filling up the Root disk on Xenserver.

2013-10-27 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13735/#review27596 --- Ship it! Ship It! - Devdeep Singh On Aug. 22, 2013, 3:45 p.m

[Merge] hyperv branch to master

2013-10-28 Thread Devdeep Singh
Hi, I would like to merge the support for Hyperv to the master branch. Development for this has been done by Donal, Rajesh, Anshul and I on branch [1]. The feature was proposed for merge earlier [3] but unit tests for hyperv agent code were requested [4]. Checklist: The development for the fea

RE: [Merge] hyperv branch to master

2013-10-31 Thread Devdeep Singh
sions for more in depth explaination of the changes. Darren On Tue, Oct 29, 2013 at 9:35 AM, David Nalley wrote: > On Mon, Oct 28, 2013 at 7:41 AM, Devdeep Singh > wrote: >> Hi, >> >> I would like to merge the support for Hyperv to the master branch. >> Developm

RE: [Merge] hyperv branch to master

2013-11-04 Thread Devdeep Singh
The changes have been merged to the master branch. Regards, Devdeep -Original Message- From: Devdeep Singh [mailto:devdeep.si...@citrix.com] Sent: Thursday, October 31, 2013 3:38 PM To: dev@cloudstack.apache.org Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar Subject: RE: [Merge

Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh
commit id f9f196a46ad820198894ef1ebba3d61413d5da59. - Devdeep Singh On Sept. 5, 2013, 8:29 a.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh
commit id f9f196a46ad820198894ef1ebba3d61413d5da59. - Devdeep Singh On Sept. 5, 2013, 8:29 a.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

RE: [ACS43] Schedule reminder : feature freeze this Friday 11/08

2013-11-08 Thread Devdeep Singh
Merged the changes to master. Regards, Devdeep -Original Message- From: Chip Childers [mailto:chip.child...@gmail.com] Sent: Friday, November 8, 2013 12:29 AM To: dev@cloudstack.apache.org Subject: Re: [ACS43] Schedule reminder : feature freeze this Friday 11/08 I was off - this morning

RE: uploading to maven repository

2013-11-12 Thread Devdeep Singh
Hi Alex, One of the reasons we have a copy of the java bindings is because cloudstack makes a customization to set the connection timeout. Will these customizations be kept when the stubs are uploaded to maven? Regarding, Devdeep -Original Message- From: Alex Huang [mailto:alex.hu...@c

Re: Review Request: CLOUDSTACK-1901: API:MS: DeleteEvents deletes Archived events as well

2013-05-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11177/#review20750 --- Ship it! Ship It! - Devdeep Singh On May 17, 2013, 12:56 p.m

Re: Review Request: CLOUDSTACK-1901: API:MS: DeleteEvents deletes Archived events as well

2013-05-19 Thread Devdeep Singh
> On May 20, 2013, 6:44 a.m., Devdeep Singh wrote: > > Ship It! Committed to master. Kindly close the request. - Devdeep --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11177/#re

Re: Review Request: CLOUDSTACK -2297 : Delete Account/Domain is not updating the resources usage of the parent domain

2013-05-20 Thread Devdeep Singh
> On May 20, 2013, 8:53 a.m., Devdeep Singh wrote: > > Ship It! Committed in commid-id a2fea4d4499a963a7bb5aed73b4ee03016684529. Kindly close the request. - Devdeep --- This is an automatically generated e-mail. To reply, vis

Re: Review Request: CLOUDSTACK -2297 : Delete Account/Domain is not updating the resources usage of the parent domain

2013-05-20 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11081/#review20754 --- Ship it! Ship It! - Devdeep Singh On May 14, 2013, 6:55 a.m

RE: [ANNOUNCE] New committer: Sailaja Mada

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

RE: [ANNOUNCE] New committer: Venkata Swamy

2013-05-23 Thread Devdeep Singh
Congrats Swamy. Regards, Devdeep > -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 > Venka

RE: [ANNOUNCE] New committer: Wei Zhou

2013-05-23 Thread Devdeep Singh
Congrats Regards, Devdeep > -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 becom

Re: Review Request: FIxed CLOUDSTACK-2783: listHypervisorCapabilities API response to include all the fields

2013-06-05 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11564/#review21470 --- Ship it! Ship It! - Devdeep Singh On May 31, 2013, 9:05 a.m

Re: Review Request: FIxed CLOUDSTACK-2783: listHypervisorCapabilities API response to include all the fields

2013-06-05 Thread Devdeep Singh
> On June 5, 2013, 8:34 a.m., Devdeep Singh wrote: > > Ship It! Committed the changes. Commit id: 4da995238b0f4b9d4109afaa505e9213e07486f5. Kindly close the review request - Devdeep --- This is an automatically generated e

Re: Review Request: CLOUDSTACK-1960: Key remapping fails to input the keys \ | ? of UK Keyboard

2013-06-05 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11650/#review21472 --- Can someone familiar with localization look at it? - Devdeep Singh

Re: Review Request: CLOUDSTACK-2824: API:MS:Ability to delete events and alerts-Auto Purge for Alerts doesn't work

2013-06-05 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11623/#review21474 --- Ship it! Changes look fine. Committed to master. - Devdeep Singh

RE: StoragePoolForMigrationResponse and StoragePoolResponse

2013-06-06 Thread Devdeep Singh
Hi, StoragePoolResponse should really only be used for listing storage pools. Putting a suitableformigration flag etc. makes it weird for other apis. If tomorrow the response object is updated to include more statistics for admin user to make a better decision, then such information gets pushed

RE: StoragePoolForMigrationResponse and StoragePoolResponse

2013-06-06 Thread Devdeep Singh
ses just for some specific apis. In some way, > suitableFormigration is some kind of attribute that can be set on a storage > pool or not. If you don't want to show it to listStoragePool call, you can > set that > as null so that json serialization will ignore it. > > Just my two cen

Re: Review Request: CLOUDSTACK-2286: Volume created from snapshot state is in allocated state instead of Ready state which is letting Primary storage not to increment the resources

2013-06-07 Thread Devdeep Singh
/VolumeServiceImpl.java <https://reviews.apache.org/r/11298/#comment44616> Why not get an updated volumeObject before calling processEvent (from the volFactory like you are doing in the callback)? That way the change is not required in the callback. - Devdeep Singh On May 21, 2013, 8:

Re: Review Request: CLOUDSTACK-2167: The Vlan ranges displayed are not in ascending order.

2013-06-07 Thread Devdeep Singh
tps://reviews.apache.org/r/11708/#comment44617> How about using the StringUtils.join? It'll remove the need of appending the string together and them trimming it to remove the trailing separator. - Devdeep Singh On June 7, 2013, 10:36 a.m., Saksham Sriva

Re: Review Request: CLOUDSTACK-2286: Volume created from snapshot state is in allocated state instead of Ready state which is letting Primary storage not to increment the resources

2013-06-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11298/#review21575 --- Ship it! Ship It! - Devdeep Singh On June 7, 2013, 11:41 a.m

Re: Review Request 12101: CLOUDSTACK-3082: System VMs are failed to start with Xen 6.2.0( Failing to create VIF's)

2013-06-26 Thread Devdeep Singh
. Verify that system vms and instances are coming up on xenserver 6.1 or an older version 2. Confirm that besides system vms, instance creation also works on xen 6.2 - Devdeep Singh On June 26, 2013, 7:58 a.m., Sanjay Tripathi wrote

Re: Review Request 11324: remove dead allocations

2013-06-27 Thread Devdeep Singh
nerated e-mail. To reply, visit: > https://reviews.apache.org/r/11324/ > --- > > (Updated June 17, 2013, 1:40 p.m.) > > > Review request for cloudstack and Devdeep Singh. > > > Repository: cloudstack-git >

Re: Review Request 12028: CLOUDSTACK-3064: Able to create VM from different account of the same domain without using Affinity group even the Zone is dedicated to an Account.

2013-06-27 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12028/#review22482 --- Ship it! Ship It! - Devdeep Singh On June 21, 2013, 2:09 p.m

Re: Review Request 12028: CLOUDSTACK-3064: Able to create VM from different account of the same domain without using Affinity group even the Zone is dedicated to an Account.

2013-06-27 Thread Devdeep Singh
> On June 27, 2013, 7:04 p.m., Devdeep Singh wrote: > > Ship It! Committed the changes. Please close the request. - Devdeep --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12028/#re

Re: Review Request 12149: Fix for template download failure for vmware deployments on cloudstack

2013-06-28 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12149/#review22518 --- Ship it! Ship It! - Devdeep Singh On June 28, 2013, 2:10 a.m

Re: Review Request 12149: Fix for template download failure for vmware deployments on cloudstack

2013-06-28 Thread Devdeep Singh
> On June 28, 2013, 5:52 p.m., Devdeep Singh wrote: > > Ship It! Committed in 78922589bbdc7914b9d4ce3b97a9fcf03d4b7b57. Kindly close the request - Devdeep --- This is an automatically generated e-mail. To reply, vis

Re: Review Request 12101: CLOUDSTACK-3082: System VMs are failed to start with Xen 6.2.0( Failing to create VIF's)

2013-06-28 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12101/#review22554 --- Ship it! Ship It! - Devdeep Singh On June 26, 2013, 7:58 a.m

Re: Review Request 12101: CLOUDSTACK-3082: System VMs are failed to start with Xen 6.2.0( Failing to create VIF's)

2013-06-28 Thread Devdeep Singh
> On June 29, 2013, 12:29 a.m., Devdeep Singh wrote: > > Ship It! Kindly close the review request - Devdeep --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12101/#re

RE: Secondary Storage/TinyLinux

2013-07-01 Thread Devdeep Singh
When you see it under Templates, what is the property of 'Ready' attribute of the template? Regards, Devdeep > -Original Message- > From: Soheil Eizadi [mailto:seiz...@infoblox.com] > Sent: Monday, July 01, 2013 4:06 PM > To: dev@cloudstack.apache.org > Subject: Secondary Storage/TinyLin

Re: Review Request 11906: CLOUDSTACK-1047: tracking in logs using job id

2013-07-09 Thread Devdeep Singh
re an updated patch. - Devdeep Singh On July 1, 2013, 2:50 p.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Review Request 11906: CLOUDSTACK-1047: tracking in logs using job id

2013-07-10 Thread Devdeep Singh
request. - Devdeep Singh On July 10, 2013, 7:03 a.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12245: CLOUDSTACK-1960: Fix for UK Keyboard Bar symbol.

2013-07-10 Thread Devdeep Singh
master and 4.2 branch. Kindly close the request. - Devdeep Singh On July 3, 2013, 11:43 a.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 11649: CLOUDSTACK-1768: Ability to delete Events and Alerts: Delete by a time period is required.

2013-07-10 Thread Devdeep Singh
rc/com/cloud/alert/dao/AlertDaoImpl.java <https://reviews.apache.org/r/11649/#comment46776> Same as above. start date should be removed (it will never get here) engine/schema/src/com/cloud/event/dao/EventDaoImpl.java <https://reviews.apache.org/r/11649/#comment46777> Same as above - D

Re: Review Request 11649: CLOUDSTACK-1768: Ability to delete Events and Alerts: Delete by a time period is required.

2013-07-11 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11649/#review23007 --- Ship it! Committed to master and 4.2 - Devdeep Singh On July 11

Re: Review Request 12467: CLOUDSTACK-3462: NumberFormatException with TemplateService while performing template synchronization.

2013-07-11 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12467/#review23008 --- Ship it! Committed to master and 4.2 - Devdeep Singh On July 11

RE: [acs42] [live storage migration]

2013-07-12 Thread Devdeep Singh
In your setup, there may not have been a suitable storage pool available for a volume to be migrated too. That is why the list is empty. Which hypervisor (and version) are you trying? How may storage pools are there in the cluster? Regards, Devdeep From: Paul Angus [mailto:paul.an...@shapeblue.

RE: [acs42] [live storage migration]

2013-07-12 Thread Devdeep Singh
be improved to handle this. Please open another bug request for the same. > > > > Regards, > > Paul Angus > S: +44 20 3603 0540 | M: +447711418784 > paul.an...@shapeblue.com > > -Original Message- > From: Devdeep Singh [mailto:devdeep.si...@citrix.com] > Sent: 12

RE: Template Question

2013-07-18 Thread Devdeep Singh
Changes were made recently to allow some commands to execute in parallel on a hypervisor resource. Maybe that is causing it. Regards, Devdeep > -Original Message- > From: Koushik Das [mailto:koushik@citrix.com] > Sent: Thursday, July 18, 2013 3:11 PM > To: dev@cloudstack.apache.org >

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

2013-07-18 Thread Devdeep Singh
taking place. That isn't handled right now. 2. The check to raise alerts should be abstracted into a function. 3. In case of implicit the scenario when a vm is migrated from a dedicated host of one account to dedicated host of another account isn't handled. - Devdeep Singh On July 18

Re: Review Request 12277: Add VHDX image support

2013-07-18 Thread Devdeep Singh
876a7b3361a17bc16b1342fdebbff0f0306674a6. - Devdeep Singh On July 15, 2013, 9:12 a.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12277: Add VHDX image support

2013-07-18 Thread Devdeep Singh
> On July 18, 2013, 1:36 p.m., Devdeep Singh wrote: > > Committed to master with commit id 876a7b3361a17bc16b1342fdebbff0f0306674a6. Kindly close the request. - Devdeep --- This is an automatically generated e-mail. To rep

Re: Review Request 12747: Fix configuration of Jetty-based execution so that CloudStack can execute scripts properly

2013-07-18 Thread Devdeep Singh
, it'll be great. - Devdeep Singh On July 18, 2013, 9:34 p.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Review Request 12222: CLOUDSTACK-2510: Getting NPE while executing listStoragePools API command.

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1/#review23501 --- Ship it! Ship It! - Devdeep Singh On July 2, 2013, 5:26 a.m

Re: Review Request 12544: CLOUDSTACK-3496: [Regression]Primary Storage resource usage is not getting updated if detached volume is deleted.

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12544/#review23503 --- Ship it! Ship It! - Devdeep Singh On July 15, 2013, 9:16 a.m

Re: Review Request 12629: CLOUDSTACK-3492: [Regression]Update Resource Count for an account is updating the primary storage incorretly

2013-07-19 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12629/#review23502 --- Ship it! Ship It! - Devdeep Singh On July 17, 2013, 5:11 a.m

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

2013-07-21 Thread Devdeep Singh
f the given destination host is dedicated. We shouldn't be required to list all dedicated hosts and parse through them to check if destination is dedicated. - Devdeep Singh On July 19, 2013, 11:52 a.m., Saksham Srivastava wrote: > > ---

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

2013-07-23 Thread Devdeep Singh
89b94bb75 respectively. - Devdeep Singh On July 22, 2013, 11:28 a.m., Saksham Srivastava wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12838: CLOUDSTACK-3488: listStoragePools throws NPE if any of the storage pools are not in the right state.

2013-07-26 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12838/#review23935 --- Ship it! Looks good. Kindly close the request. - Devdeep Singh

Re: Review Request 12898: CLOUDSTACK-3544: Xen 6.2 hosts are tagged with Xen 6.1.

2013-07-26 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12898/#review23936 --- Ship it! Changes pushed to 4.2 and master - Devdeep Singh On

Re: Review Request 12965: CLOUDSTACK-3780: G11N: JA: Garbled code display on Pod configuration & Host configuration pages during "Add zone".

2013-07-26 Thread Devdeep Singh
and following the pattern in the rest of the file. - Devdeep Singh On July 26, 2013, 10:03 a.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 11858: CLOUDSTACK-2340 [AWS Style Health Checks] Response of the API listLoadBalancerRuleInstances should show the service state of a VM if health check is configured for it

2013-07-26 Thread Devdeep Singh
incorporated. Applied the patch. Kindly close the request. - Devdeep Singh On July 10, 2013, 12:37 p.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12996: Unable to Resize Volume (kvm, vmware)

2013-07-29 Thread Devdeep Singh
. - Devdeep Singh On July 27, 2013, 3:05 p.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12997: usage event is not generated for volume resize

2013-07-29 Thread Devdeep Singh
. - Devdeep Singh On July 27, 2013, 3:11 p.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 12994: CLOUDSTACK-3216 Logs in the Software router are not being rotated

2013-07-29 Thread Devdeep Singh
. - Devdeep Singh On July 27, 2013, 1:09 p.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13015: CLOUDSTACK-1961: Common issues found in English OS with EN-US standard Keyboard.

2013-07-29 Thread Devdeep Singh
and give an updated patch. - Devdeep Singh On July 29, 2013, 1:04 p.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13057: CLOUDSTACK-2909: patch to fix TAB in US keyboard.

2013-07-30 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13057/#review24226 --- Ship it! Ship It! - Devdeep Singh On July 30, 2013, 8:50 a.m

Re: Review Request 13015: CLOUDSTACK-1961: Common issues found in English OS with EN-US standard Keyboard.

2013-07-30 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13015/#review24225 --- Ship it! Ship It! - Devdeep Singh On July 29, 2013, 1:04 p.m

Re: Review Request 13267: CLOUDSTACK-4075: User unable to archive events.

2013-08-05 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13267/#review24648 --- Ship it! Ship It! - Devdeep Singh On Aug. 5, 2013, 12:54 p.m

RE: CloudStack Storage Plugin for CloudByte

2013-08-07 Thread Devdeep Singh
The next release will probably be 4.3. You can target the CloudByte plugin for that release. The can find more information on CloudStack release cycle here [1]. [1] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Releases Regards, Devdeep > -Original Message- > From: Amit Das [m

Re: Review Request 13316: CLOUDSTACK-3723: System VM Template for XenServer has wrong guest OS type Information on the Database.

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13316/#review24782 --- Ship it! Ship It! - Devdeep Singh On Aug. 6, 2013, 9:14 a.m

Re: Review Request 12747: Fix configuration of Jetty-based execution so that CloudStack can execute scripts properly

2013-08-07 Thread Devdeep Singh
49c9fbfb70413f86642956423c4bbba2e43d8aec. Kindly mark the request as submitted. - Devdeep Singh On July 18, 2013, 9:34 p.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13324: CLOUDSTACK-4104: VM Creation should fail if a zone is dedicated to a domain and user of the same domain deploys vm without using affinity group "Explicit Dedication"

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13324/#review24796 --- Ship it! Ship It! - Devdeep Singh On Aug. 6, 2013, 12:53 p.m

Re: Review Request 13370: CLOUDSTACK-4101: Update the Xenserver template/checksum in template.sql to the one verified.

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13370/#review24797 --- Ship it! Ship It! - Devdeep Singh On Aug. 7, 2013, 8:12 a.m

Re: Review Request 13372: CLOUDSTACK-4062: SLES 11 SP2, SLES 11 SP3, CENTOS 6.4, RHEL 6.3, RHEL 6.4 OS type missing from OS Type drop down list when registering template.

2013-08-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13372/#review24798 --- Ship it! Ship It! - Devdeep Singh On Aug. 7, 2013, 9:07 a.m

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

2013-08-08 Thread Devdeep Singh
Sent: Thursday, August 08, 2013 6:40 PM > To: dev@cloudstack.apache.org; cloudstack-iss...@incubator.apache.org > Cc: Devdeep Singh; Prachi Damle > Subject: RE: [jira] [Commented] (CLOUDSTACK-4179) [Performance Testing] > Time taken for Deploy VM async job to complete is considerably h

Re: Review Request 13428: CLOUDSTACK-4187: 2.2.1 - 4.2 Upgrade: Few Guest OS Types information is inconsistent on the Upgraded Setup when compared with the fresh 4.2 Installation.

2013-08-09 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13428/#review24907 --- Ship it! Ship It! - Devdeep Singh On Aug. 8, 2013, 12:25 p.m

Re: Review Request 13431: CLOUDSTACK-3842: G11N: SC: Some regression issues occurred on build#4.2-256.

2013-08-09 Thread Devdeep Singh
applied successfully on 4.2 but is failing on master. - Devdeep Singh On Aug. 9, 2013, 6:27 a.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13438: CLOUDSTACK-2448: XenServer Guest OS Mapping Corrections.

2013-08-09 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13438/#review24913 --- Ship it! Ship It! - Devdeep Singh On Aug. 9, 2013, 11:07 a.m

Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.

2013-08-12 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13469/#review24985 --- Ship it! Ship It! - Devdeep Singh On Aug. 12, 2013, 6:48 a.m

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

2013-08-12 Thread Devdeep Singh
See if this helps https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=history;f=setup/db/db/schema-410to420.sql;h=60335318c7b792a401cab36b8b456e652d47f2a6;hb=e61a38087d158da51f7b945d16e4749a83512ab5 Regards, Devdeep > -Original Message- > From: Koushik Das [mailto:

Re: Review Request 13493: CLOUDSTACK-2909: Fix for ; and : keys for IE and chrome.

2013-08-12 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13493/#review25000 --- Ship it! Ship It! - Devdeep Singh On Aug. 12, 2013, 9:02 a.m

Re: Review Request 13495: CLOUDSTACK-4189: [UI] In multi zone setup, dedicating a single zone to a domain, shows other zones dedicated as well

2013-08-12 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13495/#review24999 --- Ship it! Ship It! - Devdeep Singh On Aug. 12, 2013, 12:28 p.m

RE: Unable to create link local network exception

2013-08-13 Thread Devdeep Singh
This looks similar to https://issues.apache.org/jira/browse/CLOUDSTACK-1236. I'll increase the priority and work on addressing it. Regards, Devdeep > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Wednesday, August 14, 2013 3:25 AM > To: dev@cloud

Re: Review Request 13641: Fix for build failure in 4.2

2013-08-19 Thread Devdeep Singh
6c593b0b16f37951c0c30b2ad3cff69835085fe2 - Devdeep Singh On Aug. 19, 2013, 6:26 a.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

RE: System VMs not running

2013-08-22 Thread Devdeep Singh
Did you seed the secondary storage with the systemvm template before starting the setup? If not, can you check if under your secondary storage the ../template/tmpl/1/1 directory is present? Regards, Devdeep > -Original Message- > From: Amit Das [mailto:amit@cloudbyte.com] > Sent: Th

[ACS42] cherry pick commit 5b319ba0cb46160451ec05de32a8409b44a08f5b

2013-08-23 Thread Devdeep Singh
Hi Animesh, This is a fix for bug https://issues.apache.org/jira/browse/CLOUDSTACK-4470. Can you cherry pick commit 5b319ba0cb46160451ec05de32a8409b44a08f5b to 4.2 branch? Regards, Devdeep

Re: Review Request 13877: CIFS Secondary Storage

2013-09-11 Thread Devdeep Singh
master branch. - Devdeep Singh On Aug. 28, 2013, 8:58 p.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 13877: CIFS Secondary Storage

2013-09-11 Thread Devdeep Singh
- Devdeep Singh On Aug. 28, 2013, 8:58 p.m., Donal Lafferty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request: Summary: Fix for CLOUDSTACK-1537 - Restart network with clean up set to true causes Autoscaled LB rule to get mangled and unusable

2013-03-21 Thread Devdeep Singh
. - Devdeep Singh On March 21, 2013, 11:19 a.m., Vijay Venkatachalam wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request: Storage motion changes for xenserver

2013-03-29 Thread Devdeep Singh
applies cleanly. Thanks, Devdeep Singh

Review request for storage motion on xenserver

2013-03-29 Thread Devdeep Singh
I have put the feature proposed [1] and developed in the feature branch [2] up for review. Code for this feature conforms to what was proposed in FS [3]. The patch available at [4]. It includes marvin tests and unit tests for verifying the functionality. Please take a look at it and let me know

RE: [DISCUSS] Enabling storage xenmotion on xenserver 6.1

2013-04-05 Thread Devdeep Singh
on in the api to say > it's findHostsForMigration. > - The above will invalidate open question #4. > - The same goes for open question #5. > - I apologize for this. I think we discussed this before but I've forgotten. > Why > a separate api for migrateVmWithVolume? Why

RE: Review request for storage motion on xenserver

2013-04-05 Thread Devdeep Singh
on multiple > volumes at one time. > > 4. Don't need to add a new api called migrateasync, copyasync has the same > meaning. > > Does it make sense? > > > -Original Message- > > From: Devdeep Singh [mailto:devdeep.si...@citrix.com] > > Sent: Fr

Re: Review Request: Syslog messages not getting generated for the "alertType:: 0"

2013-04-10 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10392/#review18941 --- Ship it! Ship It! - Devdeep Singh On April 10, 2013, 8:53 a.m

Re: Review Request: Syslog messages not getting generated for the "alertType:: 0"

2013-04-10 Thread Devdeep Singh
> On April 10, 2013, 9:18 a.m., Devdeep Singh wrote: > > Ship It! Looks good. Someone with appropriate rights needs to pick up the patch and apply it on master. - Devdeep --- This is an automatically generated e-mail. To rep

Re: Review Request: Storage motion changes for xenserver

2013-04-11 Thread Devdeep Singh
ays on the same host. 4.5 For volume migration verified that storage tags are honored. Other tests done to verify patch: 1. Verified that there are no rat failures. 2. Applied the patch to verify it applies cleanly. Thanks, Devdeep Singh

RE: Review request for storage motion on xenserver

2013-04-11 Thread Devdeep Singh
> -Original Message- > From: Devdeep Singh [mailto:devdeep.si...@citrix.com] > Sent: Friday, April 05, 2013 3:37 PM > To: dev@cloudstack.apache.org > Subject: RE: Review request for storage motion on xenserver > > Hi Edison, > > Thanks a lot for looking at the changes. I am

Re: Review Request: Storage motion changes for xenserver

2013-04-15 Thread Devdeep Singh
done to verify patch: 1. Verified that there are no rat failures. 2. Applied the patch to verify it applies cleanly. Thanks, Devdeep Singh

Review Request: NPE while migrating volume

2013-04-16 Thread Devdeep Singh
--- 1. migrated a volume from one storage pool to another and made sure it was successful. Attached it to a vm after migrating. 2. Also verified that the patch applies successfully. Thanks, Devdeep Singh

Re: Review Request: CLOUDSTACK-1966 Primary Storage Limits are not validated while deploying the VM

2013-04-16 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10541/#review19304 --- Ship it! Ship It! - Devdeep Singh On April 16, 2013, 11:47 a.m

Re: Review Request: CLOUDSTACK-1966 Primary Storage Limits are not validated while deploying the VM

2013-04-16 Thread Devdeep Singh
> On April 17, 2013, 6:21 a.m., Devdeep Singh wrote: > > Ship It! Changes look fine. Someone with the right karma needs to merge the change. - Devdeep --- This is an automatically generated e-mail. To reply, vis

Re: Review Request: CLOUDSTACL-2046 Primary Storage & Secondary Storage is max limit is set 0 with listResourceLimits API response as ROOT admin

2013-04-17 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10566/#review19312 --- Ship it! Ship It! - Devdeep Singh On April 17, 2013, 6:38 a.m

Re: Review Request: CLOUDSTACL-2046 Primary Storage & Secondary Storage is max limit is set 0 with listResourceLimits API response as ROOT admin

2013-04-17 Thread Devdeep Singh
> On April 17, 2013, 10:20 a.m., Devdeep Singh wrote: > > Ship It! Someone with the right karma should commit this change. - Devdeep --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.or

  1   2   3   >