Re: Review Request: CLOUDSTACK-2130: UpdateDefaultNicForVirtualMachine api should also create usage events for updating new default network

2013-05-16 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11156/#review20636 --- Ship it! Ship It! - mice xia On May 14, 2013, 3:53 p.m.,

[ACS42] Release Status Update - 15 days left to freeze

2013-05-16 Thread Animesh Chaturvedi
Folks We are 15 days from ACS 42 feature freeze Out of 98 proposed features /improvements, the status is Closed: 5 Resolved: 32 In Progress: 15 Reopened: 1: Ready to review: 2 Open: 43 For the issues that are Open I will ask to take a moment to update the jira items assigned to you with In

Re: [ACS41] Current blocker status

2013-05-16 Thread Prasanna Santhanam
On Wed, May 15, 2013 at 03:07:41PM -0700, Animesh Chaturvedi wrote: CLOUDSTACK-2516 Create User API compability broken now Nobody has taken this on yet. [Animesh] I will update the JIRA ticket, looks like this was changed by Kishan to use clear text password. I think functionality wise

RE: [MERGE] [CLOUDSTACK-2056] DeploymentPlanner choice via ServiceOffering

2013-05-16 Thread Prachi Damle
I had written the test as per the old wiki page, now will refer to the new wiki and make the changes to the test. Thanks for the wiki example! If there is no objection, I will do these changes post-merge. Thanks, Prachi -Original Message- From: Prasanna Santhanam

Re: Review Request: CLOUDSTACK-2130: UpdateDefaultNicForVirtualMachine api should also create usage events for updating new default network

2013-05-16 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11156/#review20637 --- Commit e26081731898e2b3de53fdfb9f58d6fdb017f09f in branch

Re: [MERGE] [CLOUDSTACK-2056] DeploymentPlanner choice via ServiceOffering

2013-05-16 Thread Prasanna Santhanam
On Wed, May 15, 2013 at 11:22:13PM -0700, Prachi Damle wrote: I had written the test as per the old wiki page, now will refer to the new wiki and make the changes to the test. Thanks for the wiki example! If there is no objection, I will do these changes post-merge. Sure - not a problem. +1

RE: [ACS41] Current blocker status

2013-05-16 Thread Kishan Kavala
-Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Thursday, 16 May 2013 11:51 AM To: dev@cloudstack.apache.org Subject: Re: [ACS41] Current blocker status On Wed, May 15, 2013 at 03:07:41PM -0700, Animesh Chaturvedi wrote: CLOUDSTACK-2516 Create User

Re: [ACS41] Current blocker status

2013-05-16 Thread Prasanna Santhanam
On Thu, May 16, 2013 at 11:51:06AM +0530, Prasanna Santhanam wrote: On Wed, May 15, 2013 at 03:07:41PM -0700, Animesh Chaturvedi wrote: CLOUDSTACK-2516 Create User API compability broken now Nobody has taken this on yet. [Animesh] I will update the JIRA ticket, looks like this was

Review Request: Change unit of network statistics from 1000 to 1024 for KVM

2013-05-16 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11195/ --- Review request for cloudstack. Description --- As the unit of network

Review Request: remove the dependency on config file in the test_secondary_storage.py

2013-05-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11196/ --- Review request for cloudstack and Prasanna Santhanam. Description --- Get

Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11194/ --- Review request for cloudstack and Chip Childers. Summary (updated)

RE: [ACS41] Current blocker status

2013-05-16 Thread Kishan Kavala
-Original Message- From: Kishan Kavala [mailto:kishan.kav...@citrix.com] Sent: Thursday, 16 May 2013 1:24 PM To: dev@cloudstack.apache.org Subject: RE: [ACS41] Current blocker status -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent:

RE: [jira] [Commented] (CLOUDSTACK-2354) Unable to create Windows VMs using ISO

2013-05-16 Thread Sanjay Tripathi
Nitin, I have unassigned this ticket as it is related to your feature Scaling up Memory of running VMs and the fix for this bug is disabling the scaling up functionality. I have updated the ticket with my findings. Can you please assign it to yourself as you have the better idea about the

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Ove Ewerlid
Hi Kishan! Did you verify that adding the plain text authenticator will not allow login using the hash value itself? from AccountManagerImpl.java; ... getUserAccount ... ... boolean authenticated = false; for(UserAuthenticator authenticator : _userAuthenticators) {

Review Request: Fix router tests for basic zone

2013-05-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11197/ --- Review request for cloudstack and Prasanna Santhanam. Description ---

Jira Karma

2013-05-16 Thread Dave Brosius
Greetings, Would someone be kind enough to give me Jira Karma to assign myself to issues? username: dbrosius

Re: [ACS42] System VM's are failed to start with NPE with latest master(VMWARE)

2013-05-16 Thread Nitin Mehta
Is this only for vmware ? Did you try other hypervisors since it looks generic enough ? On 16/05/13 5:19 PM, Sailaja Mada sailaja.m...@citrix.com wrote: Hi, With latest master , System VM's are failed to start with NPE. Is there any issue with recent fixes ? ( VMWARE) 2013-05-16 17:12:20,523

RE: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Kishan Kavala
Ove, Plain text authenticator will allow logging using the hash value. Or else, clients sending MD5 hash will fail to login. This is primarily for backward compatibility. To avoid logging in using has value itself, plain text authenticator can be removed from auth adapter list, provided the

RE: [ACS42] System VM's are failed to start with NPE with latest master(VMWARE)

2013-05-16 Thread Likitha Shetty
I saw the same issue with Xen. Thanks, Likitha -Original Message- From: Nitin Mehta [mailto:nitin.me...@citrix.com] Sent: Thursday, May 16, 2013 5:54 PM To: dev@cloudstack.apache.org Cc: Sailaja Mada Subject: Re: [ACS42] System VM's are failed to start with NPE with latest master(VMWARE)

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Ove Ewerlid
On 05/16/2013 02:16 PM, Kishan Kavala wrote: Ove, Plain text authenticator will allow logging using the hash value. Or else, clients sending MD5 hash will fail to login. This is primarily for backward compatibility. To avoid logging in using has value itself, plain text authenticator can be

RE: [ACS42] System VM's are failed to start with NPE with latest master(VMWARE)

2013-05-16 Thread Kishan Kavala
It happens only when Vm is staring for the first time irrespective of the hypervisor. This is due to 6d57393629bb93fbf0944e462c841f245b40f919. Below check on CapacityManagerImpl.java:684 should change from if (vm.getLastHostId().equals(vm.getHostId())) { to if

Re: Review Request: Implicit dedication of resources to an account

2013-05-16 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11045/ --- (Updated May 16, 2013, 1:14 p.m.) Review request for cloudstack and Prachi

Re: Review Request: Implicit dedication of resources to an account

2013-05-16 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11045/ --- (Updated May 16, 2013, 1:15 p.m.) Review request for cloudstack and Prachi

RE: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Kishan Kavala
-Original Message- From: Ove Ewerlid [mailto:ove.ewer...@oracle.com] Sent: Thursday, 16 May 2013 6:25 PM To: dev@cloudstack.apache.org Subject: Re: Review Request: Added PlainTextAuthenticator On 05/16/2013 02:16 PM, Kishan Kavala wrote: Ove, Plain text authenticator will

Re: Review Request: Change unit of network statistics from 1000 to 1024 for KVM

2013-05-16 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11195/#review20641 --- Is there a bug ID for this? And is there a branch target(s) for

Re: Review Request: Change unit of network statistics from 1000 to 1024 for KVM

2013-05-16 Thread Chip Childers
On Thu, May 16, 2013 at 01:42:39PM +, Wei Zhou wrote: On May 16, 2013, 1:39 p.m., Chip Childers wrote: Is there a bug ID for this? And is there a branch target(s) for the patch? Chip, no Bug ID. The target branch are master and 4.1. Mind putting a bug into the system for

RE: [ACS42] System VM's are failed to start with NPE with latest master(VMWARE)

2013-05-16 Thread Sailaja Mada
Thanks Kishan for the quick fix provided with https://issues.apache.org/jira/browse/CLOUDSTACK-2544 . Regards, Sailaja.M From: Kishan Kavala Sent: Thursday, May 16, 2013 6:35 PM To: dev@cloudstack.apache.org Cc: Sailaja Mada Subject: RE: [ACS42] System

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-16 Thread Chip Childers
On Thu, May 16, 2013 at 03:53:31PM +0200, nicolas.lamira...@orange.com wrote: Our PROD is in 2.2.13, using Advaned Zone and SG enabled. But we don't use SG features. We use vmWare. We try to upgrade our pre-production environment to 4.1, but we failed due to the CLOUDSTACK-2463 bug. We have

Re: Review Request: Change unit of network statistics from 1000 to 1024 for KVM

2013-05-16 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11195/ --- (Updated May 16, 2013, 1:57 p.m.) Review request for cloudstack. Changes

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-16 Thread nicolas.lamirault
Le 16/05/2013 15:56, Chip Childers a écrit : If we were able to upgrade you correctly to 4.1, but security groups were not yet functional, would that suffice? yes. -- Nicolas Lamirault

Re: [ACS41] Discuss CLOUDSTACK-2463 being resolved in 4.1 vs 4.2

2013-05-16 Thread Chip Childers
On Thu, May 16, 2013 at 03:59:06PM +0200, nicolas.lamira...@orange.com wrote: Le 16/05/2013 15:56, Chip Childers a écrit : If we were able to upgrade you correctly to 4.1, but security groups were not yet functional, would that suffice? yes. Anthony, Since VMware support for SG's in

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Ove Ewerlid
I vote -1 for enabling plain text authentication allowing auth directly against hashes. I'm not clear if this functionality exists in ACS4.0, I would assume not. The API breakage reported was in createUser, where the ability to pass in a hash has value. Think migration scenarios where only

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Prasanna Santhanam
On Thu, May 16, 2013 at 04:03:14PM +0200, Ove Ewerlid wrote: I vote -1 for enabling plain text authentication allowing auth directly against hashes. I'm not clear if this functionality exists in ACS4.0, I would assume not. The API breakage reported was in createUser, where the ability to

Re: Review Request: CLOUDSTACK-2115 : [BasicZone-XenServer] Unable to add host to basic zone that is configured with bridge

2013-05-16 Thread Prasanna Santhanam
On May 14, 2013, 4:22 p.m., Prasanna Santhanam wrote: LGTM. Fairly straightforward to strip the word endings. Any concerns? I'll merge this tomorrow if none. Unfortunately 1518e7ee was fixed without applying your patch. I'm closing this as discarded. - Prasanna

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Chiradeep Vittal
+1 On 5/16/13 9:20 AM, Prasanna Santhanam t...@apache.org wrote: On Thu, May 16, 2013 at 04:03:14PM +0200, Ove Ewerlid wrote: I vote -1 for enabling plain text authentication allowing auth directly against hashes. I'm not clear if this functionality exists in ACS4.0, I would assume not.

Re: About testing of OVS Tunnel Manager, Fail on create new instance

2013-05-16 Thread Chiradeep Vittal
Did you specify the guest network vlan range in the create zone wizard? The exception is complaining that there are no virtual network ids available. Vnet ids are populated in the op_dc_vnet_alloc table when the updatePhysicalNetwork api is called. On 5/16/13 8:07 AM, 錦為 rumania0...@hotmail.com

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-16 Thread Sheng Yang
Hi Hiroaki, Ping again. I've got a patch(pvlan migration support for xen/kvm) based on your patch here. I was assuming it would be soon for you to submit/merge the patch, but it has been one month since last update. And now the lacking of your patch blocked my fix... Could you tell what's

RE: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Koushik Das
Nice explanation Prasanna. +1 to Kishan's fix. -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Thursday, May 16, 2013 9:50 PM To: dev@cloudstack.apache.org Cc: Kishan Kavala Subject: Re: Review Request: Added PlainTextAuthenticator On Thu, May 16, 2013

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-16 Thread Chiradeep Vittal
Sheng, I'd suggest that you go ahead and merge the patch. On 5/16/13 11:20 AM, Sheng Yang sh...@yasker.org wrote: Hi Hiroaki, Ping again. I've got a patch(pvlan migration support for xen/kvm) based on your patch here. I was assuming it would be soon for you to submit/merge the patch, but it

Re: Review Request: (CLOUDSTACK-1638) Network plugins won't be notified VM migration.

2013-05-16 Thread Chip Childers
On Thu, May 16, 2013 at 06:55:30PM +, Chiradeep Vittal wrote: Sheng, I'd suggest that you go ahead and merge the patch. +1

Re: [ACS41] Current blocker status

2013-05-16 Thread Chiradeep Vittal
See this discussion http://s.apache.org/gW On 5/16/13 12:53 AM, Kishan Kavala kishan.kav...@citrix.com wrote: -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Thursday, 16 May 2013 12:39 PM To: dev@cloudstack.apache.org Subject: Re: [ACS41] Current

Re: [GSOC] Student interviews

2013-05-16 Thread ADAMA COULIBALY
Hi Sebastien, You and I have appointment today at 3pm. I cannot find you on Skype. Please let me know what time you will be calling me my phone number is +1 267-243-3954 Adama On Mon, May 13, 2013 at 6:24 AM, Sebastien Goasguen run...@gmail.comwrote: On May 13, 2013, at 7:45 AM, ADAMA

Properties file question

2013-05-16 Thread Mike Tutkowski
Hi everyone, I'm in the process of adding a few fields to the Add Disk Offering dialog (Min, Max, and Burst IOPS). I was wondering which of the three English .properties files I need to add my new text into? Thanks! -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e:

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Chip Childers
On Thu, May 16, 2013 at 09:50:21PM +0530, Prasanna Santhanam wrote: On Thu, May 16, 2013 at 04:03:14PM +0200, Ove Ewerlid wrote: I vote -1 for enabling plain text authentication allowing auth directly against hashes. I'm not clear if this functionality exists in ACS4.0, I would assume not.

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11194/#review20655 --- Ship it! Ship It! - Chip Childers On May 16, 2013, 10:39 a.m.,

Re: [ACS41] Current blocker status

2013-05-16 Thread Chip Childers
On Wed, May 15, 2013 at 03:07:41PM -0700, Animesh Chaturvedi wrote: -Original Message- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Wednesday, May 15, 2013 12:43 PM To: dev@cloudstack.apache.org Subject: [ACS41] Current blocker status The following are

Re: Review Request: Change unit of network statistics from 1000 to 1024 for KVM

2013-05-16 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11195/#review20657 --- Ship it! Ship It! - Chip Childers On May 16, 2013, 1:57 p.m.,

Example code for GUI

2013-05-16 Thread Mike Tutkowski
Hi everyone, I'm trying to add a few fields to the GUI's Add Disk Offering dialog (min, max, and burst IOPS). Once I have these fields in place, they will of course need to pass this information to the backend and the data will need to end up in new fields in the DB. I was wondering if anyone

RE: [MERGE]PVLAN branch to MASTER(v2)

2013-05-16 Thread Vijayendra Bhamidipati
I'm done with implementation of pvlan changes for VMWare - I've tested and uploaded the diffs as a file on top of the current master branch to https://reviews.apache.org/r/11019/ I wasn't able to update the patch on review board though - I run into the error error: short SHA1 01a0384 is

Re: Review Request: PVLAN provisioning support for vmware Distributed Virtual Switch deployments on cloudstack.

2013-05-16 Thread Sheng Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11019/#review20664 --- Ship it! Applied to master. Thanks! commit

Re: [MERGE]PVLAN branch to MASTER(v2)

2013-05-16 Thread Sheng Yang
I've successfully applied the patch anyway. Thank you! --Sheng On Thu, May 16, 2013 at 2:55 PM, Vijayendra Bhamidipati vijayendra.bhamidip...@citrix.com wrote: I'm done with implementation of pvlan changes for VMWare - I've tested and uploaded the diffs as a file on top of the current

Re: Review Request: Added PlainTextAuthenticator

2013-05-16 Thread Ove Ewerlid
On 05/16/2013 10:55 PM, Chip Childers wrote: For those moving from 2.2.x, 3.0.x, 4.0 to 4.1: 1. We remove the incorrect auth mechanism and put in the right fix of encoding at the server and not doing any UI magic. 2. We correct the API docs and other docs to indicate the user to send in

Re: Example code for GUI

2013-05-16 Thread Wei ZHOU
Mike, I am working on Disk I/O Throttling now. The source code has been verified on 4.0.2. It need more test on master branch. I hope to finish it before the deadline of 4.2 feature freeze (31-5-2013). For more information: https://issues.apache.org/jira/browse/CLOUDSTACK-1301

RE: [MERGE] [CLOUDSTACK-2056] DeploymentPlanner choice via ServiceOffering

2013-05-16 Thread Prachi Damle
Merged this branch to master. Thanks, Prachi -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: Wednesday, May 15, 2013 11:28 PM To: dev@cloudstack.apache.org Subject: Re: [MERGE] [CLOUDSTACK-2056] DeploymentPlanner choice via ServiceOffering On Wed, May 15,

Re: Example code for GUI

2013-05-16 Thread Mike Tutkowski
Thanks for the info, Wei. I am in the process of writing a storage plug-in for Edison's new storage framework. The idea is to be able to have a single SAN volume associated with a single data disk. In our SAN, you can set Min, Max, and Burst IOPS values on a volume-by-volume basis and I'd like to

Re: Example code for GUI

2013-05-16 Thread Mike Tutkowski
It looks like your feature revolves around rate limiting, if I understand correctly. It seems like I could leverage a bit of what you're doing here, but I'd need additional fields in the Add Disk Offering dialog (and DB) since we support min, max, and burst values for IOPS (as opposed to just a

Re: Example code for GUI

2013-05-16 Thread Wei ZHOU
Mike, The features are totally different. For the VM Disk I/O Throttling, I want to limit the vm disk I/O in the hypervisors (support KVM only now). However, you do it in the storage device (such as SAN). I added two fields: bps and iops (maximum, not average/minimum/burst) in disk_offering and

RE: Example code for GUI

2013-05-16 Thread Musayev, Ilya
We actually have a similar question on the GUI part, we've done lots of backend work, but the GUI part does not seem very straightforward. Would someone know how to add options into GUI? Also, How can I change the field type from in UI from text field to multi-select or checkbox group? What

Re: Example code for GUI

2013-05-16 Thread Wei ZHOU
From my point view: (1) UI: ui/scripts/*.js (such as network.js, configuration.js) ui/dictionary.jsp client/WEB-INF/classes/resources/messages.properties (2) Java *Cmd.java (commands) *VO.java (tables) *Manager.java *ManagerImpl.java Wei 2013/5/17 Musayev, Ilya

Re: Example code for GUI

2013-05-16 Thread Musayev, Ilya
Thank you Wei . Any idea on how can I change the field type from in UI from text field to multi-select or checkbox group? Is this CSS only change or is there more to it? Original message From: Wei ZHOU ustcweiz...@gmail.com Date: To: dev@cloudstack.apache.org Subject: Re:

Re: Example code for GUI

2013-05-16 Thread Isaac Chiang
Hi Musayev, You'll find out that there are createForm object under listView(or detailView) actions in each file, e.g., instance.js, accounts.js. If you need a selection, you can just define a attribute for createForm like following: select: function(args) { //... } and pass

Re: Example code for GUI

2013-05-16 Thread Musayev, Ilya
Thanks Isaac Regards Ilya Original message From: Isaac Chiang isaacchi...@gmail.com Date: To: dev@cloudstack.apache.org,Musayev, Ilya imusa...@webmd.net Subject: Re: Example code for GUI Hi Musayev, You'll find out that there are createForm object under

Re: Example code for GUI

2013-05-16 Thread Mike Tutkowski
Hi Wei, Thank you for the information. Your commit info should be quite valuable for me as a template for my own work. Talk to you later! On Thu, May 16, 2013 at 4:50 PM, Wei ZHOU ustcweiz...@gmail.com wrote: Mike, The features are totally different. For the VM Disk I/O Throttling, I

Re: Properties file question

2013-05-16 Thread Mike Tutkowski
Perhaps someone knows why we sometimes use strings in English and other times use keys that resolve to the selected language. Trying to learn more about how our GUI works (example below). Thanks! + ramovercommit:{** + label: 'RAM overcommit ratio',** +

Re: About testing of OVS Tunnel Manager, Fail on create new instance

2013-05-16 Thread 錦為
Hi Chiradeep, Thanks for your suggestion. When I configures sdn.ovs.controller to true, I do not specify the vnet (vlan) range of guest network in the create advanced zone wizard. And, the field vnet is null in database table physical_network. How to specify this parameters?

RE: Example code for GUI

2013-05-16 Thread Pranav Saxena
Hey Ilya , So this is how you declare the following items - object name :{ createForm:{ fields:{ //CheckBox Item item object name :{ Label:'Label Name', isBoolean:true,// This statement creates a checkbox

Re: [DOCS] Quick Install Guide updated lately?

2013-05-16 Thread Jessica Tomechak
On Mon, May 13, 2013 at 5:46 PM, David Nalley da...@gnsa.us wrote: On Mon, May 13, 2013 at 8:33 PM, Jessica Tomechak jessica.tomec...@gmail.com wrote: There is a Quick Install Guide published on the docs site for 4.0.x. Has that been updated for 4.1, or is it even going to be published for

Fw: About testing of OVS Tunnel Manager, Fail on create new instance

2013-05-16 Thread Garnet Tang
Hi Chiradeep, Thanks for your suggestion. When I configures sdn.ovs.controller to true, I do not specify the vnet (vlan) range of guest network in the create advanced zone wizard. And, the field vnet is null in database table physical_network. How to specify this parameters?

Re: Example code for GUI

2013-05-16 Thread Mike Tutkowski
Hi Pranav, Thanks for posting the link to your commit, as well. :) I'm trying to set up my fields based on the pattern we already have today where the Admin can check a box to allow the User to specify a disk size. In my case, I'd like the Admin to check a box if he wants the User to

Re: Example code for GUI

2013-05-16 Thread Mike Tutkowski
Perhaps I need to place my checks in the Action area? On Thu, May 16, 2013 at 10:10 PM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hi Pranav, Thanks for posting the link to your commit, as well. :) I'm trying to set up my fields based on the pattern we already have today where

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

2013-05-16 Thread Jayapal Reddy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9696/#review20678 --- Changes are looking fine. Nithin can you also please review the

RE: Example code for GUI

2013-05-16 Thread Pranav Saxena
Yeah , so these dependencies/relationships would be handled when you are passing these parameters to the API call within the actions block. Below is the code snippet which will help you accomplish your tasks (might require little tweaking ) but that's how the structure should look like .

RE: Example code for GUI

2013-05-16 Thread Pranav Saxena
You might also need to provide a logic at the API layer that It correctly handles the case for all 3 parameters being supplied together and thrown an exception if any one of the three parameters is missing . Moreover , you should also implement an API level check for the minlops and maxlops to

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

2013-05-16 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9696/#review20681 --- Ship it! Ship It! - Nitin Mehta On March 15, 2013, 9:54 a.m.,

RE: Example code for GUI

2013-05-16 Thread Pranav Saxena
So within this block , you could use anyone of the exception handling mechanisms as shown below - $.ajax({ url:'API CMD', data:data , success:function(args){ args.response.success(); },

Re: Review Request: debian: fix build of cloudstack-awsapi package

2013-05-16 Thread Animesh Chaturvedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11180/ --- (Updated May 17, 2013, 5:24 a.m.) Review request for cloudstack, Prachi Damle

Re: [ACS41] Current blocker status

2013-05-16 Thread Prasanna Santhanam
On Thu, May 16, 2013 at 07:12:14PM +, Chiradeep Vittal wrote: See this discussion http://s.apache.org/gW Nice - thanks for digging it up from the archives. I couldn't find it and had to write a long email explaining things :) -- Prasanna., Powered by

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

2013-05-16 Thread Animesh Chaturvedi
On May 17, 2013, 5:18 a.m., Nitin Mehta wrote: Ship It! Nitin can you commit the patch - Animesh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9696/#review20681