[GitHub] cloudstack pull request: Implementation for the ability to disable...

2015-05-18 Thread devdeep
GitHub user devdeep opened a pull request: https://github.com/apache/cloudstack/pull/257 Implementation for the ability to disable a storage pool for provisioning of new volumes. Implementation for the ability to disable a storage pool for provisioning of new volumes. Following

[GitHub] cloudstack pull request: Implementation for the ability to disable...

2015-05-19 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/257#issuecomment-103402775 @terbolous FS for this can be found here https://cwiki.apache.org/confluence/display/CLOUDSTACK/Disabling+Storage+Pool+for+Provisioning. To answer your query on

Re: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

2015-11-21 Thread Devdeep
Have you seeded the secondary storage with the default systemvm template? Regards, Devdeep On Sat, Nov 21, 2015 at 8:33 AM, B Prakash wrote: > Thanks Jayapal, Abhinandan. > > log SMlog doesnt giveout much, I am able to mount and edit file on mount > from both server and agent. I

[GitHub] cloudstack pull request: This branch implements the CSV and native...

2015-07-19 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/351#issuecomment-122773398 I'll take a look and get back if I have any queries --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-07-29 Thread devdeep
GitHub user devdeep opened a pull request: https://github.com/apache/cloudstack/pull/635 CLOUDSTACK-8687: Prepare template only on a given storage pool Update prepare template api to seed/prepare a template only on a given primary storage. Currently, the prepare template api will

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-07-30 Thread devdeep
Github user devdeep commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/635#discussion_r35849928 --- Diff: api/src/org/apache/cloudstack/api/command/admin/template/PrepareTemplateCmd.java --- @@ -60,6 +61,15 @@ description

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-04 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-127850003 @DaanHoogland Added unit tests for prepare template functionality too. Can you take a look. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-06 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-128292255 @DaanHoogland When prepareTemplate is called, it either falls into prepareTemplateInOneStoragePool if one primary has to be seeded with the template; or

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-10 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-129411154 @DaanHoogland That is what I am looking into; how to get the number of threads that have been started by the executor service. --- If your project is set up for it

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-14 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-131020409 Added another test to validate the prepareTemplate function. Looked into if it was possible to mock the threadpool executor service in templatemanagerimpl. However

[GitHub] cloudstack pull request: This branch implements the CSV and native...

2015-08-16 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/351#issuecomment-131683283 Changes looks good to me. @remibergsma Are you ok with these changes? I'll go ahead and merge them then. --- If your project is set up for it, you can rep

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-17 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-132072881 @DaanHoogland added some additional tests to validate the templates get scheduled for seeding. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-18 Thread devdeep
Github user devdeep commented on the pull request: https://github.com/apache/cloudstack/pull/635#issuecomment-132439804 @remibergsma I'll close this pull request and generate a new one where the unit tests are squashed in one commit. --- If your project is set up for it, yo

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-18 Thread devdeep
Github user devdeep closed the pull request at: https://github.com/apache/cloudstack/pull/635 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack pull request: CLOUDSTACK-8687: Prepare template only on...

2015-08-18 Thread devdeep
GitHub user devdeep opened a pull request: https://github.com/apache/cloudstack/pull/715 CLOUDSTACK-8687: Prepare template only on a given storage pool Update prepare template api to seed/prepare a template only on a given primary storage. Currently, the prepare template api will

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
ding-your-microsoft-solution-with.html Regards, Devdeep

RE: [Merge] hyperv branch to master

2013-10-31 Thread Devdeep Singh
Thanks Darren. I'll start working on getting it merged into the master. I hope to finish it by Friday. Regards, Devdeep -Original Message- From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] Sent: Tuesday, October 29, 2013 10:45 PM To: dev@cloudstack.apache.org Cc:

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

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

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

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

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 ha

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
pushed in there which makes it unnatural for apis that just need the list of storage pools. I am planning to update StoragePoolForMigrationResponse to include the StoragePoolResponse object and any other flag; suitableformigration in this case. I'll file a bug for the same. Regards, De

RE: StoragePoolForMigrationResponse and StoragePoolResponse

2013-06-06 Thread Devdeep Singh
o not match then it is flagged as unsuitable. For another volume it may be flagged suitable. So it really isn't an attribute of a storage pool and I believe it doesn't belong in the StoragePoolResponse object. Regards, Devdeep > -Original Message- > From: Min Chen [mailt

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
> On June 18, 2013, 6:08 a.m., Prasanna Santhanam wrote: > > 93938ca716f85c24f2d9fb07c5856055f0061415 Dave, the request has been merged to master by Prasanna. Can you close this request. - Devdeep --- This is an auto

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 > S

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

RE: [acs42] [live storage migration]

2013-07-12 Thread Devdeep Singh
Hi Paul, Please see inline. Regards, Devdeep > -Original Message- > From: Paul Angus [mailto:paul.an...@shapeblue.com] > Sent: Friday, July 12, 2013 2:58 PM > To: dev@cloudstack.apache.org; Srikanteswararao Talluri > Subject: RE: [acs42] [live storage migration] > >

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@cloudstac

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: Am

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
Hi Koushik, I have updated the bug with my comments. We don't have to loop through resources and check them one at a time. We need to run a better query and add all the dedicated resources to the avoid set. Regards, Devdeep > -Original Message- > From: Koushik Das >

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: Koush

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:2

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]

[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

  1   2   3   >