Re: Review Request: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-03-20 Thread Nitin Mehta
: Animesh Chaturvedi mailto:animesh.chaturv...@citrix.com>> Reply-To: Animesh Chaturvedi mailto:animesh.chaturv...@citrix.com>> Date: Wednesday 20 March 2013 10:26 PM To: Nitin Mehta mailto:nitin.me...@citrix.com>> Cc: Animesh Chaturvedi mailto:animesh.chaturv...@citrix.co

Review Request: StatsCollector was not running, fixed it

2013-03-20 Thread Nitin Mehta
/r/10043/diff/ Testing --- UI now shows stats for vm, host and the storage now. Thanks, Nitin Mehta

Re: [MERGE] scaleupvm functionality branch to master

2013-03-19 Thread Nitin Mehta
Chip - Thanks for looking into this. Please find my answers inline. On 18/03/13 10:34 PM, "Chip Childers" wrote: >On Mon, Mar 18, 2013 at 04:10:25PM +, Nitin Mehta wrote: >> I would like to merge scale up vm functionality branch to master >> >> Spec :

[MERGE] scaleupvm functionality branch to master

2013-03-18 Thread Nitin Mehta
I would like to merge scale up vm functionality branch to master Spec : https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dynamic+scaling+of+CPU+and+RAM JIRA ticket : https://issues.apache.org/jira/browse/CLOUDSTACK-658 Branch: Scaleupvm Notes:- 1. I have removed the vm state change in

Re: Experimenting on Resource Allocator in CloudStack

2013-03-17 Thread Nitin Mehta
loudStack >source such that it works accordingly. Am I on right track? >Kausal Malladi >www.kausalmalladi.tk > > >On Sun, Mar 17, 2013 at 6:31 PM, Nitin Mehta >wrote: >> In addition you can refer to >> >>http://ke4qqq.fedorapeople.org/CloudStack-2.2-Allocato

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts

2013-03-17 Thread Nitin Mehta
<https://reviews.apache.org/r/9541/#comment38121> increment needs to check for exceeding once again correct because the check happenned before snapshot creation - Nitin Mehta On March 13, 2013, 5:44 a.m., Sanjay Tripathi

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts

2013-03-17 Thread Nitin Mehta
eed to check primary storage depending on the global config right ? - Nitin Mehta On March 13, 2013, 5:44 a.m., Sanjay Tripathi wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://r

Re: Extending the CloudStack API

2013-03-17 Thread Nitin Mehta
Would you mind sharing your use case ? We could possibly put it in CS itself. On 16/03/13 11:30 PM, "Mike Tutkowski" wrote: >Awesome! I might have a use case for that (just thought of this after >talking to a couple customers yesterday). > > >On Sat, Mar 16, 2013 at 11:52 AM, Chip Childers >wrot

Re: Experimenting on Resource Allocator in CloudStack

2013-03-17 Thread Nitin Mehta
In addition you can refer to http://ke4qqq.fedorapeople.org/CloudStack-2.2-Allocator_Implementation-en-U S.pdf as well. On 16/03/13 10:23 PM, "Alex Huang" wrote: >Kausal, > >Resource allocation is all done by implementations of DeploymentPlanner. >You can go to that interface and then see which

Re: Review Request: (CLOUDSTACK-1475) update keystore in SSVM and change download iso/template url after Update SSL Certificate

2013-03-15 Thread Nitin Mehta
> On March 15, 2013, 5:56 a.m., Nitin Mehta wrote: > > Can you please also tell me a list of tests done for this ? > > Wei Zhou wrote: > Nitin, I add a list of tests in "Testing Done" part. Please have a look. > > consoleproxy.url.domain is no

Re: Review Request: (CLOUDSTACK-1475) update keystore in SSVM and change download iso/template url after Update SSL Certificate

2013-03-14 Thread Nitin Mehta
<https://reviews.apache.org/r/9696/#comment37962> can you please use SecondaryStorageVmManager instead ? server/src/com/cloud/storage/upload/UploadMonitorImpl.java <https://reviews.apache.org/r/9696/#comment37963> can you put an example here...seems some hardcoding - Nitin Mehta On March

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-03-14 Thread Nitin Mehta
> >Sorry to reply late, have been busy working on a patch release for >customer. > >AsyncJob manager does provide a mechanism that you can do synchronized the >job execution against a object. You may check out >AsyncJobManager.syncAsyncJobExecution(). > >Kelven

Re: Local versus Shared Storage Type for Compute Offering

2013-03-13 Thread Nitin Mehta
I don¹t think this behavior changed ever. The reason for the hard check should be as follows Imagine the tag is for premium offering and the customer is paying premium for that. Now he says deploy vm with this offering and say there is no host with premium tag left to deploy the vm. Now if we don'

Re: [ACS402] Status of 4.0.2 - two blocker bugs remaining

2013-03-12 Thread Nitin Mehta
An update on this, I am unable to test my work because of the blocker @ https://issues.apache.org/jira/browse/CLOUDSTACK-1648 Thanks, -Nitin On 07/03/13 2:31 PM, "Nitin Mehta" wrote: >Joe - I will take up 643, but might take a day or two to put in the fix as >its not straight

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-03-12 Thread Nitin Mehta
;VM States are different because there can be outside changes (through >other vm managers) that cause vm life cycle to behave differently. > >--Alex > >> -Original Message- >> From: Nitin Mehta >> Sent: Saturday, March 9, 2013 2:35 AM >> To: cloudstack-dev@in

Re: Review Request: CLOUDSTACK-924: source_template_id set to NULL when a template is created from a snapshot

2013-03-11 Thread Nitin Mehta
Pushed with commit 75261eb317a763114a6054664f4adf3f7a2c2709 Please build and run it at your end From: Saksham Srivastava mailto:saksham.srivast...@citrix.com>> Reply-To: Saksham Srivastava mailto:saksham.srivast...@citrix.com>> To: Nitin Mehta mailto:nitin.me...@citrix.com>&g

Re: Review Request: Bugfix CLOUDSTACK-1594: Secondary storage host always remains Alert status

2013-03-10 Thread Nitin Mehta
Roxanne - Thanks very much for working on this. My take on the fix was to remove the entry for secondary storage from host table as we do not create an agent for it. We should rather have another table for secondary storage and related info. On 11/03/13 7:59 AM, "roxanne chang" wrote: > >---

Re: [MERGE] Support VM Snapshot

2013-03-10 Thread Nitin Mehta
ed, not sure if >it's a good idea to handle lock sync in them. > >Regards >Mice > >-Original Message- >From: Nitin Mehta [mailto:nitin.me...@citrix.com] >Sent: Saturday, March 09, 2013 12:59 AM >To: cloudstack-dev@incubator.apache.org; Alex Huang >Cc: An

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-03-09 Thread Nitin Mehta
Hi Alex, I had one more question. Say the MS is shut down or restarted, when do we clear this attribute in this case ? On 08/03/13 6:01 PM, "Nitin Mehta" wrote: >Alex - Thanks very much for pointing out earlier this week that for >scaling up the vm we shouldn't change the

Re: [MERGE] Support VM Snapshot

2013-03-08 Thread Nitin Mehta
Not to nitpick but for better understanding of what Alex's comment on the vm lifecycle. I went through the commit and saw the code change below. This change successfully avoids other vm state changes when the vm is in snapshotting operation but what about the case when the user triggers "vm snapsh

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-03-08 Thread Nitin Mehta
(say during failure) how do I do it w/o coupling the code changes or is it ok for now to do so ? Thanks, -Nitin On 15/02/13 5:42 AM, "Hari Kannan" wrote: >Hi Nitin, > >Please see below > >Hari > >-Original Message- >From: Nitin Mehta [mailto:nitin

Re: [ACS402] Status of 4.0.2 - two blocker bugs remaining

2013-03-07 Thread Nitin Mehta
Joe - I will take up 643, but might take a day or two to put in the fix as its not straightforward. Kishan has taken up 1151 as he is more familiar. On 07/03/13 5:22 AM, "Joe Brockmeier" wrote: >Hey all, > >A quick update on the status of 4.0.2. We have two blocker bugs that >need attention: > >

Re: Trouble with system VMs on master

2013-03-06 Thread Nitin Mehta
Rohit - are you saying that you see it in Running state and not starting state as Mike says ? Mike - do the Ms logs point at something ? Can you paste them ? On 07/03/13 11:21 AM, "Rohit Yadav" wrote: >On Thu, Mar 7, 2013 at 10:35 AM, Mike Tutkowski > wrote: >> Hi, >> >> One of the branches I'm

Re: Master branch build is broken again

2013-03-05 Thread Nitin Mehta
s fix on these problems alone. > >Nitin, > >Shouldn't a change this big be in a feature branch, with a MERGE request >to bring it into master? > >> >> --Alex >> >> > -Original Message- >> > From: Alex Huang [mailto:alex.hu...@citr

Re: Review Request: CLOUDSTACK-924: source_template_id set to NULL when a template is created from a snapshot

2013-03-05 Thread Nitin Mehta
Hi Animesh, Saksham is supposed to send a rebased request to me. Below is what he wrote to me. The code has changed a lot after the storage refactoring. I will submit a new patch after resolving the conflicts. Saksham On Thursday 28 February 2013 12:03 PM, Nitin Mehta wrote: >Nitins-MacB

Re: Master branch build is broken again

2013-03-04 Thread Nitin Mehta
AM >> To: cloudstack-dev@incubator.apache.org >> Cc: Nitin Mehta >> Subject: Master branch build is broken again >> Importance: High >> >> Hi there, >> >> I could not compile master after pulling latest code today, build >>failed with &

Re: cloudstack management server resource allocated

2013-03-01 Thread Nitin Mehta
er host and crate too much VMS in the all cloudstack >invironment? > >2013/2/28 Nitin Mehta > >> Please refer to >> https://cwiki.apache.org/CLOUDSTACK/understanding-dashboard.html for the >> first question you asked. >> >> Let me know if you have furth

Re: HA question

2013-02-27 Thread Nitin Mehta
loudstack-dev@incubator.apache.org >Cc: cloudstack-dev@incubator.apache.org >Subject: Re: HA question > >I could get one started. > >I've saved this thread, and I'll try to get some stuff up in the next >couple days. > >Sent from my iPhone > >On Feb 27, 2013, at

Re: HA question

2013-02-27 Thread Nitin Mehta
T) [mailto:kel...@backbonetechnology.com] On >>>Behalf >>> Of kdam...@apache.org >>> Sent: Wednesday, February 27, 2013 8:51 PM >>> To: cloudstack-dev@incubator.apache.org >>> Cc: Hari Kannan >>> Subject: RE: HA question >>> >>> S

Re: HA question

2013-02-27 Thread Nitin Mehta
CS has its own HA logic and doesn't use the native HA of the HV and so the question for enabling the HA for hosts doesn't arise. This is true for XS. For Vmware and KVM, I will let the guru's speak :) On 28/02/13 9:55 AM, "kdam...@apache.org" wrote: >Thanks that¹s awesome, but not quite the answ

vm state transition related question

2013-02-27 Thread Nitin Mehta
On vm state transition I need to communicate some extra information to one of the listeners which is listening for the transition. How do I achieve that ? It is not generic enough and specific to only one of the listeners. My use case is as follows When vm is dynamically scaled up on a host I ne

Re: cloudstack management server resource allocated

2013-02-27 Thread Nitin Mehta
Please refer to https://cwiki.apache.org/CLOUDSTACK/understanding-dashboard.html for the first question you asked. Let me know if you have further questions on that On 27/02/13 5:56 PM, "wenbing yu" wrote: >The environment of my cloudstack deployed is about cloudstack 4.0.1 and >Hypervisor KV

Re: Review Request: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-02-27 Thread Nitin Mehta
: patch failed: server/src/com/cloud/template/TemplateManagerImpl.java:843 error: server/src/com/cloud/template/TemplateManagerImpl.java: patch does not apply From: Nitin Mehta mailto:nitin.me...@citrix.com>> Reply-To: Nitin Mehta mailto:nitin.me...@citrix.com>> To: Nitin Mehta mai

Re: Review Request: when a template is deleted and then copied over again , it is still marked as "Removed" in template_zone_ref table.

2013-02-27 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7018/#review17183 --- Ship it! Ship It! - Nitin Mehta On Jan. 13, 2013, 5:14 p.m

Re: Error : cannot find class [com.cloud.storage.allocator.LocalStoragePoolAllocator]

2013-02-26 Thread Nitin Mehta
Culprit commit - 8e360f342d6e72ad3355008456bec13904fe3e2b ? Verify On 27/02/13 11:48 AM, "Rajesh Battala" wrote: >On Latest master while running the cloud am getting this Error. > >T/cloud-plugin-network-f5-4.2.0-SNAPSHOT.pom >cannot find class [com.cloud.storage.allocator.LocalStoragePoolAllo

Re: Review Request: CLOUDSTACK-1156: Limit Primary and Secondary storage for domain/accounts

2013-02-26 Thread Nitin Mehta
we not make all the increment business state transition based - like when volume goes from allocated to ready we increment the volume count and the size etc. ? - Nitin Mehta On Feb. 27, 2013, 5:21 a.m., Sanjay Tripathi wrote: > > ---

Re: New committer: Radhika Puthiyetath

2013-02-26 Thread Nitin Mehta
Congrats Radhika !! Keep up the tempo :) On 27/02/13 10:23 AM, "Devdeep Singh" wrote: >Congratulations Radhika. > >Regards, >Devdeep > >> -Original Message- >> From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] >> Sent: Wednesday, February 27, 2013 3:28 AM >> To: CloudStack Deve

Re: New committer: Sateesh Chodapuneedi

2013-02-26 Thread Nitin Mehta
Congrats Sateesh ! Great work On 27/02/13 7:47 AM, "Ahmad Emneina" wrote: >Congrats Sateesh! > > >On Tue, Feb 26, 2013 at 6:15 PM, Alex Huang wrote: > >> The Podling Project Management Committee (PPMC) for Apache CloudStack >>has >> asked Sateesh Chodapuneedi to become a committer and we are pl

Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size.

2013-02-25 Thread Nitin Mehta
> On Feb. 26, 2013, 5:09 a.m., Nitin Mehta wrote: > > Ship It! Pushed with commit 10236bee26f79391838c103972d0afa551e5efd4 - Nitin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.o

Re: Nominating Committers

2013-02-25 Thread Nitin Mehta
Can the community guy also self nominate him/her self as well if he/she feels they feel they have graduated to being a committer ? Thanks, -Nitin On 22/02/13 9:47 PM, "Joe Brockmeier" wrote: >Hey all, > >The thread earlier today about nominating committers has made me realize >that the communit

Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size.

2013-02-25 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9558/#review17072 --- Ship it! Ship It! - Nitin Mehta On Feb. 25, 2013, 4:15 p.m

Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size.

2013-02-25 Thread Nitin Mehta
une...@citrix.com>>, "mice_...@tcloudcomputing.com<mailto:mice_...@tcloudcomputing.com>" mailto:mice_...@tcloudcomputing.com>>, Min Chen mailto:min.c...@citrix.com>>, Devdeep Singh mailto:devdeep.si...@citrix.com>> Cc: Nitin Mehta mailto:nitin.

Re: Review Request: CLOUDSTACK-1374: Volume table is not getting updated with volume size.

2013-02-25 Thread Nitin Mehta
<https://reviews.apache.org/r/9558/#comment36084> What if the size is already set ? why do we need to set it again ? - Nitin Mehta On Feb. 25, 2013, 12:31 p.m., Sanjay Tripathi wrote: > > --- > This is an automatically g

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-25 Thread Nitin Mehta
Code committed with 4d573ddd1bcd4ab27edfb91791a830308236521b From: Deepti Dohare mailto:deepti.doh...@citrix.com>> Reply-To: Deepti Dohare mailto:deepti.doh...@citrix.com>> To: Min Chen mailto:min.c...@citrix.com>> Cc: Nitin Mehta mailto:nitin.me...@citrix.com>&g

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-24 Thread Nitin Mehta
orage/download/DownloadMonitorImpl.java: patch does not apply error: patch failed: server/src/com/cloud/template/TemplateAdapterBase.java:76 error: server/src/com/cloud/template/TemplateAdapterBase.java: patch does not apply From: Nitin Mehta mailto:nitin.me...@citrix.com>> Reply-To:

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-24 Thread Nitin Mehta
. - Nitin Mehta On Feb. 21, 2013, 10:01 a.m., deepti dohare wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apach

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-21 Thread Nitin Mehta
please confirm me that this iso wont get deleted during template sync which gets triggered during stop start ssvm or MS restart ? Also can you please put in all the test cases u used for my benefit ? - Nitin Mehta On Feb. 19, 2013, 5:21 p.m., deepti dohare wrote

Re: Commits with tabs & trailing white space

2013-02-20 Thread Nitin Mehta
Mice - +1 to your suggestion. Things like these should be enforced programmatically rather than relying on good intentions. On 21/02/13 11:11 AM, "Mice Xia" wrote: >Sateesh, > >I just run some scripts: > >find . -name *.java |wc -l >4220 > >find . -name *.java | xargs egrep -c "^[[:space:]]+$"

Re: Regarding cpu and ram overcommit.

2013-02-20 Thread Nitin Mehta
Sateesh - Please find my answers inline On 20/02/13 10:28 PM, "Sateesh Chodapuneedi" wrote: >> -Original Message- >> From: Bharat Kumar [mailto:bharat.ku...@citrix.com] >> Sent: 20 February 2013 18:18 >> To: cloudstack-dev@incubator.apache.org >> Subject: Regarding cpu and ram overcommit

Re: Review Request: CLOUDSTACK-356 Snapshot errors with multiple secondary storage in one zone.

2013-02-20 Thread Nitin Mehta
e.org<mailto:cloudstack-dev@incubator.apache.org>" mailto:cloudstack-dev@incubator.apache.org>>, Deepti Dohare mailto:deepti.doh...@citrix.com>>, Nitin Mehta mailto:nitin.me...@citrix.com>> Subject: Re: Review Request: CLOUDSTACK-356 Snapshot errors with multiple seco

[jira] [Resolved] (CLOUDSTACK-1252) Failed to download default template in VMware

2013-02-20 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta resolved CLOUDSTACK-1252. - Resolution: Fixed Root cause is this issue - https://issues.apache.org/jira/browse

Re: Call for Agenda Items/IRC Weekly Meeting Reminder

2013-02-20 Thread Nitin Mehta
We will never be able to hit a time convenient for all 3 TZ. Lets have 2 meetings and share notes or rotate the timings ? On 20/02/13 4:55 PM, "Abhinandan Prateek" wrote: >Yes, if it can be moved to some earlier timing or our early morning that >would be great ! > >On 20/02/13 4:10 PM, "Rohit Ya

Re: [Discuss] CloudStack Collab Summit 2013

2013-02-20 Thread Nitin Mehta
+1 to Rohit's comment. I would recommend having the Europe collab in May or sooner than Nov. as well. Folks here would be eager to participate and contribute in it. Thanks, -Nitin On 20/02/13 1:33 PM, "Rohit Yadav" wrote: >On Wed, Feb 20, 2013 at 2:50 AM, Mark Hinkle >wrote: >> Given that we a

Re: randomlyIncreasingVMInstanceDaoImpl used instead of UserVmDaoImpl

2013-02-19 Thread Nitin Mehta
Would it break something ? And are the other resources like volumes, snapshots following the same logic as well ? Thanks, -Nitin On 20/02/13 11:45 AM, "Kishan Kavala" wrote: >Kelven, >randomlyIncreasingVMInstanceDaoImpl is being used instead of >UserVmDaoImpl in ComponentContext.xml. Was this

Re: [ACS4.2] test plan review: Dynamic Scaling up of CPU and RAM

2013-02-18 Thread Nitin Mehta
> >Details >1-Test plan >https://cwiki.apache.org/confluence/display/CLOUDSTACK/Scaling+up+CPU+and+ >RAM+for+running+VMs > >2-FS >https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dynamic+scaling+of+ >CPU+and+RAM > > > >Thanks >Prashant Kumar Mishra > >

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-18 Thread Nitin Mehta
ache.org>>, Deepti Dohare mailto:deepti.doh...@citrix.com>>, Nitin Mehta mailto:nitin.me...@citrix.com>> Subject: Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM This

Re: android cloudstack client

2013-02-16 Thread Nitin Mehta
As a cloud admin, I would love to keep a tab on my cloud so ideally would want to see the dashboard, general health and alerts on my cloud. It will be also useful for time taking functionalities like taking snapshot, creating a vm etc. so would like to be notified when these operations are done. So

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-15 Thread Nitin Mehta
: Deepti Dohare mailto:deepti.doh...@citrix.com>> Reply-To: Deepti Dohare mailto:deepti.doh...@citrix.com>> To: "cloudstack-dev@incubator.apache.org<mailto:cloudstack-dev@incubator.apache.org>" mailto:cloudstack-dev@incubator.apache.org>>, Nitin Mehta mailt

Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM

2013-02-15 Thread Nitin Mehta
, which is not desirable. I would suggest you to do the following. When you delete iso, mark it removed in DB so that its not available for further use. In the storage cleanup thread check whether the iso is detached for the vms and if it is then delete it from the sec. storage. - Nitin Mehta

Re: Review Request: CLOUDSTACK-356 Snapshot errors with multiple secondary storage in one zone.

2013-02-14 Thread Nitin Mehta
unique for the cloud. #3 I am not completely convinced that this fix solves the problem. I know there is a bug of multiple sec. storage not having paths on the host and should be fixed but your example doesn't seem right to me. - Nitin Mehta On Feb. 15, 2013, 6:31 a.m., deepti dohare

Re: [BLOCKER] SystemVMs come up but don't have agent running

2013-02-14 Thread Nitin Mehta
I am facing this issue as well. On 14/02/13 4:53 PM, "Rohit Yadav" wrote: >I mentioned in a recent thread about build failure about this blocker, >and wanted to confirm. > >Looks like this is an issue and not just for me: >A fresh deployment of basic zone would start systemvms and they are >show

Re: [PROPOSAL] DatabaseCreator: the db deployer, dbv upgrader and db cleaner

2013-02-14 Thread Nitin Mehta
Thanks Rohit. Had few questions for you. Please find them inline. On 14/02/13 2:58 PM, "Rohit Yadav" wrote: >Had discussions with Alex and Abhi on DatabaseCreator. I was earlier >not sure why we need it or how sysadmins would use it. >We had a hangout on G+: https://www.youtube.com/watch?v=Xp2AG

Re: [DISCUSS] Google Summer of Code?

2013-02-14 Thread Nitin Mehta
+1. I would love to participate as well. On 14/02/13 9:21 AM, "Rohit Yadav" wrote: >On Thu, Feb 14, 2013 at 1:45 AM, Joe Brockmeier wrote: >> Hey all, >> >> Google has announced its dates [1] for Summer of Code. We've touched on >> this previously [2] and there was some concern it wasn't a gre

Re: New committer: Likitha Shetty

2013-02-13 Thread Nitin Mehta
Congrats Likitha !!! Well deserved On 13/02/13 8:07 PM, "David Nalley" wrote: >Hi folks: > >The Podling Project Management Committee (PPMC) for Apache CloudStack >has asked Likitha Shetty to become a committer and we are pleased to >announce that they have accepted. > >Being a committer enables

Re: [DISCUSS] Correctly fix cloud- to cloudstack- rename refactoring

2013-02-13 Thread Nitin Mehta
I am would say lets fix real bugs than making these fixes. This can potentially break migration paths and the value add from this change is little except consistency in naming. On 13/02/13 12:05 PM, "Rohit Yadav" wrote: >Pl. rename any script references which have renamed from cloud- to >cloudst

Re: [ACS4.2] test plan review: Dynamic Scaling up of CPU and RAM

2013-02-12 Thread Nitin Mehta
Please categorize the test plans in different buckets. When presented as a linear list, its difficult to understand what you have covered and what you missed. On 13/02/13 10:50 AM, "Abhinandan Prateek" wrote: >Nitin, > > Please review the following test plan for your 4.2 feature and sign off.

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-02-12 Thread Nitin Mehta
Apologize for the delayed response. Was involved in other issues. Please find answers inline. > >-Original Message- >From: Prashant Kumar Mishra [mailto:prashantkumar.mis...@citrix.com] >Sent: Thursday, January 24, 2013 12:26 PM >To: cloudstack-dev@incubator.apache.org &

Re: Regarding cpu and ram overcommit feature

2013-02-12 Thread Nitin Mehta
What is the down side if we do not update the overcommit ratios on existing vms ? I don¹t see these ratios being changed quite often in the cloud. On 12/02/13 2:57 PM, "Bharat Kumar" wrote: >Hi all, > >As a part of this feature we need to implement a method to update the >overcommit ratios. > >w

[jira] [Updated] (CLOUDSTACK-658) Scaling up CPU and RAM for running VMs

2013-02-12 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-658?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-658: --- Fix Version/s: (was: 4.1.0) 4.2.0 > Scaling up CPU and

Re: Xenserver Host unable to reconnect

2013-02-11 Thread Nitin Mehta
Would it not interfere when you are adding it back ? Do you not have to set the removed column in the host table ? On 12/02/13 12:00 PM, "Devdeep Singh" wrote: >Hi Caleb, > >Do you have any instances running on the host? If not, then removing it >shouldn't cause any problems. Can you also check

Re: 100 brownie points :P

2013-02-11 Thread Nitin Mehta
I think the reason is that the exception class where these are used implement the Serializable interface. The purpose of the serialization version UID is to keep track of different versions of a class in order to perform valid serialization of objects. The idea is to generate an ID that is unique

Re: Review Request: BUG-ID: CLOUDSTACK-1206

2013-02-08 Thread Nitin Mehta
Pradeep - It would be great if you could also summarize what the issue was and what was your fix Thanks, -Nitin On 08/02/13 2:20 PM, "Pradeep Soundararajan" wrote: > >--- >This is an automatically generated e-mail. To reply, visit: >https:

[jira] [Updated] (CLOUDSTACK-1185) Scaling up vm and cluster capacity runs out, dont fail but live migrate.

2013-02-07 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-1185: Assignee: Nitin Mehta Description: With live migration coming in I think we

[jira] [Created] (CLOUDSTACK-1185) Scaling up vm and cluster capacity runs out, dont fail but live migrate.

2013-02-07 Thread Nitin Mehta (JIRA)
Nitin Mehta created CLOUDSTACK-1185: --- Summary: Scaling up vm and cluster capacity runs out, dont fail but live migrate. Key: CLOUDSTACK-1185 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-1185

Re: Review Request: CLOUDSTACK-667: VM's base image updation facility

2013-02-06 Thread Nitin Mehta
please check ? - Nitin Mehta On Feb. 1, 2013, 2:20 p.m., Harikrishna Patnala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apach

Re: Review Request: CLOUDSTACK-924: source_template_id set to NULL when a template is created from a snapshot

2013-02-06 Thread Nitin Mehta
> On Jan. 9, 2013, 8:43 a.m., Nitin Mehta wrote: > > You need to take care of migration as well. Any thoughts on that ? > > Saksham Srivastava wrote: > The fix is only for the new volumes which are created from snapshots > which were themselves created from volumes

Re: docs.cloudstack.org

2013-02-06 Thread Nitin Mehta
That¹s a bad design. Clicking to show children every time. I am looking for High availability doc that used to be there. On 06/02/13 8:43 PM, "Abhinandan Prateek" wrote: >Probably you are looking for this >https://cwiki.apache.org/CLOUDSTACK/design.html >You need to click on "Show Children" on

[jira] [Updated] (CLOUDSTACK-714) S3-based secondary Storage - eliminating a dependence on NFS as a buffer for object storage

2013-02-05 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-714?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-714: --- Fix Version/s: (was: 4.1.0) 4.2.0 > S3-based second

Re: master broken?

2013-02-01 Thread Nitin Mehta
Fixed this. Sorry missed a file. On 01/02/13 4:06 PM, "Prasanna Santhanam" wrote: >https://builds.apache.org/view/A-F/view/Cloudstack/job/cloudstack-master-m >aven/591/ > >missed files in checkin? 777147ce8a47238125a5439f207c225aa9db5304 >-- >Prasanna.,

Re: Review Request: Multiple snapshots on a volume failing on XenServer 6.1.

2013-02-01 Thread Nitin Mehta
Submitted with commit e162876605a31a4687ac77c5554e0a15ea7dda55 Author: Devdeep Singh Date: Fri Feb 1 15:25:26 2013 +0530 From: Nitin Mehta mailto:nitin.me...@citrix.com>> Reply-To: Nitin Mehta mailto:nitin.me...@citrix.com>> To: "cloudstack-dev@incubator.apache.org<m

Re: Review Request: CLOUDSTACK-606: Starting VM fails with 'ConcurrentOperationException' in a clustered MS scenario

2013-02-01 Thread Nitin Mehta
Submitted with commit 777147ce8a47238125a5439f207c225aa9db5304 Author: Koushik Das Date: Fri Feb 1 15:34:41 2013 +0530 On 01/02/13 2:55 PM, "Koushik Das" wrote: > >--- >This is an automatically generated e-mail. To reply, visit: >http

Re: Review Request: Multiple snapshots on a volume failing on XenServer 6.1.

2013-02-01 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8623/#review15999 --- Ship it! Ship It! - Nitin Mehta On Dec. 16, 2012, 10:46 a.m

Re: Review Request: Cloudstack-711 Cpu and ram overcommit at cluster level.

2013-01-31 Thread Nitin Mehta
re that the logging is consistent...please...you would want to show the over provisioned cpu and ram. server/src/com/cloud/resource/ResourceManagerImpl.java <https://reviews.apache.org/r/9156/#comment34292> What does this mean ? Is this logic broken for other hypervisors ? - Nitin M

Re: new committer: Koushik Das

2013-01-31 Thread Nitin Mehta
CongratsŠnow commit the code you reviewed :) On 01/02/13 8:02 AM, "Abhinandan Prateek" wrote: >Congrats Da ! It was long due. > > >On 01-Feb-2013, at 4:13 AM, "Alex Huang" wrote: > >> The Podling Project Management Committee (PPMC) for Apache CloudStack >> has asked Koushik Das to become a comm

Re: Review Request: CLOUDSTACK-667: VM's base image updation facility

2013-01-31 Thread Nitin Mehta
..wht do you think ? server/src/com/cloud/vm/UserVmManagerImpl.java <https://reviews.apache.org/r/9158/#comment34233> correct tabs here n across the code as well - Nitin Mehta On Jan. 31, 2013, 4:17 p.m., Harikrishna Patnala wrote: > > --

Re: Review Request: Cloudstack-711 Cpu and ram overcommit at cluster level.

2013-01-31 Thread Nitin Mehta
ewhr server/src/com/cloud/capacity/CapacityManagerImpl.java <https://reviews.apache.org/r/9156/#comment34220> add ram over commit ratio as well server/src/com/cloud/vm/VirtualMachineManagerImpl.java <https://reviews.apache.org/r/9156/#comment34221>

Re: Review Request: CLOUDSTACK-745: Reset a VM on reboot

2013-01-30 Thread Nitin Mehta
omment34136> You would need the same change for stop start vm as well... server/src/com/cloud/vm/UserVmManagerImpl.java <https://reviews.apache.org/r/9157/#comment34134> improve the log msg - Nitin Mehta On Jan. 30, 2013, 11:32 a

Re: Review Request: CLOUDSTACK-667: VM's base image updation facility

2013-01-30 Thread Nitin Mehta
com/cloud/vm/UserVmManagerImpl.java <https://reviews.apache.org/r/9158/#comment34128> dont you have to do access check whether this user is allowed to access this template ? - Nitin Mehta On Jan. 30, 2013, 11:32 a.m., Harikrishna Patnala wrote: > >

Re: API Problem with listAsyncJobs

2013-01-30 Thread Nitin Mehta
Is the job work done – can you please check the logs ? Some of the jobs can take long time to execute – say creating a 100gb snapshot. From: Antonio Petrocelli mailto:a.petroce...@netsons.com>> Reply-To: "cloudstack-dev@incubator.apache.org" mailto:

Re: [DISCUSS] DeployVirtualMachine userdata enhancements

2013-01-29 Thread Nitin Mehta
eb service in the future. > >And by the way, does 'create resource tag(s) API' have the same >requirement? Max length of tag value is 255 now. > >Regards >Mice > >-Original Message- >From: Nitin Mehta [mailto:nitin.me...@citrix.com] >Sent: Wed

Re: [DISCUSS] DeployVirtualMachine userdata enhancements

2013-01-29 Thread Nitin Mehta
Good point Rohit, but I suggest making only those API's POST which create a resource (like deployVm here) and not all of them. Ideally, we need to be using the right http methods for a true restful web service. Also about the length part, how then we allow user data to be of size 2kb currently ?

Re: Review Request: CLOUDSTACK-713 Limit Resources to domain/accounts

2013-01-28 Thread Nitin Mehta
https://reviews.apache.org/r/9110/#comment33903> I see this boilerplate code everywhere. Please make this a single line function call - Nitin Mehta On Jan. 25, 2013, 1:52 p.m., Sanjay Tripathi wrote: > > --- > This is an au

[jira] [Updated] (CLOUDSTACK-1073) Scaling up CPU and RAM for running VMs - Xenserver

2013-01-28 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-1073: Fix Version/s: (was: 4.0.1) 4.2.0 > Scaling up CPU and

[jira] [Created] (CLOUDSTACK-1074) Scaling up CPU and RAM for running VMs - KVM

2013-01-28 Thread Nitin Mehta (JIRA)
Nitin Mehta created CLOUDSTACK-1074: --- Summary: Scaling up CPU and RAM for running VMs - KVM Key: CLOUDSTACK-1074 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-1074 Project: CloudStack

[jira] [Updated] (CLOUDSTACK-1074) Scaling up CPU and RAM for running VMs - KVM

2013-01-28 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-1074: Fix Version/s: (was: 4.1.0) 4.2.0 > Scaling up CPU and

[jira] [Created] (CLOUDSTACK-1073) Scaling up CPU and RAM for running VMs - Xenserver

2013-01-28 Thread Nitin Mehta (JIRA)
Nitin Mehta created CLOUDSTACK-1073: --- Summary: Scaling up CPU and RAM for running VMs - Xenserver Key: CLOUDSTACK-1073 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-1073 Project: CloudStack

[jira] [Commented] (CLOUDSTACK-1047) tracking in logs using job id

2013-01-25 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13562614#comment-13562614 ] Nitin Mehta commented on CLOUDSTACK-1047: - Alex Huang's idea -

Re: New committer: John Burwell

2013-01-24 Thread Nitin Mehta
Congrats John and thanks for your S3 work :) On 24/01/13 6:54 PM, "David Nalley" wrote: >The Podling Project Management Committee (PPMC) for Apache CloudStack >has asked John Burwell to become a committer and we are pleased to >announce that they have accepted. > >Being a committer enables easie

[jira] [Updated] (CLOUDSTACK-1047) tracking in logs using job id

2013-01-24 Thread Nitin Mehta (JIRA)
[ https://issues.apache.org/jira/browse/CLOUDSTACK-1047?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Nitin Mehta updated CLOUDSTACK-1047: Summary: tracking in logs using job id (was: jobid tracking in logs) > track

Re: jobid tracking in logs

2013-01-23 Thread Nitin Mehta
and extends the logs but I'm okay >both ways. > >--Alex > >> -Original Message- >> From: Nitin Mehta [mailto:nitin.me...@citrix.com] >> Sent: Tuesday, January 22, 2013 10:59 PM >> To: cloudstack-dev@incubator.apache.org >> Cc: Kelven Yang;

  1   2   3   >