Review Request: format TestCaseExecuteEngine

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12051/ --- Review request for cloudstack and Sebastien Goasguen. Description --- form

Review Request: format asyncJobMgr

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12052/ --- Review request for cloudstack and Sebastien Goasguen. Description --- form

Review Request: format cloudstackException

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12053/ --- Review request for cloudstack and Sebastien Goasguen. Description --- form

Review Request: format cloudstackTestCase

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12054/ --- Review request for cloudstack and Sebastien Goasguen. Description --- form

Review Request: format cloudstackTestClient

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12055/ --- Review request for cloudstack and Sebastien Goasguen. Description --- form

Re: Review Request: format TestCaseExecuteEngine

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12051/#review22300 --- Ship it! applied to master 3fdcf1836c22567746c34a42db60505a42fbf6bc

Re: Review Request: format asyncJobMgr

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12052/#review22301 --- Ship it! applied to master 34fc209419dc0f9b6d347e7a33371bc0941069b6

Re: Review Request: format cloudstackException

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12053/#review22302 --- Ship it! applied to master 3f4204b8bc5f31196cbd7c8ff8b343eb7e6ce88f

Re: Review Request: format cloudstackTestCase

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12054/#review22303 --- Ship it! applied to master d53f1c0946ca05504cff4f4af54a6b40413da06b

Re: Review Request: format cloudstackTestClient

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12055/#review22305 --- This was already patched in https://reviews.apache.org/r/12051/ You

Re: Review Request: format TestCaseExecuteEngine

2013-06-23 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12051/ --- (Updated June 23, 2013, 3:54 p.m.) Review request for cloudstack and Sebastien

Re: Review Request: format cloudstackTestClient

2013-06-23 Thread daan Hoogland
> On June 23, 2013, 3:35 p.m., Sebastien Goasguen wrote: > > This was already patched in https://reviews.apache.org/r/12051/ > > You are missing testCaseExecuteEngine, I think you attached the wrong patch > > to 12051. > > So now we are only missing the testCaseExecuteEngine You are right, I fi

Re: Review Request: format TestCaseExecuteEngine

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12051/#review22307 --- Ship it! ok applied 3e430ccbf07e80656e130fbd42afd5a5fa955af5 - Seb

Re: Review Request: format cloudstackTestClient

2013-06-23 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12055/#review22308 --- I guess you can discard this one... since I applied the patch via 12

[OFFLINE] July 1 - July 7

2013-06-23 Thread Joe Brockmeier
Taking vacation first week of July and will offline (for any form of work purposes) from July 1st through 7th. Best, jzb -- Joe Brockmeier j...@zonker.net Twitter: @jzb http://www.dissociatedpress.net/

Re: [DISCUSS] Issue with cloudmonkey-4.1.0-0 on pypi

2013-06-23 Thread David Nalley
I've created this repo based on Rohit's work to preserve history. The repo is currently read only. I'd like several reviews of this repo before I open it up for writes. https://git-wip-us.apache.org/repos/asf/cloudstack-cloudmonkey.git --David On Wed, Jun 19, 2013 at 10:27 PM, David Nalley wro

Re: git commit: Initial commit with LICENSE, NOTICE, and README.

2013-06-23 Thread David Nalley
Uhhh - can we start with a fresh license and notice file. There are clearly not some of the things mentioned in LICENSE/NOTICE in this repo. --David On Sun, Jun 23, 2013 at 4:50 PM, wrote: > Updated Branches: > refs/heads/master [created] 6abd98356 > > > Initial commit with LICENSE, NOTICE, a

Re: git commit: Initial commit with LICENSE, NOTICE, and README.

2013-06-23 Thread Joe Brockmeier
On Sun, Jun 23, 2013, at 04:01 PM, David Nalley wrote: > Uhhh - can we start with a fresh license and notice file. > There are clearly not some of the things mentioned in LICENSE/NOTICE > in this repo. NOTICE should have been fine, but I corrected the LICENSE file. Best, jzb -- Joe Brockmeier

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Hiroaki KAWAI
Current patch/systemvm/debian is based on debian squeeze, which kernel is 2.6.32-5-686-bigmem. In that system vm, cloud-early-config silently fails : /etc/init.d/cloud-early-config: line 109: /dev/vport0p1: No such file or directory So I've upgraded to wheezy (which includes virtio-console.ko) #

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Marcus Sorensen
Yes, I brought this up previously and was told that we require systemvm upgrade between many versions and that it is nothing new. There are other features in 4.2 as well that require the new system vm, so it will be made known in the release notes and upgrade instructions. On Jun 23, 2013 8:33 PM,

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Marcus Sorensen
I personally thought it had been publicized pretty well on various threads that there is a new system vm for master/4.2, but if you were unaware of it, do you think more needs to be done to call it out and make it known to the devs working on it? On Jun 23, 2013 8:33 PM, "Hiroaki KAWAI" wrote: >

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Hiroaki KAWAI
No I don't think we need to make such effort (sending emails) for devs, I think we should fix the code itself (and comments in the codes) because we're devs. (2013/06/24 12:20), Marcus Sorensen wrote: I personally thought it had been publicized pretty well on various threads that there is a new

Re: [DISCUSS] Issue with cloudmonkey-4.1.0-0 on pypi

2013-06-23 Thread Rohit Yadav
On Mon, Jun 24, 2013 at 2:20 AM, David Nalley wrote: > I've created this repo based on Rohit's work to preserve history. > > Great, thanks David. Maybe fix the repo description to "Apache CloudStack CLI" etc. > The repo is currently read only. I'd like several reviews of this repo > before I op

Re: [VOTE] Update by-laws to add section for non-technical decision making (Was: Re: [RESULTS][SUMMARY][DISCUSS][VOTE] List CloudStack related books on the website)

2013-06-23 Thread Mathias Mullins
Noah, I agree that there needs to be a delineation. Here's my option on wording describing what is non-technical: +3.4.2. Non-Technical Decisions +Non-technical decisions should normally be made by the entire community using +discussion-lead consensus-building, and not through formal voting. +

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Prasanna Santhanam
Kawai-san, commit 009da930580fba039b4b8a7532a8a6809d00ed02 + patches/systemvm/debian/buildsystemvm.sh The file under there is not used to build systemvm templates as recommended anymore. The new way to build systemVMs is using veewee-vagrant definitions [1] and the definition is based on a Debian

Re: [DISCUSS] Issue with cloudmonkey-4.1.0-0 on pypi

2013-06-23 Thread Prasanna Santhanam
On Mon, Jun 24, 2013 at 09:17:16AM +0530, Rohit Yadav wrote: > On Mon, Jun 24, 2013 at 2:20 AM, David Nalley wrote: > > > I've created this repo based on Rohit's work to preserve history. > > > > > Great, thanks David. Maybe fix the repo description to "Apache CloudStack > CLI" etc. Just curious

Re: git commit: Initial commit with LICENSE, NOTICE, and README.

2013-06-23 Thread Prasanna Santhanam
On Sun, Jun 23, 2013 at 04:14:47PM -0500, Joe Brockmeier wrote: > On Sun, Jun 23, 2013, at 04:01 PM, David Nalley wrote: > > Uhhh - can we start with a fresh license and notice file. > > There are clearly not some of the things mentioned in LICENSE/NOTICE > > in this repo. > > NOTICE should have b

Re: [DISCUSS] getting rid of KVM patchdisk

2013-06-23 Thread Marcus Sorensen
So what is the proposed fix? Just to find some way to warn when the user isn't using the correct systemvm template (the new one required for 4.2 has been based on Wheezy for awhile), other than nothing working and them having to look in the logs? This is probably a good idea, we can perhaps have a

Be In Our CloudStack Videos: See You At Cloud Collab 2013

2013-06-23 Thread Jessica Tomechak
If you’re attending the CloudStack Collaboration Conference this week in Santa Clara, you will see videographer Gregg Witkin shooting footage for CloudStack videos. We’ll be getting content for both a “Top 10 Coolest Clouds” video and also a “Conference Highlights” reel. Please note, this is separ

Re: [DISCUSS] Issue with cloudmonkey-4.1.0-0 on pypi

2013-06-23 Thread David Nalley
On Mon, Jun 24, 2013 at 12:40 AM, Prasanna Santhanam wrote: > On Mon, Jun 24, 2013 at 09:17:16AM +0530, Rohit Yadav wrote: >> On Mon, Jun 24, 2013 at 2:20 AM, David Nalley wrote: >> >> > I've created this repo based on Rohit's work to preserve history. >> > >> > >> Great, thanks David. Maybe fix

Re: Review Request: Fix for CLOUDSTACK-2181: Scale down is allowed, which is not expected

2013-06-23 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11936/#review22317 --- Commit 1eb744fa1600f9400e407c6e3615da40b18758c1 in branch refs/heads

Re: Review Request: Fix for CLOUDSTACK-2916, CLOUDSTACK-2457: validation for weight based configuration parameters

2013-06-23 Thread Nitin Mehta
> On June 20, 2013, 10:12 a.m., Nitin Mehta wrote: > > Ship It! Please rebase Nitins-MacBook-Air:cloudstack nitinmehta$ git apply ../0001-CLOUDSTACK-2916-admin-is-not-able-to-login-through-U.patch error: patch failed: server/src/com/cloud/configuration/ConfigurationManagerImpl.java:336 error