Re: Trouble with system VMs on master

2013-03-07 Thread Rohit Yadav
On Thu, Mar 7, 2013 at 11:36 AM, Nitin Mehta wrote: > Rohit - are you saying that you see it in Running state and not starting > state as Mike says ? > Nitin, I've come across several occasions also while doing the DIY systemvmtemplate when your systemvm starts and ACS shows it as Running but the

Re: Security Groups in Advanced Zone - Plugin Support

2013-03-07 Thread Dave Cahill
Hi Chiradeep, Thanks for jumping in, great to get feedback on this one. Let me back up and explain where we're coming from. Let's take the Firewall service as an example. When a user sets Firewall rules via the UI / API, the request (skipping a few steps for brevity) ends up in FirewallManagerIm

rat build

2013-03-07 Thread Abhinandan Prateek
I tried rat build on clean branch and got following error: de: patches/systemvm/debian/config/var/www/html/latest/.htaccess [INFO] Exclude: patches/systemvm/debian/vpn/etc/ipsec.d/l2tp.conf [INFO] [INFO] BUILD FAILURE [INFO]

Re: rat build

2013-03-07 Thread prasanna
Stuff in **/WEB-INF/** directories needs to be excluded as they are generated. I'll look into that. On 7 March 2013 13:59, Abhinandan Prateek wrote: > --Attaching rat.txt. > > On 07/03/13 1:57 PM, "Abhinandan Prateek" > wrote: > >> >>I tried rat build on clean branch and got following error: >>

Re: rat build

2013-03-07 Thread Rohit Yadav
Hi Abhi, Yes, this is expected and when rat fails it creates the artifact which is rat.txt in target/rat.txt. You read this text file to know your defaulters. Regards. On Thu, Mar 7, 2013 at 1:59 PM, Abhinandan Prateek wrote: > --Attaching rat.txt. > > On 07/03/13 1:57 PM, "Abhinandan Prateek"

Re: rat build

2013-03-07 Thread Abhinandan Prateek
Hi Rohit, Yes, that is fine. I was just trying this utility to check how easy or difficult it is for developers to run it before checkin there code. I guess once Prasanna fixes the "WEB-INF" thing it should give more reliable output. -abhi On 07/03/13 2:04 PM, "Rohit Yadav" wrote: >Hi Abhi,

Re: rat build

2013-03-07 Thread Abhinandan Prateek
Prasanna, thanks for taking this up. On 07/03/13 2:04 PM, "prasanna" wrote: >Stuff in **/WEB-INF/** directories needs to be excluded as they are >generated. I'll look into that. > >On 7 March 2013 13:59, Abhinandan Prateek >wrote: >> --Attaching rat.txt. >> >> On 07/03/13 1:57 PM, "Abhinandan P

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Prasanna Santhanam
> On March 4, 2013, 7:38 a.m., Dave Cahill wrote: > > This looks good to me - and great to see someone is fixing it! > > > > One slightly strange edge case springs to mind - I think the behavior in > > this edge case is fine, but wanted to bring it up explicitly just in case: > > > > In KVM,

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9727/#review17532 --- Hari - the suffix .pub is applied by ssh-keygen. The key will not end

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Prasanna Santhanam
> On March 4, 2013, 7:38 a.m., Dave Cahill wrote: > > This looks good to me - and great to see someone is fixing it! > > > > One slightly strange edge case springs to mind - I think the behavior in > > this edge case is fine, but wanted to bring it up explicitly just in case: > > > > In KVM,

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: Review Request: Feature SNMP Alerts support in CloudStack

2013-03-07 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9587/ --- (Updated March 7, 2013, 9:17 a.m.) Review request for cloudstack and Rajesh Batt

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9727/ --- (Updated March 7, 2013, 9:19 a.m.) Review request for cloudstack and Abhinandan

RE: [DOCS] [ACS41] Status Update for 4.1.0

2013-03-07 Thread Radhika Puthiyetath
Thanks Joe. I have gone through my defect list and made necessary changes. Now there are only two unassigned defects marked for 4.1. -Radhika -Original Message- From: Joe Brockmeier [mailto:j...@zonker.net] Sent: Thursday, March 07, 2013 2:04 AM To: CloudStack Developers Sub

Re: Review Request: Feature SNMP Alerts support in CloudStack

2013-03-07 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9587/ --- (Updated March 7, 2013, 9:26 a.m.) Review request for cloudstack and Rajesh Batt

Re: Review Request: Feature SNMP Alerts support in CloudStack

2013-03-07 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9587/ --- (Updated March 7, 2013, 9:31 a.m.) Review request for cloudstack and Rajesh Batt

Re: rat build

2013-03-07 Thread Abhinandan Prateek
Rat build works fine now. Developers need to do the following to get a good output from rat: Make sure that there are no unwanted files in the local git directory. For rat check, run the following command: mvn --projects='org.apache.cloudstack:cloudstack' org.apache.rat:apache-rat-plugin:0.8:che

Re: Review Request: Feature SNMP Alerts support in CloudStack

2013-03-07 Thread Anshul Gangwar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9587/ --- (Updated March 7, 2013, 9:40 a.m.) Review request for cloudstack and Rajesh Batt

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9727/#review17535 --- Ship it! d51e964 on master - Prasanna Santhanam On March 7, 2013,

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

2013-03-07 Thread Sebastien Goasguen
Joe, This is not a blocker, but I wanted to mention that the translations of the runbook will be valid for 4.0.2. Since the runbook is not yet updated for 4.1. We need to make sure to release those at the same time and put it in the release notes. -sebastien On Mar 7, 2013, at 4:01 AM, Nitin

Re: GIT & JIRA integration!

2013-03-07 Thread Sebastien Goasguen
Is it possible to integrate with review board as well. When a patch is applied, comments gets in JIRA and review gets shipped and submitted ? On Mar 6, 2013, at 6:07 PM, Noah Slater wrote: > Nope! I meant this for CloudStack. I just used a CouchDB ticket as a demo! > > I think this may be ol

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Harikrishna Patnala
> On March 7, 2013, 9:42 a.m., Prasanna Santhanam wrote: > > d51e964 on master Can you push this on 4.1 also. This could be the problem on both master and 4.1 - Harikrishna --- This is an automatically generated e-mail. To reply, visit:

Re: IPv6 plan - questions

2013-03-07 Thread Dave Cahill
Hi Sheng, Thanks for the quick reply, that helps a lot. My main purpose was to figure out how these changes affect virtual networking and pluggability. Having read through the IPv6 code today, it looks like it will work very nicely with virtual networks. For example, when VMs are assigned an IPv

Re: [DISCUSS] Cloudstack agent - standardized directory for plugin jars

2013-03-07 Thread Wido den Hollander
Hi, See this commit: 9e02ed139fe8f7cd9fcb6a989dbe94c326774c6b That should include all JAR files in the plugins directory in the classpath. Wido On 03/06/2013 05:14 PM, Marcus Sorensen wrote: Yes, Ubuntu is doing that, but for example on CentOS, apache is packaged such that you have : [marcus

RE: [DOCS] [ACS41] Status Update for 4.1.0

2013-03-07 Thread Radhika Puthiyetath
Is https://issues.apache.org/jira/browse/CLOUDSTACK-723 ready to document? ( is this feature in for 4.1 ?) -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Thursday, March 07, 2013 2:54 PM To: cloudstack-dev@incubator.apache.org Subject: RE: [DO

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

2013-03-07 Thread Kishan Kavala
Joe, I fixed 1151 in 4.0 branch. Fix has to go into 4.1 and master also. Upgrade steps also need to be documented. ~kishan > -Original Message- > From: Nitin Mehta > Sent: Thursday, 7 March 2013 2:32 PM > To: Joe Brockmeier; CloudStack Developers > Cc: Kishan Kavala > Subject: Re: [ACS4

Re: Review Request: CLOUDSTACK-874 Ability to delete Events and Alerts

2013-03-07 Thread Sanjay Tripathi
> On March 6, 2013, 7:02 p.m., Min Chen wrote: > > server/src/com/cloud/api/query/QueryManagerImpl.java, line 434 > > > > > > So with archive feature, previous listAlerts or listEvents will not > > show archived alerts

[OFFLINE] AFK 8th March - 10th March 2013

2013-03-07 Thread Rohit Yadav
AFK but reachable on email. Regards. PS. Let's use AFK instead of OFFLINE?

Re: Review Request: CLOUDSTACK-874 Ability to delete Events and Alerts

2013-03-07 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9753/ --- (Updated March 7, 2013, 10:46 a.m.) Review request for cloudstack, Devdeep Singh

Re: [PROPOSAL] BVT for CloudStack checkins

2013-03-07 Thread Prasanna Santhanam
On Thu, Mar 07, 2013 at 01:19:19AM +0530, Frank Zhang wrote: > > - improve simulator for runtime testability > > - customize to model and inject failures > > - make a habit of writing tests around bug reports (I started tagging tests > > since api_refactoring on JIRA already. > > look for the integ

RE: [ACS41][QA]Limit Resources per domain/accounts TestPlan Review request

2013-03-07 Thread Sanjay Tripathi
Sailaja, Test Cases looks good to me, you can go ahead and test the feature. Regards, Sanjay > -Original Message- > From: Sailaja Mada [mailto:sailaja.m...@citrix.com] > Sent: Thursday, February 28, 2013 3:21 PM > To: cloudstack-dev@incubator.apache.org > Subject: RE: [ACS41][QA]Limit Res

Re: StoragePoolVO Issue with 4.1

2013-03-07 Thread Prasanna Santhanam
On Thu, Mar 07, 2013 at 02:00:08AM +0530, Ryan Dietrich wrote: > A new column was added to storage_pool, called storage_provider_id. > When trying to access a StoragePoolVO instance (when I go to attach > a volume to a VM) I popped a null pointer. This may be related to > CLOUDSTACK-1547, but I'm

Re: GIT & JIRA integration!

2013-03-07 Thread Prasanna Santhanam
On Thu, Mar 07, 2013 at 03:20:13PM +0530, Sebastien Goasguen wrote: > Is it possible to integrate with review board as well. > > When a patch is applied, comments gets in JIRA and review gets > shipped and submitted ? Yes - I'd like that too! -- Prasanna.,

Re: [DISCUSS]Incremental build

2013-03-07 Thread Prasanna Santhanam
How about we break the cloudstack job on jenkins down to -server, -core, -utils, -plugins etc? Each of the projects takes less than a minute or two to execute. It's the combined project that takes >20m to run on jenkins delaying broken build emails. On Thu, Mar 07, 2013 at 09:13:41AM +0530, Alex

[PROPOSE] Refactor appliance building code under one tools/appliance

2013-03-07 Thread Rohit Yadav
Apart from building systemvms, we also have some code that should build us devcloud and base boxes for development. The proposal is to fix/refactor devcloud appliance building scripts from tools/devcloud/src to tools/appliance. The next proposal is to create a new devcloud that has all the build to

Re: GIT & JIRA integration!

2013-03-07 Thread Abhinandan Prateek
+1 would love such integration. On 07-Mar-2013, at 4:35 PM, "Prasanna Santhanam" wrote: > On Thu, Mar 07, 2013 at 03:20:13PM +0530, Sebastien Goasguen wrote: >> Is it possible to integrate with review board as well. >> >> When a patch is applied, comments gets in JIRA and review gets >> shipp

Review Request: CLOUDSTACK-1574 : updateResourceCount API is failed saying to specify valida resource type even after parsing the valid resource type

2013-03-07 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9799/ --- Review request for cloudstack, Devdeep Singh and Sateesh Chodapuneedi. Descripti

Re: Review Request: CLOUDSTACK-1574 : updateResourceCount API is failed saying to specify valida resource type even after parsing the valid resource type

2013-03-07 Thread Sateesh Chodapuneedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9799/#review17537 --- Ship it! Ship It! - Sateesh Chodapuneedi On March 7, 2013, 11:27

Re: Review Request: CLOUDSTACK-1574 : updateResourceCount API is failed saying to specify valida resource type even after parsing the valid resource type

2013-03-07 Thread Sateesh Chodapuneedi
> On March 7, 2013, 11:36 a.m., Sateesh Chodapuneedi wrote: > > Ship It! Committed to master. Commit 9b288c758b0dbe288b197e4ea85d801fc7cc2217 - Sateesh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apac

Re: Review Request: CLOUDSTACK-1574 : updateResourceCount API is failed saying to specify valida resource type even after parsing the valid resource type

2013-03-07 Thread Devdeep Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9799/#review17539 --- Ship it! Ship It! - Devdeep Singh On March 7, 2013, 11:27 a.m., S

Re: [DISCUSS]Incremental build

2013-03-07 Thread David Nalley
On Thu, Mar 7, 2013 at 6:06 AM, Prasanna Santhanam wrote: > How about we break the cloudstack job on jenkins down to -server, > -core, -utils, -plugins etc? > > Each of the projects takes less than a minute or two to execute. It's > the combined project that takes >20m to run on jenkins delaying b

Re: [PROPOSAL] BVT for CloudStack checkins

2013-03-07 Thread David Nalley
On Thu, Mar 7, 2013 at 12:11 AM, Murali Reddy wrote: > On 06/03/13 11:52 PM, "Kelven Yang" wrote: > >>First +1 on BVT. >> >>Second, should we consider the idea of having a staging area for people to >>check-in? Which is that making master always the stable(reasonable) branch >>for main developmen

RE: [DISCUSS]Incremental build

2013-03-07 Thread Pradeep Soundararajan
I think we can run them in multi thread. Have to test it with multi core machine. Thanks, Pradeep S -Original Message- From: David Nalley [mailto:da...@gnsa.us] Sent: Thursday, March 07, 2013 5:52 PM To: cloudstack-dev@incubator.apache.org Subject: Re: [DISCUSS]Incremental build On Thu

Re: git commit: refs/heads/master - rat check: exclude the WEB-INF directories in check

2013-03-07 Thread David Nalley
There is also a number of non-generated files in WEB-INF directories... can you explain this a bit more? --David On Thu, Mar 7, 2013 at 4:06 AM, wrote: > Updated Branches: > refs/heads/master 7fde29888 -> 81a42b959 > > > rat check: exclude the WEB-INF directories in check > > WEB-INF director

RE: [PROPOSAL] BVT for CloudStack checkins

2013-03-07 Thread Simon Waterhouse
+1 for a staging area - a master branch that passes BVT would be a big improvement. -Original Message- From: Koushik Das [mailto:koushik@citrix.com] Sent: 07 March 2013 06:05 To: cloudstack-dev@incubator.apache.org Subject: RE: [PROPOSAL] BVT for CloudStack checkins +1 to the idea o

Re: git commit: refs/heads/master - rat check: exclude the WEB-INF directories in check

2013-03-07 Thread Prasanna Santhanam
Hmm - I see client/WEB-INF has the web.xml which isn't generated. The mvn rat check doesn't pass on the dev environment because of many of the .txt files within awsapi/. These aren't part of the repo but are generated during the build. So it's hard to tell whether the license check passed or not fr

Re: git commit: refs/heads/master - rat check: exclude the WEB-INF directories in check

2013-03-07 Thread David Nalley
On Thu, Mar 7, 2013 at 7:48 AM, Prasanna Santhanam wrote: > Hmm - I see client/WEB-INF has the web.xml which isn't generated. The > mvn rat check doesn't pass on the dev environment because of many of > the .txt files within awsapi/. These aren't part of the repo but are > generated during the bui

Review Request: CLOUDSTACK-1541 NPE while deleting snapshot

2013-03-07 Thread deepti dohare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9801/ --- Review request for cloudstack, Devdeep Singh and Sateesh Chodapuneedi. Descripti

Re: git commit: refs/heads/master - rat check: exclude the WEB-INF directories in check

2013-03-07 Thread Prasanna Santhanam
On Thu, Mar 07, 2013 at 06:20:44PM +0530, David Nalley wrote: > On Thu, Mar 7, 2013 at 7:48 AM, Prasanna Santhanam wrote: > > Hmm - I see client/WEB-INF has the web.xml which isn't generated. The > > mvn rat check doesn't pass on the dev environment because of many of > > the .txt files within aws

RE: [DISCUSS] Method/Algorithm to gather Health Check states

2013-03-07 Thread Vijay Venkatachalam
Any votes/re-assurances to the thought process from the Networking Universe? My Vote: Method-2 -Vijay -Original Message- From: Vijay Venkatachalam Sent: Wednesday, March 06, 2013 7:58 PM To: cloudstack-dev@incubator.apache.org Subject: [DISCUSS] Method/Algorithm to gather Health Chec

RE: [DISCUSS] Method/Algorithm to gather Health Check states

2013-03-07 Thread Rajesh Battala
I prefer Method-2. Thanks Rajesh Battala > -Original Message- > From: Vijay Venkatachalam [mailto:vijay.venkatacha...@citrix.com] > Sent: Thursday, March 7, 2013 7:17 PM > To: cloudstack-dev@incubator.apache.org > Subject: RE: [DISCUSS] Method/Algorithm to gather Health Check states > >

Re: [DISCUSS] Method/Algorithm to gather Health Check states

2013-03-07 Thread Chip Childers
On Thu, Mar 07, 2013 at 07:17:17PM +0530, Vijay Venkatachalam wrote: > > Any votes/re-assurances to the thought process from the Networking Universe? > > My Vote: Method-2 > > -Vijay So I'm struggling to understand the implications of doing it via method 1 vs method 2. Can you provide what y

Re: Trouble with system VMs on master

2013-03-07 Thread Mike Tutkowski
I think something was messed up in my DevCloud2 VM. I restored it to its initial state and started over with everything and now it works. On Thu, Mar 7, 2013 at 1:15 AM, Rohit Yadav wrote: > On Thu, Mar 7, 2013 at 11:36 AM, Nitin Mehta > wrote: > > Rohit - are you saying that you see it in Ru

Re: [ACS41][Patch Request] CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5

2013-03-07 Thread Chip Childers
On Wed, Mar 06, 2013 at 04:54:40PM -0800, Sheng Yang wrote: > commit 27ba0208f068e60f588d25a5d0ad9c4d6503ad14 > Author: Sheng Yang > Date: Wed Mar 6 16:49:44 2013 -0800 > > CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5 > > Still return a valid object when rever

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

2013-03-07 Thread Chip Childers
On Thu, Mar 07, 2013 at 04:09:57PM +0530, Kishan Kavala wrote: > Joe, > I fixed 1151 in 4.0 branch. Fix has to go into 4.1 and master also. Upgrade > steps also need to be documented. Please provide a patch/commit-sh and confirmation that this was tested against 4.1, and I'll cherry pick or appl

Re: Review Request: CLOUDSTACK-1013 : running cloudstack overwrites default public/private ssh keys

2013-03-07 Thread Chip Childers
On Thu, Mar 07, 2013 at 09:54:12AM +, Harikrishna Patnala wrote: > > > > On March 7, 2013, 9:42 a.m., Prasanna Santhanam wrote: > > > d51e964 on master > > Can you push this on 4.1 also. This could be the problem on both master and > 4.1 Applied to 4.1. > > > - Harikrishna > > > -

RE: [DISCUSS] Method/Algorithm to gather Health Check states

2013-03-07 Thread Vijay Venkatachalam
I have mentioned Pros and Cons, thanks for asking!. Hopefully, it gives a better picture now > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Thursday, March 07, 2013 7:40 PM > To: cloudstack-dev@incubator.apache.org > Subject: Re: [DISCUSS] Method/A

Re: [DISCUSS] Support lifetime

2013-03-07 Thread Chip Childers
On Mon, Mar 04, 2013 at 10:59:49AM -0600, Joe Brockmeier wrote: > On Mon, Mar 4, 2013, at 09:03 AM, David Nalley wrote: > > So software typically has several stages: > > > > Does end of support mean both of these things simultaneously. > > No more bugfixes > > No more security fixes > > > > So we

Re: [DISCUSS] Support lifetime

2013-03-07 Thread Chip Childers
On Mon, Mar 04, 2013 at 10:03:51AM -0500, David Nalley wrote: > So software typically has several stages: > > Does end of support mean both of these things simultaneously. > No more bugfixes > No more security fixes > > So wearing your enterprise software consumer hat - does a support > lifetime

Re: [DISCUSS] Support lifetime

2013-03-07 Thread Chip Childers
On Tue, Mar 05, 2013 at 04:36:10AM -0500, Sebastien Goasguen wrote: > > On Mar 4, 2013, at 11:17 AM, "Musayev, Ilya" wrote: > > > > > > >> -Original Message- > >> From: Musayev, Ilya [mailto:imusa...@webmd.net] > >> Sent: Monday, March 04, 2013 11:05 AM > >> To: cloudstack-dev@incubato

RE: [DISCUSS]Incremental build

2013-03-07 Thread Alex Huang
Chiradeep said he tried it. It basically fails and then you run again and it fails and then run again and one of the runs it will succeed. It fails due to dependencies. Doesn't sound like a good thing to me. --Alex > -Original Message- > From: Pradeep Soundararajan [mailto:pradeep.so

RE: Please Review + Improve: Jira Guidelines

2013-03-07 Thread Alex Huang
Thanks Joe! I like to see a better explanation of the Jira Workflow. Particularly for contributors who has to go through the review board. I think Animesh signed up to write that. --Alex > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent:

RE: Please Review + Improve: Jira Guidelines

2013-03-07 Thread Animesh Chaturvedi
Yes I did I will work on it > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Thursday, March 07, 2013 8:59 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: Please Review + Improve: Jira Guidelines > > Thanks Joe! > > I like to see a better explan

Re: Please Review + Improve: Jira Guidelines

2013-03-07 Thread Joe Brockmeier
On Thu, Mar 7, 2013, at 01:15 AM, Animesh Chaturvedi wrote: > Joe you are awesome, it was my action item from last week Augh. Sorry - didn't mean to step on that, then. Writer's impulse - I found an empty page and decided to put something down while I had some time last night. Best, jzb -- Joe

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

2013-03-07 Thread Sanjay Tripathi
> On Feb. 27, 2013, 5:38 a.m., Nitin Mehta wrote: > > server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java, line 311 > > > > > > What will happen if u get an exception that the size limit exceeds ? > > Do

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

2013-03-07 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9541/ --- (Updated March 7, 2013, 5:47 p.m.) Review request for cloudstack, Devdeep Singh,

Re: Using newer DIY SystemVMs

2013-03-07 Thread Marcus Sorensen
Hmm. I think I was deceived by the /etc/init directory existing. I'm not sure why it's there, but I don't think the template is using upstart. I'm having a hard time reliably recreating the issue, but I think it's related to other reports where the default gateway is missing (I've seen this myself

RE: [DISCUSS]- Dynamic Rendering of tooltip Content from the documentation

2013-03-07 Thread Parth Jagirdar
Is this what you have in mind? http://www.sagehill.net/docbookxsl/CrossRefs.html#OptsGenXref Thanks, .. Parth -Original Message- From: Pranav Saxena Sent: Wednesday, March 06, 2013 8:26 PM To: cloudstack-dev@incubator.apache.org Cc: Brian Federle; Parth Jagirdar; Jessica Tomechak; Davi

RE: Please Review + Improve: Jira Guidelines

2013-03-07 Thread Animesh Chaturvedi
> -Original Message- > From: Joe Brockmeier [mailto:j...@zonker.net] > Sent: Thursday, March 07, 2013 9:38 AM > To: cloudstack-dev@incubator.apache.org > Subject: Re: Please Review + Improve: Jira Guidelines > > On Thu, Mar 7, 2013, at 01:15 AM, Animesh Chaturvedi wrote: > > Joe you are

RE: [DISCUSS]- Dynamic Rendering of tooltip Content from the documentation

2013-03-07 Thread Pranav Saxena
Yes exactly . These set of changes are from the documentation point of view . These Id's and xref tags would be picked up by a py script to convert the content into a json response which can further be rendered by javascript. Thanks for looking into this. Thanks, Pranav -Original Message

RE: [DISCUSS]Incremental build

2013-03-07 Thread Animesh Chaturvedi
> -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Wednesday, March 06, 2013 7:44 PM > To: cloudstack-dev@incubator.apache.org > Subject: RE: [DISCUSS]Incremental build > > +1 to that. My suggestion is to fix maven. Can't be that hard and probably > less eff

RE: [DISCUSS]Incremental build

2013-03-07 Thread Animesh Chaturvedi
> -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Thursday, March 07, 2013 10:30 AM > To: cloudstack-dev@incubator.apache.org > Subject: RE: [DISCUSS]Incremental build > > > > > -Original Message- > > From: Alex Huang [mailto:alex.hu

Re: Using newer DIY SystemVMs

2013-03-07 Thread Marcus Sorensen
I see the qcow2 is now compressed, thanks. You can probably skip putting it in a bz2 now, it's about the same size. On Thu, Mar 7, 2013 at 10:58 AM, Marcus Sorensen wrote: > Hmm. I think I was deceived by the /etc/init directory existing. I'm > not sure why it's there, but I don't think the temp

RE: [PROPOSAL] BVT for CloudStack checkins

2013-03-07 Thread Frank Zhang
> -Original Message- > From: prasanna [mailto:srivatsav.prasa...@gmail.com] On Behalf Of > Prasanna Santhanam > Sent: Thursday, March 07, 2013 2:53 AM > To: cloudstack-dev@incubator.apache.org > Subject: Re: [PROPOSAL] BVT for CloudStack checkins > > On Thu, Mar 07, 2013 at 01:19:19AM +0

Re: Review Request: Remove VNS isolation option from UI

2013-03-07 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9787/#review17561 --- Ship it! builds fine - Chiradeep Vittal On March 6, 2013, 11:39 p

Re: Review Request: Remove VNS isolation option from UI

2013-03-07 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9787/#review17563 --- Ship it! Thanks for the patch! Applied to 4.1: commit 06e7fc1117ce

Re: IPv6 plan - questions

2013-03-07 Thread Sheng Yang
Hi Dave, I am glad it fits your need. That's our target. :) --Sheng On Thu, Mar 7, 2013 at 2:14 AM, Dave Cahill wrote: > Hi Sheng, > > Thanks for the quick reply, that helps a lot. > > My main purpose was to figure out how these changes affect virtual > networking and pluggability. Having read

Re: Review Request: CLOUDSTACK-1541 NPE while deleting snapshot

2013-03-07 Thread deepti dohare
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9801/ --- (Updated March 7, 2013, 7:09 p.m.) Review request for cloudstack, Devdeep Singh,

Question about Primary Storage ID

2013-03-07 Thread Mike Tutkowski
Hi, I created a Primary Storage based on a PreSetup iSCSI target (presetup from XenServer). It works fine, but I notice the ID of the Primary Storage is not a UUID, but rather the name iSCSI_SF (which is the name of the Storage Repository). Is this expected behavior for 4.1 (I didn't try this wi

[ACS41][Patch Request] CLOUDSTACK-1541 NPE while deleting snapshot

2013-03-07 Thread Deepti Dohare
> -Original Message- > From: deepti dohare [mailto:nore...@reviews.apache.org] On Behalf Of > deepti dohare > Sent: Friday, March 08, 2013 12:40 AM > To: Chip Childers; Sateesh Chodapuneedi; Abhinandan Prateek; Devdeep > Singh > Cc: cloudstack; Deepti Dohare > Subject: Re: Review Request:

Re: Review Request: CLOUDSTACK-1541 NPE while deleting snapshot

2013-03-07 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9801/#review17566 --- Ship it! Applied to 4.1: commit e942df93e6915d103f80efa014295d040fb

Re: git commit: refs/heads/4.1 - CLOUDSTACK-1451 Pass randomly generated UUID instead of empty string when create internal account as a part of project creation. Otherwise entityexistsexception would

2013-03-07 Thread Chip Childers
Hi Alena! Thanks for the fix, but can you please follow the workflow for 4.1 agreed to here: http://markmail.org/message/uuffaapkhcothzcj Thanks! On Thu, Mar 07, 2013 at 07:29:53PM +, alena1...@apache.org wrote: > Updated Branches: > refs/heads/4.1 e942df93e -> 45c51d1fd > > > CLOUDSTA

Re: git commit: refs/heads/4.1 - CLOUDSTACK-1451 Pass randomly generated UUID instead of empty string when create internal account as a part of project creation. Otherwise entityexistsexception would

2013-03-07 Thread Alena Prokharchyk
Chip, thanks for the heads up. I'll revert the commit and will submit the review patch. -Alena. On 3/7/13 11:34 AM, "Chip Childers" wrote: >Hi Alena! > >Thanks for the fix, but can you please follow the workflow for 4.1 >agreed to here: http://markmail.org/message/uuffaapkhcothzcj > >Thanks! >

Re: git commit: refs/heads/4.1 - CLOUDSTACK-1451 Pass randomly generated UUID instead of empty string when create internal account as a part of project creation. Otherwise entityexistsexception would

2013-03-07 Thread Chip Childers
So it actually built fine. No reason to have reverted. Go ahead and keep it. The point isn't to be a pain for folks, or specifically to use reviewboard. It's to confirm that 4.1 stays in a solid state, and that we are only moving important patches into that branch. Thanks! On Thu, Mar 07, 201

Re: [DISCUSS]- Dynamic Rendering of tooltip Content from the documentation

2013-03-07 Thread Rohit Yadav
Oh I just saw tooltip and javascript, and I dislike that js implementations for tooltips consume a lot of resources; so let me promote my friend's tooltip framework that is pure css :) https://github.com/chinchang/hint.css Regards. On Thu, Mar 7, 2013 at 11:51 PM, Pranav Saxena wrote: > Yes exac

Re: git commit: refs/heads/4.1 - CLOUDSTACK-1451 Pass randomly generated UUID instead of empty string when create internal account as a part of project creation. Otherwise entityexistsexception would

2013-03-07 Thread Alena Prokharchyk
Chip, totally understand the reasons. I've already reverted the commit and created the RB ticket with you as an assignee: https://reviews.apache.org/r/9807/ Thanks! Alena. On 3/7/13 11:48 AM, "Chip Childers" wrote: >So it actually built fine. No reason to have reverted. Go ahead and >keep it

Re: git commit: refs/heads/4.1 - CLOUDSTACK-1451 Pass randomly generated UUID instead of empty string when create internal account as a part of project creation. Otherwise entityexistsexception would

2013-03-07 Thread Chip Childers
On Thu, Mar 07, 2013 at 12:04:45PM -0800, Alena Prokharchyk wrote: > Chip, totally understand the reasons. I've already reverted the commit and > created the RB ticket with you as an assignee: > > https://reviews.apache.org/r/9807/ It's not published yet. I think you have to click a button still

Re: Using newer DIY SystemVMs

2013-03-07 Thread Rohit Yadav
On Fri, Mar 8, 2013 at 12:08 AM, Marcus Sorensen wrote: > I see the qcow2 is now compressed, thanks. You can probably skip > putting it in a bz2 now, it's about the same size. Had this thought but since we use the cloud-install-sys-tmplt script, it assumes a qcow2.bz2, see: http://incubator.apach

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-07 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9776/#review17569 --- I'm waiting for resolution on the question asked before applying this

Re: Using newer DIY SystemVMs

2013-03-07 Thread Marcus Sorensen
Ok, didn't realize we were going to point people to this particular system vm for production installs, I thought this was just a proof-of-concept for building custom ones. On Thu, Mar 7, 2013 at 12:54 PM, Rohit Yadav wrote: > On Fri, Mar 8, 2013 at 12:08 AM, Marcus Sorensen wrote: >> I see the q

Re: Using newer DIY SystemVMs

2013-03-07 Thread Chip Childers
On Thu, Mar 07, 2013 at 01:07:26PM -0700, Marcus Sorensen wrote: > Ok, didn't realize we were going to point people to this particular > system vm for production installs, I thought this was just a > proof-of-concept for building custom ones. The intent was to be able to actually reproduce our sys

[ACS41][QA] Current Blockers - 3/7

2013-03-07 Thread Sudha Ponnaganti
CLOUDSTACK-1526 Template registration fails in the VMware Setup. Jayapal Reddy 3/5/2013 20:45 4.1.0 CLOUDSTACK-1515 None of the cloudstack packges are marked for upgrade when tried to upgrade from.4.0/4.0.1 to 4.1 David Nalley 3/5/2013 7:43 4

Review Request: CLOUDSTACK-1451 - fix the bug when no more than 1 project can be created

2013-03-07 Thread Alena Prokharchyk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9807/ --- Review request for cloudstack and Chip Childers. Description --- CLOUDS

Re: Review Request: CLOUDSTACK-1451 - fix the bug when no more than 1 project can be created

2013-03-07 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9807/#review17571 --- Ship it! Applied to 4.1: commit feee6bd60a27bd6593ead77fe67eba74881

Re: Review Request: Adding awsapi in the require section of cloud.spec as it resolves issue with EC2 QUERY API Feature.

2013-03-07 Thread Chiradeep Vittal
+1 on being part of default cloudstack install. On 3/7/13 12:07 PM, "Chip Childers" wrote: > >--- >This is an automatically generated e-mail. To reply, visit: >https://reviews.apache.org/r/9776/#review17569 >

GUI question about system VMs

2013-03-07 Thread Mike Tutkowski
Hi, In the Infrastructure tab of the GUI, I notice both a System VMs and a Virtual Routers options (buttons you can click to drill down on the details of these categories). If I look under System VMs, I see my SSVM and my CPVM, but not my Virtual Router. If I look under Virtual Routers, I see th

Question about "Refresh" buttons in the GUI

2013-03-07 Thread Mike Tutkowski
Hi, I've noticed that some panels have a "Refresh" button and others don't (and in at least one case, it's actually called "Fetch latest"). I wonder if we would benefit from making this consistent across the GUI? For example, when I destroy a VM, I don't actually see it disappear from the table

Re: GUI question about system VMs

2013-03-07 Thread Wei ZHOU
Mike, I think it is because the processing of virtual routers (start, stop,reboot,destroy) are different from SSVM and CPVM. You can have a look at the source code, such as StartRouterCmd.java and StartSystemVMCmd.java Wei 2013/3/7 Mike Tutkowski > Hi, > > In the Infrastructure tab of the GUI

Question about Zone-wide Primary Storage (4.2 Feature)

2013-03-07 Thread Mike Tutkowski
Hi, It looks like in 4.2 we can assign Primary Storage to be not only for a single Cluster, but Zone-wide (or specific to a Host). I'm wondering how Zone-wide Primary Storage works for iSCSI. For example, in 4.1 when you create a Primary Storage based on iSCSI and specify, say, a XenServer Clust

Re: GUI question about system VMs

2013-03-07 Thread Mike Tutkowski
Hi, That is probably true, Wei. Do you know, though, if that is something we should separate in the GUI because it makes sense to do so from a user's point of view or are we doing it for our own (developer) "convenience?" Thanks! On Thu, Mar 7, 2013 at 2:44 PM, Wei ZHOU wrote: > Mike, > > I

  1   2   >