Review Request: listSnapshotPolicies command, volumeId parameter made optional.

2012-06-26 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5604/ --- Review request for cloudstack. Description --- Changes made to allow the ad

Re: Nicira Nvp integration

2012-06-26 Thread KAWAI Hiroaki
Hi, I tried to apply the patch, but failed with apache git master HEAD due to the file path changes recently committed. Branch (and rev) is missing in that review page, but following worked: git clone https://git-wip-us.apache.org/repos/asf/incubator-cloudstack.git cd incubator-cloudstack git ch

Re: [PROPOSAL] Further refinement of CloudStack modules...

2012-06-26 Thread Murali Reddy
In continuation to what Alex proposed, as a first step I am moving out components that are not core to the CloudStack into a physically separate jar's and in to a different projects. Please look at [1] for what components are being moved and corresponding new location. Next few days I will moving

RE: CloudStack Japanese user mailing list

2012-06-26 Thread Kimihiko Kitase
Adding cloudstack-us...@incubator.apache.org.. -Original Message- From: Kimihiko Kitase [mailto:kimihiko.kit...@citrix.co.jp] Sent: Wednesday, June 27, 2012 9:38 AM To: cloudstack-us...@incubator.apache.org Subject: RE: CloudStack Japanese user mailing list Hi, It seems someone from ment

Re: CloudStack API question

2012-06-26 Thread Alena Prokharchyk
Here is the doc explaining @ usage in API commands / API responses: http://wiki.cloudstack.org/display/dev/Annotations+use+in+API -Alena. On 6/25/12 9:37 AM, "Alena Prokharchyk" wrote: >I'll write something up, and let you know the wiki page location once I'm >done. > >-Alena. > > >On 6/25/12

Re: [PATCH] CS-15329: Instances with Error states appear in Instances

2012-06-26 Thread Jessica Tomechak
> > -Original Message- > From: Jessica Wang [mailto:jessica.w...@citrix.com] > Sent: Tuesday, June 26, 2012 3:12 PM > To: olga smola (olya.sm...@gmail.com) > Cc: cloudstack-dev@incubator.apache.org > Subject: RE: [PATCH] CS-15329: Instances with Error states appear in > Instances > > Olga,

Re: Review Request: CS-15329: Instances with Error states dissapear from Instances list after creating.

2012-06-26 Thread Brian Federle
> On June 26, 2012, 10:17 p.m., Brian Federle wrote: > > Ship It! Pushed to master by jessica. - Brian --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5575/#review8641 ---

Re: Review Request: CS-15329: Instances with Error states dissapear from Instances list after creating.

2012-06-26 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5575/#review8641 --- Ship it! Ship It! - Brian Federle On June 26, 2012, 7:20 a.m., Olg

RE: [PATCH] CS-15329: Instances with Error states appear in Instances

2012-06-26 Thread Jessica Wang
Olga, I just reviewed and pushed your patch to master branch. I also checked in another change: (1) to distinguish between getUpdatedItem() and getUpdatedData(), I renamed getUpdatedData() to getUpdatedItemWhenAsyncJobFails() since it's being called only when async job fails. (2) to fix the sp

Re: suggest change line in the custom UI guide

2012-06-26 Thread Lu Heng
Hi Sorry, but my idea does not work, it seems to me that the URL(x.x.x.x) can open the login page, however, if I tried to login, I got an error saying "invaild username or password" Same password works on x.x.x.x:8080/client. Any ideas? On Tue, Jun 26, 2012 at 10:15 PM, Jessica Tomechak wrote:

Re: Review Request: CS-15290: UI "Path" field verified like required, but isn't marked

2012-06-26 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5581/#review8639 --- Ship it! Ship It! - Brian Federle On June 26, 2012, 10:13 a.m., Ol

Re: Review Request: CS-15287: Required fields after Edit action can be removed.

2012-06-26 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5571/#review8638 --- Ship it! Ship It! - Brian Federle On June 26, 2012, 7:09 a.m., Olg

Re: suggest change line in the custom UI guide

2012-06-26 Thread Jessica Tomechak
> > -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Monday, June 25, 2012 6:07 AM > To: cloudstack-dev@incubator.apache.org > Subject: Re: suggest change line in the custom UI guide > > On Sun, Jun 24, 2012 at 3:46 PM, Lu Heng wrote: > > Hi > > > > It tooks me quite

Re: FW: Easy task for folks looking to get involved

2012-06-26 Thread Jessica Tomechak
See reply below. On Fri, Jun 22, 2012 at 9:20 PM, David Nalley wrote: > On Fri, Jun 22, 2012 at 6:40 PM, Jessica Tomechak > wrote: > > See inline below for my reply to David's question about purging the docs > > directory in the repo. > > > > > > -Original Message- > > From: David Nalle

RE: Construct / change role permissions

2012-06-26 Thread Deepti Dohare
I have added a patch (https://reviews.apache.org/r/5573/diff/#index_header) for CS-15300. It enables a domain admin account to respect the limits. This bug also state that domain admin shouldn't have the right to create infinite resources. In this case what should be the maximum limit of the

Re: Review Request: CS-15315: UI Wizard's scroll, overlay issue in FF

2012-06-26 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5572/#review8605 --- Ship it! Ship It! - Brian Federle On June 26, 2012, 7:15 a.m., Olg

Nicira Nvp integration

2012-06-26 Thread Hugo Trippaers
Heya all, Based on the feedback on the pre-review I made quite a few changes. Functionally it seems to be doing what I want it to do and the code integrates a lot better with the existing code. I've pushed the patch to reviewboard at https://reviews.apache.org/r/5590/ I hope this patch address

Re: Review Request: Nicira NVP integration for CloudStack

2012-06-26 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5590/ --- (Updated June 26, 2012, 4:56 p.m.) Review request for cloudstack. Description

Review Request: Nicira NVP integration for CloudStack

2012-06-26 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5590/ --- Review request for cloudstack. Description --- Patch to add Nicira NVP supp

Re: Review Request: License header updates for the test folder.

2012-06-26 Thread David Nalley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5444/#review8601 --- Ship it! Looks good - applied Thanks for the patch, Chip - David Na

Re: Review Request: License header updates for awsapi-setup folder.

2012-06-26 Thread David Nalley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5438/#review8599 --- Ship it! This problem was on my end. My apologies. I have things setu

Re: Review Request: Fixed broken /deps copyright headers

2012-06-26 Thread David Nalley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5553/#review8598 --- Ship it! Applied, thanks for the patch. - David Nalley On June 25,

Re: Review Request: License header updates for the ui folder.

2012-06-26 Thread David Nalley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5445/#review8597 --- Ship it! Thanks for the patch Chip,- applied - David Nalley On Ju

Re: Review Request: License header updates for the setup folder.

2012-06-26 Thread David Nalley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5443/#review8596 --- Ship it! thanks for the patch Chip. Applied - David Nalley On June

Re: [PATCH] Fix broken debian packaging

2012-06-26 Thread Tomoe Sugihara
On Tue, Jun 26, 2012 at 6:07 PM, Wido den Hollander wrote: > On 06/26/2012 10:59 AM, Tomoe Sugihara wrote: > >> Signed-off-by: Tomoe Sugihara >> --- >> debian/cloud-deps.install |2 +- >> debian/cloud-server.install |4 >> debian/cloud-utils.install |1 - >> debian/control

Re: CloudStack User Research

2012-06-26 Thread Outback Dingo
On Tue, Jun 26, 2012 at 8:58 AM, Jason Pavao wrote: > Hi Lynn, > We would like to POC with xenserver, but we don't have any licenses, would > you be able to help us pro-cure some temporary licenses so that we could do > a poc with citrix xenserver? > > Let me know. > Thanks. > -jason > WOW oracle

Re: CloudStack User Research

2012-06-26 Thread Jason Pavao
Hi Lynn, We would like to POC with xenserver, but we don't have any licenses, would you be able to help us pro-cure some temporary licenses so that we could do a poc with citrix xenserver? Let me know. Thanks. -jason On 6/26/2012 5:36 AM, Lynn Van Dyke wrote: Hello. I work for the Citrix pro

CloudStack User Research

2012-06-26 Thread Lynn Van Dyke
Hello. I work for the Citrix product design team. If you're using CloudStack with XenServer I'm hoping that you would like to give me some 1:1 user experience feedback. I'm looking to do a small number of interviews over GoToMeeting over the next few weeks. They'll last 60 - 90 minutes and cover

Re: [PATCH] Fix broken debian packaging

2012-06-26 Thread David Nalley
> > > I just checked, but we are still using the daemonize binary in the init > scripts for the agent, usage and console-proxy are still using the daemonize > binary. > > However, in wscript_build the compiling for the binary is commented out. > > How do we proceed with this? > > Wido > Frank: D

Review Request: CS-13535: UI After Closing the Error Message thrown during Add Domain operation - the Spinner never stops.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5583/ --- Review request for cloudstack and Jessica Wang. Description --- Fix for htt

[PATCH] CS-13535: UI fix when Add Domain operation failed.

2012-06-26 Thread olgasmola
--- ui/scripts/domains.js |4 1 file changed, 4 insertions(+) diff --git a/ui/scripts/domains.js b/ui/scripts/domains.js index 9179b42..7547ec1 100644 --- a/ui/scripts/domains.js +++ b/ui/scripts/domains.js @@ -168,6 +168,10 @@ success: function(json) {

Review Request: CS-15290: UI "Path" field verified like required, but isn't marked

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5581/ --- Review request for cloudstack and Brian Federle. Description --- Fix for ht

Review Request: CS-14047: Login time out does not return the user to the log in page.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5580/ --- Review request for cloudstack and Jessica Wang. Description --- Improvement

Re: UI tickets: Review request

2012-06-26 Thread Olga Smola
Pranav, thanks, done. Olga On Tue, Jun 26, 2012 at 11:35 AM, Pranav Saxena wrote: > Hi Olga, > > ** ** > > You could upload your updated patch on the review board tool ( > https://reviews.apache.org) , which you have been doing earlier also . > Brian/Jessica would carry it forward from ther

Review Request: CS-15292: Split long OS Type list when create Template.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5579/ --- Review request for cloudstack and Brian Federle. Description --- Improvemen

[PATCH] CS-15292: Split long OS Type list when create Template.

2012-06-26 Thread olgasmola
--- ui/scripts/templates.js | 59 --- 1 file changed, 45 insertions(+), 14 deletions(-) diff --git a/ui/scripts/templates.js b/ui/scripts/templates.js index e49b94f..44408c4 100644 --- a/ui/scripts/templates.js +++ b/ui/scripts/templates.js @@ -200,20

Re: [PATCH] Fix broken debian packaging

2012-06-26 Thread Wido den Hollander
On 06/26/2012 10:59 AM, Tomoe Sugihara wrote: Signed-off-by: Tomoe Sugihara --- debian/cloud-deps.install |2 +- debian/cloud-server.install |4 debian/cloud-utils.install |1 - debian/control | 21 - 4 files changed, 5 insertions(+),

Re: building debian packages from master branch

2012-06-26 Thread Tomoe Sugihara
Hi Wido, Thanks for the link. Just sent it. After playing around a little bit, I found a couple of more issues caused by missing jar files and incorporated fixes for that as well. Thanks! Tomoe On Tue, Jun 26, 2012 at 3:43 PM, Wido den Hollander wrote: > On 06/26/2012 08:40 AM, Wido den Hollan

[PATCH] Fix broken debian packaging

2012-06-26 Thread Tomoe Sugihara
Signed-off-by: Tomoe Sugihara --- debian/cloud-deps.install |2 +- debian/cloud-server.install |4 debian/cloud-utils.install |1 - debian/control | 21 - 4 files changed, 5 insertions(+), 23 deletions(-) diff --git a/debian/cloud-deps.insta

RE: UI tickets: Review request

2012-06-26 Thread Pranav Saxena
Hi Olga, You could upload your updated patch on the review board tool (https://reviews.apache.org) , which you have been doing earlier also . Brian/Jessica would carry it forward from there. Thanks & Regards, Pranav From: Olga Smola [mailto:olya.sm...@gmail.com] Sent: Tuesday, June 26, 2012 1:

RE: UI tickets: Review request

2012-06-26 Thread Pranav Saxena
Hi olga, I got the point you are trying to make. I thought that the "listOSCategories" command did not exist before and we just had "listOsTypes " command to work with. $.ajax({ url: createURL("listOsTypes"), dataType: "json" $.ajax({ url: createURL("list

Re: UI tickets: Review request

2012-06-26 Thread Olga Smola
Hi Pranav, Excuse me, I can't understand what you mean from here "...however since the backend API command hasn't been implemented for listing the specific OS types". Could you please clarify which API should be implemented? Because I've used already exists API commands "listOsCategories" and "lis

Review Request: CS-15329: Instances with Error states dissapear from Instances list after creating.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5575/ --- Review request for cloudstack and Brian Federle. Description --- Fix for ht

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

2012-06-26 Thread deepti dohare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5573/ --- (Updated June 26, 2012, 7:24 a.m.) Review request for cloudstack. Description

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

2012-06-26 Thread deepti dohare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5573/ --- Review request for cloudstack. Description --- Domain Admin accounts now re

Re: Fix for the issue CS-15143?

2012-06-26 Thread Tomoe Sugihara
Hi Rajesh, Now I think that the SSVM that is described in the installation document ( http://download.cloud.com/templates/acton/acton-systemvm-02062012.qcow2.bz2) still has the problem. Or was the culprit jar file supposed to be overridden by file injection or something? I noticed that SSVM mount

Review Request: CS-15317: Operations with accounts on the Project tab works incorrect after making another account project owner.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5574/ --- Review request for cloudstack and Brian Federle. Description --- Fix for ht

Review Request: CS-15315: UI Wizard's scroll, overlay issue in FF

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5572/ --- Review request for cloudstack and Brian Federle. Description --- Fix for ht

Review Request: CS-15287: Required fields after Edit action can be removed.

2012-06-26 Thread Olga Smola
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5571/ --- Review request for cloudstack and Brian Federle. Description --- Fix for ht