Upload Volume checked in

2012-05-01 Thread Nitin Mehta
I recently checked in the upload volume feature into the 3.0.x branch. It leverages the code path we built for template so in case you hit any issues please do let me know. More info about the feature @ http://wiki.cloudstack.org/display/RelOps/Upload+Volume+Functional+Specification Thanks,

RE: Secondary nfs server alert issue

2012-05-07 Thread Nitin Mehta
+Anthony I think we should not put the secondary storage in the host table since we do not create an agent for it and do not send command to it. Since we do not create an agent for it which connects with MS, the state remains in Alert and means nothing. I think the introduction of multiple

RE: Secondary nfs server alert issue

2012-05-07 Thread Nitin Mehta
it out and troubleshoot for the template functionality to work. We should resolve these issues since I have seen the community being most impacted by these during their first install and not knowing how what's wrong here. -Original Message- From: Nitin Mehta [mailto:nitin.me

RE: API usage records and SQL

2012-05-31 Thread Nitin Mehta
Hi Tamas, Asking for 500 records takes 5-6 seconds and 1000 records taking 12 seconds shows a linear increase and so doesn't seem like there is any performance degradation of any sort here. From a quick glance at the code it seems that your analysis It seems there is a conversion/parsing

RE: [cloudstack-devel] Hyper-V Support

2012-06-01 Thread Nitin Mehta
Hi Alessandro, SSVM needs to have agent running which connects to MS on port 8250 (it is a remote agent). The responsibility of this agent is to take commands from MS and take some action. One example of this is - MS instructing the SSVM to download a template on secondary storage and the agent

RE: vmtemplate.status values?

2012-06-01 Thread Nitin Mehta
Hi Adrian, Please find below explanation of the vmtemplate status. It would be great if we could add this explanation against the enums as comments as well. UNKNOWN - status of download is not known. Example - When the job for downloading doesn't exist during progress check. ABANDONED - the

RE: [PATCH] CS-14345 Logout API is not returning XML header Description: The logout API is not returning the standard XML header and the CloudStack version.

2012-06-04 Thread Nitin Mehta
Code change looks good except the initialization of management service which should be done inside ApiServlet(). Also if it is a static variable then append 's' before the name as per naming conventions rule 7 @

RE: how to run it on a eclipse?

2012-06-07 Thread Nitin Mehta
I think we need to keep adding more documentation to http://wiki.cloudstack.org/display/dev/Home to find out all the documentation for devs to get their hands dirty on CS. This is more detailed and should help you get going for windows (for Linux the steps are the same)

RE: Secondary storage is not working

2012-06-11 Thread Nitin Mehta
Lu, Can you paste the snippet of logs where you see the error? Can you please elaborate on what you mean by secondary storage setup is not working. In my knowledge, MS doesn't validate the secondary storage path and does very basic tests while adding it. Typically I have seen the following

RE: Secondary storage is not working

2012-06-11 Thread Nitin Mehta
Lu - Have you seeded your secondary storage with system vm templates ? That seems to be the issue here. -Original Message- From: Lu Heng [mailto:h...@anytimechinese.com] Sent: Tuesday, June 12, 2012 8:43 AM To: cloudstack-dev@incubator.apache.org Subject: Re: Secondary storage is not

RE: Secondary storage is not working

2012-06-11 Thread Nitin Mehta
if you can find something there. Thanks, -Nitin -Original Message- From: Lu Heng [mailto:h...@anytimechinese.com] Sent: Tuesday, June 12, 2012 9:19 AM To: cloudstack-dev@incubator.apache.org Cc: Nitin Mehta Subject: Re: Secondary storage is not working Hi some more information: Server IP

RE: Secondary storage is not working

2012-06-12 Thread Nitin Mehta
secondary to primary storage while creating system vms. Thanks, -Nitin From: Lu Heng [mailto:h...@anytimechinese.com] Sent: Tuesday, June 12, 2012 3:27 PM To: Nitin Mehta Cc: cloudstack-dev@incubator.apache.org Subject: Re: Secondary storage is not working Hi On Tue, Jun 12, 2012 at 6:02 AM

RE: CloudStack bugs, CS-14896

2012-06-12 Thread Nitin Mehta
to subscribe to the Distribution lists below. Thanks, -Nitin From: Olga Smola [mailto:olya.sm...@gmail.com] Sent: Tuesday, June 12, 2012 4:09 PM To: Nitin Mehta Subject: Re: CloudStack bugs, CS-14896 Hi Nitin, could you please help me to ask somebody to review the tickets I've already completed

RE: Fix for the issue CS-15143?

2012-06-27 Thread Nitin Mehta
Yes - you need to update the jars on SSVM. If you stop and start the SSVM then it should get updated for you assuming that the MS has pushed the latest jars on the host. -Original Message- From: to...@midokura.jp [mailto:to...@midokura.jp] On Behalf Of Tomoe Sugihara Sent: Tuesday, June

Re: Review Request: listSnapshotPolicies command, volumeId parameter made optional.

2012-06-28 Thread Nitin Mehta
of BaseListProjectAndAccountResourcesCmd. - Nitin Mehta On June 27, 2012, 6:47 a.m., Likitha Shetty wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5604

Re: Review Request: Fix for CS-15300 Limits on domain admin accounts

2012-06-30 Thread Nitin Mehta
https://reviews.apache.org/r/5573/#comment18543 Why is this required ? I dont think this is correct - Nitin Mehta On June 29, 2012, 7:04 p.m., deepti dohare wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request: Fix for CS-15300 Limits on domain admin accounts

2012-07-02 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5573/#review8798 --- Ship it! - Nitin Mehta On July 2, 2012, 2:58 p.m., deepti dohare

Re: Review Request: CS-15429: Create Instance should fail if creating instance results in exceeding volume resource limit for domain-admin or user accounts.

2012-07-06 Thread Nitin Mehta
://reviews.apache.org/r/5766/#comment18976 Please check this - In case the vm is booted from an iso then as well the disk offering is not null and refers to the the root disk offering and not to the data disk offering. - Nitin Mehta On July 5, 2012, 6:22 a.m., deepti dohare wrote

RE: Review Request: CS-15429: Create Instance should fail if creating instance results in exceeding volume resource limit for domain-admin or user accounts.

2012-07-06 Thread Nitin Mehta
, account limit for this data volume was not checked against. Adding a check for that Thanks, -Nitin -Original Message- From: Nitin Mehta [mailto:nore...@reviews.apache.org] On Behalf Of Nitin Mehta Sent: Saturday, July 07, 2012 1:23 AM To: cloudstack; Deepti Dohare; Nitin Mehta Subject: Re

Re: Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.

2012-07-06 Thread Nitin Mehta
/snapshot/SnapshotManagerImpl.java https://reviews.apache.org/r/5806/#comment18982 Please use a finally() to do the decrement - Nitin Mehta On July 6, 2012, 12:43 p.m., deepti dohare wrote: --- This is an automatically generated e-mail

RE: signing off patches

2012-07-06 Thread Nitin Mehta
Yes, I had the same issue as well and sadly review board was not helpful for committing the code. I had to do all the hacks like changing the author info, putting in signed off info manually. That is not the way to go. Either review board should have good integration with submitting the git

Re: Review Request: listSnapshotPolicies command, volumeId parameter made optional.

2012-07-06 Thread Nitin Mehta
On June 28, 2012, 12:21 p.m., Nitin Mehta wrote: Ideally listSnapshotPolicy should be part of BaseListProjectAndAccountResourcesCmd. Prachi Damle wrote: listSnapshotPolicy cannot extend BaseListProjectAndAccountResourcesCmd, since the snapshotPolicy entity does not carry

Re: Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.

2012-07-09 Thread Nitin Mehta
On July 6, 2012, 9:52 p.m., Nitin Mehta wrote: server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java, line 375 https://reviews.apache.org/r/5806/diff/1/?file=119898#file119898line375 Please use a finally() to do the decrement Devdeep Singh wrote: The intention here

Re: Review Request: CS-15429: Create Instance should fail if creating instance results in exceeding volume resource limit for domain-admin or user accounts.

2012-07-09 Thread Nitin Mehta
https://reviews.apache.org/r/5766/#comment19067 Don't want to sound overcritical but it would always be nice to add a comment if the logic seems to be unintuitive like here. Always makes life easier to maintain code :). - Nitin Mehta On July 9, 2012, 1:14 p.m., deepti dohare wrote

Re: Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.

2012-07-10 Thread Nitin Mehta
inquire how we solve this problem in other places in CS ? I cant approve this change since we dont want to reinvent the wheel if this has already been solved. If it hasn't been solved then I would want to raise this in the mailing list and hear out a good way of doing this. - Nitin Mehta

Re: Review Request: Fix for CS-15279

2012-07-10 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5719/#review9022 --- Ship it! Ship It! - Nitin Mehta On July 3, 2012, 3:46 p.m

Re: Review Request: Fix for CS-15279

2012-07-10 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5719/#review9023 --- Ship it! Ship It! - Nitin Mehta On July 3, 2012, 3:46 p.m

Re: Review Request: CS-15560 : Improve HVM logging of hosts

2012-07-13 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5927/#review9131 --- Ship it! Ship It! - Nitin Mehta On July 13, 2012, 6:46 a.m

RE: Local storage support for data volumes

2012-07-31 Thread Nitin Mehta
Can we not provide the flexibility while adding a host. So when the admin adds a host he has the provision of enabling/disabling local storage for that host. I think that is the lowest level of granularity that we can provide to the admin ? Another question - With this feature would upload

Re: Review Request: Fix for CS-15631

2012-07-31 Thread Nitin Mehta
/create-schema.sql https://reviews.apache.org/r/6229/#comment20541 Did you file a bug with XS that 13 is a limit since the users of CS might get confused why this being the limit instead of 14? - Nitin Mehta On July 31, 2012, 9:09 a.m., Koushik Das wrote

RE: Review Request: CS-15395 updateResourceLimit does not give any warning/error if limit is set lower than the current resources used by an account

2012-08-01 Thread Nitin Mehta
I think we shouldn’t make this change. Since resources can always be destroyed I would want to give the admin flexibility to set the resource limit to a value lower than the current count of resources an account has. At best you can inform admin that he is trying to set the resource limit to a

RE: Review Request: CS-15395 updateResourceLimit does not give any warning/error if limit is set lower than the current resources used by an account

2012-08-02 Thread Nitin Mehta
of resources but you shouldn’t disallow him doing it -Original Message- From: Deepti Dohare Sent: Wednesday, August 01, 2012 5:26 PM To: Nitin Mehta; cloudstack-dev@incubator.apache.org; Prasanna Santhanam Subject: RE: Review Request: CS-15395 updateResourceLimit does not give any warning

RE: PrepareForMigrationCommand subtype of StartCommand

2012-08-02 Thread Nitin Mehta
Prasanna - What you raise is a very interesting point. But this would be an issue only in some of the ServerResources where implementation is like below code. Here there is no exact match of the class and hence it can get into the implementation of its parent unless the order is maintained. So

Re: Review Request: CS-15780 Global configuration 'account.cleanup.interval' is used by event purge thread

2012-08-02 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6191/#review9758 --- Ship it! Ship It! - Nitin Mehta On Aug. 2, 2012, 2:01 p.m., mice

RE: Local storage support for data volumes

2012-08-02 Thread Nitin Mehta
[mailto:da...@gnsa.us] Sent: Thursday, August 02, 2012 5:22 PM To: cloudstack-dev@incubator.apache.org Subject: Re: Local storage support for data volumes On Tue, Jul 31, 2012 at 5:34 AM, Nitin Mehta nitin.me...@citrix.com wrote: Can we not provide the flexibility while adding a host. So when

Re: Review Request: CS-15823 [VMware] failed snapshot entry should be removed automatically when multiple snopshots are taken on one volume simultaneously

2012-08-02 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6325/#review9806 --- Ship it! Ship It! - Nitin Mehta On Aug. 3, 2012, 12:03 a.m., mice

RE: Review Request: CS-15823 [VMware] failed snapshot entry should be removed automatically when multiple snopshots are taken on one volume simultaneously

2012-08-02 Thread Nitin Mehta
Good point. But this is not failure per se in the snapshot creation but more of a limitation of these hypervisors and probably we want to keep that subtle difference by not putting it in Error but deleting the row altogether and surfacing the limitation through the exception to the end user.

Re: Review Request: CS-15833: PreparForMigrationCommand should extend base Command object

2012-08-03 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6338/#review9824 --- Ship it! Ship It! - Nitin Mehta On Aug. 3, 2012, 10:46 a.m

Re: Review Request: deployVirtualMachine api, added a check to see if the userdata provided is base64 encoded.

2012-08-07 Thread Nitin Mehta
/org/apache/commons/codec/binary/Base64.html#decodeBase64%28byte[]%29 - Nitin Mehta On Aug. 7, 2012, 9:37 a.m., Likitha Shetty wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6428

Re: Review Request: Fix for CS-15621

2012-08-08 Thread Nitin Mehta
, 10:09 a.m.) Review request for cloudstack, Prachi Damle and Nitin Mehta. Description --- Using migrateVolumes method which does not perform input validation. Some input validation in the migrateVolume method prevented migration of volume in READY state. Also using volume disk

RE: how to enable systemVM debug options

2012-08-09 Thread Nitin Mehta
System vms have a java agent running and all you need is to enable debug port for the agent's java process. Add this -Xrunjdwp:transport=dt_socket,address=8998,server=y And then you can use eclipse on your machine to connect to this port and private IP of system vms I googled all the settings

Re: Review Request: Fix for CS-15621

2012-08-12 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6337/#review10177 --- Ship it! Ship It! - Nitin Mehta On Aug. 8, 2012, 11:09 a.m

RE: Local storage support for data volumes

2012-08-12 Thread Nitin Mehta
: Sunday, August 12, 2012 6:38 PM To: cloudstack-dev@incubator.apache.org Subject: RE: Local storage support for data volumes Its already there. http://bugs.cloudstack.org/browse/CS-14277 -Original Message- From: Nitin Mehta [mailto:nitin.me...@citrix.com] Sent: Sunday, August 12, 2012 5:08

Re: Review Request: Support for local data disk feature. (CS-14277)

2012-08-12 Thread Nitin Mehta
by someone else as I dont have any expertise here - Nitin Mehta On Aug. 9, 2012, 5:45 p.m., Koushik Das wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6431

Re: Review Request: CS-15823 Failed snapshot should be marked as Error and cleanup asynchronously

2012-08-16 Thread Nitin Mehta
in CS as well or do we want to give snapshots a different treatment here ? I am somehow not very comfortable doing hard delete here. - Nitin Mehta On Aug. 11, 2012, 1:13 a.m., mice xia wrote: --- This is an automatically generated e-mail

RE: Two type of storage on CS

2012-08-22 Thread Nitin Mehta
Yes, you can enable use.local.storage global config param and you can then use local storage for creating user vms. While creating a vm you can choose the service offering for local storage or for shared storage and accordingly vm's root disk will reside on either of them. For data disks

Re: Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.

2012-08-22 Thread Nitin Mehta
failed here so include its uuid, name etc server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java https://reviews.apache.org/r/5806/#comment22757 It would be helpful to say which snapshot failed here so include its uuid etc - Nitin Mehta On Aug. 16, 2012, 10:46 a.m., deepti

RE: Failed to install: x != java.lang.String when register ISO in CS 3.0.x

2012-08-23 Thread Nitin Mehta
I was looking at the code and the way we are using checksum doesn't make sense to me. We are computing the checksum through the Java code and then passing the same checksum in the bash script post download of template to check whether the checksum matches or not. That's not the correct. I

Re: Review Request: Fix Bug CS-10284 can not change OS preference from OS to none in 4.0

2012-09-03 Thread Nitin Mehta
://reviews.apache.org/r/6806/#comment23574 Would have loved a comment saying that None means no os preference. - Nitin Mehta On Aug. 29, 2012, 1 a.m., mice xia wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request: CS-10219 Add option to set login attempts to portal.

2012-09-03 Thread Nitin Mehta
https://reviews.apache.org/r/6858/#comment23582 There shouldnt be - 1 here. server/src/com/cloud/user/AccountManagerImpl.java https://reviews.apache.org/r/6858/#comment23584 Please say - Account is disabled. - Nitin Mehta On Sept. 3, 2012, 2 p.m., saksham srivastava wrote

Re: Review Request: CS-10219 Add option to set login attempts to portal.

2012-09-07 Thread Nitin Mehta
The string here should be same as for the fresh install Incorrect login attempts allowed before the account is disabled - Nitin Mehta On Sept. 7, 2012, 10:07 a.m., saksham srivastava wrote: --- This is an automatically generated

Re: Review Request: CS-10219 Add option to set login attempts to portal.

2012-09-10 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6858/#review11236 --- Ship it! Ship It! - Nitin Mehta On Sept. 7, 2012, 11:46 a.m

Re: Review Request: CS-10219 Add option to set login attempts to portal.

2012-09-10 Thread Nitin Mehta
On Sept. 7, 2012, 10:29 a.m., Nitin Mehta wrote: server/src/com/cloud/user/AccountManagerImpl.java, line 1876 https://reviews.apache.org/r/6858/diff/6/?file=150620#file150620line1876 Can you suggest him corrective action ? Like contacting him an admin or resetting his password

Re: Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.

2012-09-10 Thread Nitin Mehta
On Aug. 22, 2012, 9:11 p.m., Nitin Mehta wrote: server/src/com/cloud/resourcelimit/ResourceLimitManagerImpl.java, line 310 https://reviews.apache.org/r/5806/diff/3/?file=141357#file141357line310 By changing the functionality of checkResourceLimit you are risking it for all

RE: Review Request: Support for local data disk feature. (CS-14277)

2012-09-11 Thread Nitin Mehta
Code pushed with commit 65eeeaf071fb79c717226a915944b68114919818 From: Nitin Mehta [mailto:nore...@reviews.apache.org] On Behalf Of Nitin Mehta Sent: Wednesday, September 12, 2012 9:37 AM To: Abhinandan Prateek; Nitin Mehta Cc: cloudstack; Koushik Das Subject: Re: Review Request: Support

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

2012-09-11 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7018/#review11381 --- Ship it! Ship It! - Nitin Mehta On Sept. 11, 2012, 2:41 p.m

Re: Review Request: CLOUDSTACK-121: FIx NullPointerException while logging in

2012-09-18 Thread Nitin Mehta
://reviews.apache.org/r/7130/#comment25173 You are retrieving the user account again and again checking for null. Can we not have 1 db call for this ? - Nitin Mehta On Sept. 18, 2012, 8:29 a.m., Rohit Yadav wrote: --- This is an automatically

RE: How to make Domain Admin having the right adding account?

2012-09-18 Thread Nitin Mehta
Change the bitmap in the file commands.properties.in to 7 to allow domain admin to execute this api (change 3 to 7). createAccount=com.cloud.api.commands.CreateAccountCmd;3 You then might have to go into CreateAccountCmd implementation and check if there is some ACL for restricting domain admin

RE: How to make Domain Admin having the right adding account?

2012-09-18 Thread Nitin Mehta
, createUser, deleteUser and updateUser within its domain as well .I also want to disable the user attach storage and let the domain admin do this. so, is there a document about this?or any suggestion? 2012/9/18 Nitin Mehta nitin.me...@citrix.com Change the bitmap in the file commands.properties.in

Re: Review Request: CLOUDSTACK-121: FIx NullPointerException while logging in

2012-09-18 Thread Nitin Mehta
://reviews.apache.org/r/7130/#comment25181 Trivial but we have already logged auth failure above correct ? - Nitin Mehta On Sept. 18, 2012, 9:43 a.m., Rohit Yadav wrote: --- This is an automatically generated e-mail. To reply, visit: https

RE: How to make Domain Admin having the right adding account?

2012-09-20 Thread Nitin Mehta
-cloudstack-users/201208.mbox/%3ccadwpi+gp7dzr8ypw-gnkdoynjigdw6qk0jxbwpyb9z4b6zs...@mail.gmail.com%3E 2012/9/18 Nitin Mehta nitin.me...@citrix.com I am not sure if there is any documentation around for this but I think you will need to proceed in a similar fashion as I suggested

RE: Review Request: CS-10219 Add option to set login attempts to portal.

2012-09-20 Thread Nitin Mehta
This was committed and tested last week with commit - a56631bc66713faaa99759eb374e11afd6f533d3 Saksham – Please close it From: saksham srivastava [mailto:nore...@reviews.apache.org] On Behalf Of saksham srivastava Sent: Monday, September 10, 2012 3:26 PM To: Alena Prokharchyk; Nitin Mehta

RE: How to add a new API to cloudstack

2012-09-24 Thread Nitin Mehta
For adding table and columns add it to create-schema.sql (and for upgrade put it into schema-302to40.sql ) To add API's you would need to add them to commands.properties file (which holds command name to the JAVA class file reference). You can look into createAccountCmd in commands.properties

RE: How can I specify the vCPU and ram manually?

2012-10-04 Thread Nitin Mehta
As far as implementation goes you can have a look at how to create customizable disk offering for reference and it should be on the same lines. Thinking loud you will have to change deployvm and createservice offering api, allocator logic a bit for pushing customized cpu and ram, capacity

RE: Document describing MySQL tables?

2012-10-09 Thread Nitin Mehta
For templates you can refer to volumes table and for isos refer user_vm table if that’s what you were asking for. Thanks, -Nitin -Original Message- From: Mice Xia [mailto:mice_...@tcloudcomputing.com] Sent: Wednesday, October 10, 2012 10:17 AM To: cloudstack-dev@incubator.apache.org

RE: [VOTE] Apache Cloudstack 4.0.0-incubating Release, first round

2012-10-11 Thread Nitin Mehta
Noah - Agree with you. I think whosoever is voting +1 should explicitly indicate the extent to which they have tested it as well. -Original Message- From: Noah Slater [mailto:nsla...@tumbolia.org] Sent: Thursday, October 11, 2012 2:35 PM To: cloudstack-dev@incubator.apache.org Subject:

Re: UI - Advanced Search Feature

2012-10-11 Thread Nitin Mehta
If you could paste some snapshots in the link that would be great. On 12-Oct-2012, at 12:55 AM, Brian Federle wrote: Hello all, The UI component for the new advanced search feature is now checked into master. You will see an arrow appear in the list view's search text field, which when

Re: New PPMC member: Edison Su

2012-10-11 Thread Nitin Mehta
Edison - You deserve it :) congrats man !! On 11-Oct-2012, at 11:02 PM, Kelven Yang wrote: Congratulations to Edison! Kelven On 10/11/12 7:27 AM, Chip Childers chip.child...@sungard.com wrote: The Podling Project Management Committee (PPMC) for Apache CloudStack has asked Edison to

RE: FS on cloudStack createSnapshot synchronization improvement

2012-10-15 Thread Nitin Mehta
Hi Alena - Why is it that we prefer hosts on which vm is/was running for creating snapshots - do we see performance benefits in any way or it was a convenient way of choosing the host ? For performance benefits can we not choose a host in the cluster which has the least load (in terms of

RE: Template downloads, ACS 4.0 - build 756

2012-10-19 Thread Nitin Mehta
Try step #2 and #3 and see if it helps. https://cwiki.apache.org/CLOUDSTACK/ssvm-troubleshooting.html From: Kelcey Damage (BBITS) [mailto:kel...@bbits.ca] Sent: Friday, October 19, 2012 1:02 AM To: CloudStack dev list Subject: Template downloads, ACS 4.0 - build 756 Hi Running build 756 and I

RE: New PPMC Member: Alex Huang

2012-10-30 Thread Nitin Mehta
Congrats Alex. I assumed you were already one :-) Thanks, -Nitin From: Matthew Morrissey [matthew.morris...@sungard.com] Sent: Tuesday, October 30, 2012 8:19 PM To: cloudstack-dev@incubator.apache.org Cc: Alex Huang Subject: Re: New PPMC Member: Alex Huang

Re: Should we have separate Count API?

2012-11-06 Thread Nitin Mehta
+1 to adding countOnly flag. On 06-Nov-2012, at 7:02 AM, Vijaykumar Natarajan wrote: IMHO, Instead of a count or countOnly API for each resource type, it'd be better to create a stats API which returns counts for all resources (no of vms, no of Ips etc.). This is typically the use case

Re: Should we have separate Count API?

2012-11-06 Thread Nitin Mehta
On 06-Nov-2012, at 3:09 PM, Vijaykumar Natarajan wrote: Isn't listAccounts available to an admin only? Its available for all. Let me know if it doesn't fit your case. - Original Message - From: Nitin Mehta [mailto:nitin.me...@citrix.com] Sent: Tuesday, November 06, 2012 06:08 PM

Re: cloudstack 4 incubating - local storage not working

2012-11-26 Thread Nitin Mehta
Romain - Please find my answer inline. On 26-Nov-2012, at 9:39 PM, rdartigues@orange.com wrote: Good day, i'd like to run my VM on local storage (as i don't need functionalities provided by shared primary storage). Actually: - i've local storage enabled in my zone - i've local

Re: Collab12: developers, committers, PPMC members group photo tomorrow

2012-12-02 Thread Nitin Mehta
Share the pic please :)….along with the names :) On 02-Dec-2012, at 6:53 AM, Rohit Yadav wrote: Let's meet in Room 601/Casanova at 8:30AM for a group photo, jzb will help take the photos. I'm really looking forward to have a group photo with all the developers, committers and PPMC members.

Re: Error : Cannot change RAM memory for console, ssvm, ...

2012-12-02 Thread Nitin Mehta
Yeah seem like one. Can you please file a bug for this. As a workaround can you create a system offering for ssvm and specify the same in the global settings. I think that should work. On 01-Dec-2012, at 10:07 PM, Charles Moulliard wrote: Hi, When I use the Web interface of Cloudstack

Re: How to fix bug with a local version

2012-12-02 Thread Nitin Mehta
Were you not able to find a corresponding bug in Apache Jira ? I see Sheng Yang fixed it. Thanks, -Nitin On 02-Dec-2012, at 8:17 PM, sx chen wrote: Hi, all. For my reason,My local version is 3.0.2,and we had do some development on this version. So how do i fix the bug that the 3.0.3

Re: How to fix bug with a local version

2012-12-02 Thread Nitin Mehta
...@citrix.com Date: Tue May 29 16:45:52 2012 -0700 CS-14421: Clean /etc/network/interfaces when sysytemvm reboot On 03-Dec-2012, at 12:21 AM, Nitin Mehta wrote: Were you not able to find a corresponding bug in Apache Jira ? I see Sheng Yang fixed it. Thanks, -Nitin On 02-Dec

Re: RestoreVM cmd, root volume gets recreated

2012-12-02 Thread Nitin Mehta
As discussed restore vm api is for resetting the vm with a new volume in case its existing volume gets corrupted whereas recover vm is used to recover the vm when its in the destroyed state before it gets expunged. That said currently only recover vm option is available through the UI and the

Re: Unused API Cmd classes, remove bloat?

2012-12-06 Thread Nitin Mehta
Rohit - my answers inline. On 07-Dec-2012, at 1:12 AM, Rohit Yadav wrote: I found following APIs which lack the (apiname, class and roles) mappings in commands.properties: CreatePrivateNetworkCmd DestroyConsoleProxyCmd ListRecurringSnapshotScheduleCmd I don't see all the three above in

Re: Question on ListEventsCmd

2012-12-09 Thread Nitin Mehta
Min - I will take a stab at this. See if this makes sense to you. I haven't used them but I think the intention of the code should be this. If say you want to query events within a time range then you will use these parameter. Say at this point of time you want to get all the events that took

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

2012-12-16 Thread Nitin Mehta
to hard code the path in the file rather than passing it as an argument ? - Nitin Mehta On Dec. 16, 2012, 10:46 a.m., Devdeep Singh wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8623

Re: Review Request: CLOUDSTACK-643: KVM snapshots deleted via API not deleted from physical secondary storage

2012-12-16 Thread Nitin Mehta
marked removed in DB we need to check if its deleted from secondary storage as well. - Nitin Mehta On Dec. 15, 2012, 3:22 a.m., Prasanna Santhanam wrote: --- This is an automatically generated e-mail. To reply, visit: https

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

2012-12-17 Thread Nitin Mehta
On 18-Dec-2012, at 8:54 AM, Mice Xia wrote: Great feature and I believe a lot of customers need it, hope Xenserver will support dynamic memory change one day. [Koushik] Assuming that in the updated compute offering only CPU and RAM has changed, the deployment planner can either select

Re: [DISCUSS] Enabling storage xenmotion on xenserver 6.1

2012-12-19 Thread Nitin Mehta
+1 Would this be available for system vms (ssvm, cpvm and rvm ) as well ? Thanks, -Nitin On 19-Dec-2012, at 2:33 PM, Devdeep Singh wrote: Hi Alex, I was planning to cover the following scenarios: 1. Live migrate a VM with disks from Host A to Host B within the same cluster. This

Re: [DISCUSS]API request throttling

2012-12-20 Thread Nitin Mehta
On 20-Dec-2012, at 2:18 PM, Koushik Das wrote: +1 to the idea. Should there be some API to show how many API calls are remaining for a particular user for the given interval? And should this call get counted as well? Currently in UI if you are using wizard to deploy a VM multiple API

Re: [DISCUSS] Enabling storage xenmotion on xenserver 6.1

2012-12-20 Thread Nitin Mehta
On 20-Dec-2012, at 6:57 PM, Alex Huang wrote: If the hypervisor doesn't support it, shouldn't we just fail the request with a proper message. The user/admin can shutdown the vm and then move/migrate it. The behavior would be similar to how cloudstack currently lets the VM volumes be

Re: Installing the Usage Server

2012-12-22 Thread Nitin Mehta
Hi Sarath, See if this helps http://incubator.apache.org/cloudstack/docs/en-US/Apache_CloudStack/4.0.0-incubating/html/Admin_Guide/work-with-usage.html On 21-Dec-2012, at 5:27 PM, sarath kumar wrote: HI Team, I have One question that i want to install usage server in cloud stack

Re: Merge request: Merging api_refactoring on master

2012-12-22 Thread Nitin Mehta
Hi Rohit, Please find my answers inline. On 23-Dec-2012, at 2:05 AM, Rohit Yadav wrote: Hi David, Thanks for the email, yes I really need that to know what mistakes we are doing, some of them are ack'd (we need tests and habit of contributing code with tests) but for the most part you

Re: Merge request: Merging api_refactoring on master

2012-12-23 Thread Nitin Mehta
that as much as possible…not that I am shying away from that :). You see a problem, you rant, bring on ML (maybe flame couple of culprits) and send a patch, or as a community member/contributor/any-damn-dudeness-title fix it. Regards. On 22-Dec-2012, at 10:49 PM, Nitin Mehta nitin.me

Re: [Discuss] Cpu and Ram overcommit.

2012-12-26 Thread Nitin Mehta
Thanks Bharat for the bringing this up. I have a few questions and suggestions for you. 1. Why do we need it per cluster basis and when and where do you configure this ? I hope when we change it for a cluster it would not require MS reboot and be dynamically understood - is that the case ? If

Re: [DISCUSS] Virtual machine's Base Image Updatation Facility

2012-12-26 Thread Nitin Mehta
Hi Hari, My comments inline. On 26-Dec-2012, at 12:18 PM, Harikrishna Patnala wrote: When a user tries to create VM, following sequence of events happen with current CloudStack implementation (1)User / admin adds a template (which is stored on secondary storage) (2)Template

Re: [DISCUSS] Limit Resources to domain/account

2012-12-26 Thread Nitin Mehta
Now that we are doing this can we also not do the checking limits through ACL refactoring we are going to do ? Thanks, -Nitin On 26-Dec-2012, at 2:56 PM, Sanjay Tripathi wrote: Hi all, CloudStack today offers the ability to limit the following resources to a domain/account: 1. VM -

Re: Mistakenly push the nonoss public, now reverted

2012-12-26 Thread Nitin Mehta
Gavin - Should we put in a git hook to avoid this from happening in future ? Thanks, -Nitin On 26-Dec-2012, at 7:43 AM, Gavin Lee wrote: Dear all, I made a big mistake of pushing nonoss lib into remote. commit id: a387fcdd7cc10b7443453fbe84afc06cf51d2902

Re: [Discuss] Cpu and Ram overcommit.

2012-12-26 Thread Nitin Mehta
...@citrix.com] Sent: Wednesday, December 26, 2012 4:39 AM To: cloudstack-dev@incubator.apache.org; cloudstack-us...@incubator.apache.org Subject: Re: [Discuss] Cpu and Ram overcommit. Nitin thanks for your suggestions. My comments inline On Dec 26, 2012, at 3:22 PM, Nitin Mehta nitin.me

Re: [DISCUSS] Dedicated Resources: Dedicate Pods, Clusters, Hosts to a domain

2012-12-26 Thread Nitin Mehta
On 27-Dec-2012, at 4:47 AM, Hari Kannan wrote: Hi Alex, There is no requirement for the end user administer the hardware - Regarding the OAMP, I believe the resources are still owner, administered, maintained and provisioned by the root admin - they are simply reserved for the said

Re: Restore a VM from template

2013-01-05 Thread Nitin Mehta
Hi Kelcey / Trevor - I think you got confused between 2 apis we have in Cloudstack - restoreVirtualMachine and recoverVirtualMachine. restore vm api restores a new volume for the vm keeping the IP address and other attributes intact. This action is available only when the vm is stopped or

Re: [Discuss] Cpu and Ram overcommit.

2013-01-08 Thread Nitin Mehta
suggestions. My comments inline On Dec 26, 2012, at 3:22 PM, Nitin Mehta nitin.me...@citrix.com wrote: Thanks Bharat for the bringing this up. I have a few questions and suggestions for you. 1. Why do we need it per cluster basis and when and where do you configure this ? I hope when we change

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

2013-01-09 Thread Nitin Mehta
on that ? - Nitin Mehta On Jan. 9, 2013, 7:29 a.m., saksham srivastava wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8900

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

2013-01-09 Thread Nitin Mehta
...@citrix.com, cloudstack-dev@incubator.apache.orgmailto:cloudstack-dev@incubator.apache.org cloudstack-dev@incubator.apache.orgmailto:cloudstack-dev@incubator.apache.org, Nitin Mehta nitin.me...@citrix.commailto:nitin.me...@citrix.com Subject: Re: Review Request: CLOUDSTACK-924: source_template_id set

  1   2   3   >