Re: Review Request 23915: Fixing some coverity reported issues

2014-07-24 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23915/ --- (Updated July 25, 2014, 6:10 a.m.) Review request for cloudstack, Rajani Karutu

Review Request 23915: Fixing some coverity reported issues

2014-07-24 Thread Damodar Reddy Talakanti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23915/ --- Review request for cloudstack and Rajani Karuturi. Repository: cloudstack-git

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Erik Weber
This is out of my git league, but how do you handle minor versions that way? Assuming 4.8.0 is the latest stable release and HEAD on master. Then you want to release 4.4.1. First of all can you develop bugfixes for 4.4 along the way, when both develop and master HEAD might be hugely different?

Re: Big Switch Network Plug-in for CloudStack update

2014-07-24 Thread Go Chiba
Thanks Kuang, and share BigSwitch's press release about Big Cloud Fabric. http://www.bigswitch.com/press-releases/2014/07/22/big-switch-releases-big-cloud-fabric-the-first-bare-metal-sdn-switching Go Chiba E-mail:go.ch...@gmail.com On Thu, Jul 24, 2014 at 2:53 PM, Kuang-Ching Wang wrote: >

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Mike Tutkowski
I suppose we need a formal VOTE on this? If so, perhaps Sebastien would like to call it. On Thu, Jul 24, 2014 at 10:06 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Great. Well, this seems reasonable to me then. > > If there are no objections or further comments, when can this be

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Mike Tutkowski
Great. Well, this seems reasonable to me then. If there are no objections or further comments, when can this be done and who would do it? Thanks On Thu, Jul 24, 2014 at 10:01 PM, Rajani Karuturi < rajani.karut...@citrix.com> wrote: > On 24-Jul-2014, at 10:25 pm, Mike Tutkowski > wrote: > > >

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Rajani Karuturi
On 24-Jul-2014, at 10:25 pm, Mike Tutkowski wrote: > I believe I agree with these steps. > > A couple questions: > > Is 'master' simply always going to be equal to what's the most recent > version of the code that's in production? I think so. master will always be at the latest release and al

Re: Re: Use "OVSTunnelxxx" but not "cloudbr1" result exception

2014-07-24 Thread Nguyen Anh Tu
Michael, OVSTunnelxxx is automatically created by Ovs controller. The xxx value is generated between VLAN range you set in the beginning. For each guest network you create, Ovs controller will add one OvsTunnel bridge. Thanks, --Tuna On Fri, Jul 25, 2014 at 7:46 AM, Michael Li wrote: > Hi Ng

Re:Re: Use "OVSTunnelxxx" but not "cloudbr1" result exception

2014-07-24 Thread Michael Li
Hi Nguyen, Should I create OVSTunnelxxx ahead like cloudbr0 and cloud0 ? But how to confirm the xxx in OVSTunnelxxx, because it's a dynamic value between the vm creatation. Regards At 2014-07-24 06:08:55, "Nguyen Anh Tu" wrote: >Hi Michael, > >The GRE feature on Cloudstack creates OVSTunne

Re: download template -> delete vhd

2014-07-24 Thread Nitin Mehta
Tomasz - This fix works but you should rather make the fix on the MS code which sends the deletion command rather than on the systemvm side. Moreover if you stop/start ssvm you would loose this jar. You have to rebuild systemvm.iso and put it back on the host (depends on hypervisor in use). My sug

Build failed in Jenkins: cloudstack-4.4-maven-build-noredist #317

2014-07-24 Thread jenkins
See Changes: [Daan Hoogland] Updating pom.xml version numbers for release 4.4.1-SNAPSHOT -- [...truncated 980 lines...] [INFO] Apache CloudStack Engine Storage Volume Com

Re: Getting built artifacts from Jenkins

2014-07-24 Thread Ian Duffy
Great! Thanks guys. On 24 July 2014 19:00, David Nalley wrote: > What Hugo said. > You don't need permission from me. > > On Thu, Jul 24, 2014 at 12:53 PM, Ian Duffy wrote: > > Hi David, > > > > Can I go ahead and make a configuration change to jenkins to allow built > > artifacts to be downlo

RE: [PROPOSAL]remove 4.4-forward

2014-07-24 Thread Animesh Chaturvedi
Yes that’s what we did for 4.3. Merge 4.4-forward into 4.4 > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Thursday, July 24, 2014 7:48 AM > To: dev@cloudstack.apache.org > Subject: Re: [PROPOSAL]remove 4.4-forward > > Yep...what Wei said. :) > >

Re: jclouds support for CloudStack

2014-07-24 Thread Sebastien Goasguen
Great, then you should probably join the jclouds dev list and start testing and contributing there. On Jul 24, 2014, at 6:11 AM, Nguyen Anh Tu wrote: > +1. I'm willing to contribute. > > --Tuna > > > On Thu, Jul 24, 2014 at 1:05 PM, Priyanka Deepala < > priyanka.deepal...@gmail.com> wrote:

RE: Review Request 22863: CLOUDSTACK-6823 : First code drop for Brocade Network plugin to orchestrate Brocade VDX switches for L2 connectivity.

2014-07-24 Thread Ritu Sabharwal
Hi Hugo, Where can I submit the Brocade Plugin Usage document? Thanks & Regards, Ritu S. -Original Message- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Wednesday, July 23, 2014 2:15 AM To: Cc: Ritu Sabharwal Subject: Re: Review Request 22863: CLOUDSTACK-68

Re: Getting built artifacts from Jenkins

2014-07-24 Thread David Nalley
What Hugo said. You don't need permission from me. On Thu, Jul 24, 2014 at 12:53 PM, Ian Duffy wrote: > Hi David, > > Can I go ahead and make a configuration change to jenkins to allow built > artifacts to be downloaded? > > Thanks, > Ian > > > On 23 July 2014 21:11, Ian Duffy wrote: >> >> Hi Al

Re: Getting built artifacts from Jenkins

2014-07-24 Thread Hugo Trippaers
Hey Ian, Go ahead, just make sure only the latest artifact is saved to prevent disk space issues. Hugo Sent from my iPhone > On 24 jul. 2014, at 18:53, Ian Duffy wrote: > > Hi David, > > Can I go ahead and make a configuration change to jenkins to allow built > artifacts to be downloaded?

Re: Getting built artifacts from Jenkins

2014-07-24 Thread Ian Duffy
Hi David, Can I go ahead and make a configuration change to jenkins to allow built artifacts to be downloaded? Thanks, Ian On 23 July 2014 21:11, Ian Duffy wrote: > Hi All, > > For getting the simulator up fast it would be very handy to just pull down > the cloud-client-ui.war from jenkins. >

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Mike Tutkowski
I believe I agree with these steps. A couple questions: Is 'master' simply always going to be equal to what's the most recent version of the code that's in production? Also, would 'develop' be for 4.6 code then and 4.5 code would go directly into RELEASE-4.5? On Thu, Jul 24, 2014 at 12:39 AM,

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Tracy Phillips
If you are on Ubuntu (and who isn't these days... right?), you can install git-flow easily (git-completion is nice as well) aptitude install git-flow git-completion and git-flow command completion is nice https://github.com/bobthecow/git-flow-completion On Thu, Jul 24, 2014 at 12:36 PM, Stephe

Re: [CLOUDSTACK-6181] RBD Support not implemented

2014-07-24 Thread Logan Barfield
I think I've found the cause of the resize failure. When using RBD for primary storage the disk is not allocated before the VM starts. The resize operation runs before the disk exists, and instead of failing completely it just updates the database with the new disk size and returns success. Depl

RE: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Stephen Turner
I agree, every new ticket should be on a new branch. If you have two changesets you might ever want to merge separately, put them on separate branches. -- Stephen Turner -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: 24 July 2014 17:30 To: dev Subject: R

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Tracy Phillips
Yes, that is what he is saying. Its really the git way. Starting a branch with the ticket-id and short description works best. I think this is what you are looking for https://wiki.diasporafoundation.org/Git_workflow On Thu, Jul 24, 2014 at 12:29 PM, Daan Hoogland wrote: > On Thu, Jul 24, 201

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Daan Hoogland
On Thu, Jul 24, 2014 at 5:08 PM, Tracy Phillips wrote: > Good read http://www.draconianoverlord.com/2013/09/07/no-cherry-picking.html he agrees with our thread that every work sould start with creating a branch, doesn't he. I think we need to say that a lot of times more. Start a new branch when

4.4-forward usage

2014-07-24 Thread Daan Hoogland
People (especially test developers), please stop using 4.4-forward and switch to 4.4 If we do a code freeze for 4.4.1 make a new branch per fix, the model of a -forward branch is fundamentally broken and has cost me a few hours of merge trouble over the past two months. thanks, -- Daan

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Tracy Phillips
The best thread I have read in awhile... Good read http://www.draconianoverlord.com/2013/09/07/no-cherry-picking.html On Thu, Jul 24, 2014 at 8:06 AM, Daan Hoogland wrote: > On Thu, Jul 24, 2014 at 1:43 PM, Leo Simons > wrote: > > lsimons > > > has access > > -- > Daan >

Re: [PROPOSAL]remove 4.4-forward

2014-07-24 Thread Tracy Phillips
git-flow with master always golden would be a good method. Right now the development model just isn't working. fwiw, I am not a developer, just a tinkerer with git and some bash and a lurker :) On Thu, Jul 24, 2014 at 10:47 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Yep...what

Re: Review Request 23879: CLOUDSTACK-7127: Fix for addRegion failure, avoiding regionid 1 while creating new region through test case

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23879/#review48655 --- Commit ca59f01602823a6d6fe84233f3a3d3ea499efa06 in cloudstack's bran

Re: Review Request 23879: CLOUDSTACK-7127: Fix for addRegion failure, avoiding regionid 1 while creating new region through test case

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23879/#review48654 --- Commit 59c77fbaa4c07e2de9578369e69f47d0e71039b8 in cloudstack's bran

Re: Review Request 23882: CLOUDSTACK-7125: Fixed test_blocker_bugs.py, added code to wait for snapshots to be in 'BackedUp' state

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23882/#review48653 --- Commit 4395308bd8b469fb02fc71d1e22f25c7d89b6dff in cloudstack's bran

Re: Review Request 23882: CLOUDSTACK-7125: Fixed test_blocker_bugs.py, added code to wait for snapshots to be in 'BackedUp' state

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23882/#review48652 --- Commit 49005b324f18e19166bcbb801bcd07609e44d8db in cloudstack's bran

Re: [PROPOSAL]remove 4.4-forward

2014-07-24 Thread Mike Tutkowski
Yep...what Wei said. :) On Thu, Jul 24, 2014 at 8:20 AM, Wei ZHOU wrote: > yes. I think 4.4-forward can be merged into 4.4 branch after 4.4.0 release, > then delete it. > > > 2014-07-24 15:08 GMT+02:00 Daan Hoogland : > > > As we will probably need a 4.4.1 I will spend some tine cherry picking

Re: Review Request 23885: CLOUDSTACK-7178: Correcting imports in test_escalations* test cases

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23885/#review48651 --- Commit 4e45b296891f71fc7991dc507dc7ae9cec6f5496 in cloudstack's bran

Re: Review Request 23885: CLOUDSTACK-7178: Correcting imports in test_escalations* test cases

2014-07-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23885/#review48650 --- Commit 78a6d557f86db354a073090fc92ea693ee235e1f in cloudstack's bran

Re: Review Request 23879: CLOUDSTACK-7127: Fix for addRegion failure, avoiding regionid 1 while creating new region through test case

2014-07-24 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23879/ --- (Updated July 24, 2014, 8:14 p.m.) Review request for cloudstack and Girish Shi

Re: Review Request 23882: CLOUDSTACK-7125: Fixed test_blocker_bugs.py, added code to wait for snapshots to be in 'BackedUp' state

2014-07-24 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23882/ --- (Updated July 24, 2014, 8:13 p.m.) Review request for cloudstack and Girish Shi

Re: [PROPOSAL]remove 4.4-forward

2014-07-24 Thread Wei ZHOU
yes. I think 4.4-forward can be merged into 4.4 branch after 4.4.0 release, then delete it. 2014-07-24 15:08 GMT+02:00 Daan Hoogland : > As we will probably need a 4.4.1 I will spend some tine cherry picking > through 4.4-forward to get any changes we will need from there (plus > test cases) > >

Re: Review Request 23856: Fix VPC distributed routing integration tests

2014-07-24 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23856/ --- (Updated July 24, 2014, 1:46 p.m.) Review request for cloudstack, daan Hoogland

Re: Review Request 23856: Fix VPC distributed routing integration tests

2014-07-24 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23856/ --- (Updated July 24, 2014, 1:44 p.m.) Review request for cloudstack, daan Hoogland

[PROPOSAL]remove 4.4-forward

2014-07-24 Thread Daan Hoogland
As we will probably need a 4.4.1 I will spend some tine cherry picking through 4.4-forward to get any changes we will need from there (plus test cases) I will remove 4.4-forward after I am done. As per the git flow way of working any fixes on 4.4 will be done on a per fix branch of off 4.4. plea

Re: Review Request 23856: Use local test configuration to enable testing of VPC distributed routing

2014-07-24 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23856/ --- (Updated July 24, 2014, 12:54 p.m.) Review request for cloudstack, daan Hooglan

Re: Review Request 23856: Use local test configuration to enable testing of VPC distributed routing

2014-07-24 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23856/ --- (Updated July 24, 2014, 12:51 p.m.) Review request for cloudstack, daan Hooglan

Cloudstack UI - Workflow

2014-07-24 Thread Prakash Rao Banuka
Hi All: I would like to know the sequence the .js files called in cloudstack. This would be help me in quickly troubleshooting the steps. Example: a. When enter the url in the browser (http://localhost:8080/client) we get login screen. (I would like to the sequence in which the .js files

Re: release schedule

2014-07-24 Thread Sebastien Goasguen
On Jul 24, 2014, at 6:29 AM, Daan Hoogland wrote: > H Sebastien, > > With 4.4.0 out and to be announced sometime today or tomorrow, Do you > still plan to bring out 4.3.1? As per the gitflow discussion we will > need to bring out a 4.4.1 by hand as well. A new way of working has to > start from

Review Request 23885: CLOUDSTACK-7178: Correcting imports in test_escalations* test cases

2014-07-24 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23885/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7178 h

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Daan Hoogland
On Thu, Jul 24, 2014 at 1:43 PM, Leo Simons wrote: > lsimons has access -- Daan

Re: Test Failures

2014-07-24 Thread Daan Hoogland
Priyanka, I just ran this on a windows machine in cygwin with latest master. It passed the server package without problems. I am afraid you are on your own at this point unless you can give more data. On Thu, Jul 24, 2014 at 11:23 AM, Daan Hoogland wrote: > on the latest master? > > I run on a m

Review Request 23884: Fixed CLOUDSTACK-5212: [UI]Need Support for the LXC for the Report sockets

2014-07-24 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23884/ --- Review request for cloudstack and Jessica Wang. Bugs: CLOUDSTACK-5212 https

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Leo Simons
On Jul 24, 2014, at 1:19 PM, Daan Hoogland wrote: > Leo, are you prepared to write up a how-to cwiki page? Sure, though it would be good to get a few provisional +1s before spending a lot of time drawing pretty pictures :) Also, I don’t seem to have edit permissions on https://cwiki.apache.org

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Rajani Karuturi
It makes sense to me now. +1 from me one minor thing i would like to add is git-flow doesn’t force you to its naming convention. You can choose any and stick to it. But, it makes sense to use the defaults. ~Rajani On 24-Jul-2014, at 2:52 pm, Leo Simons wrote: > On Jul 24, 2014, at 10:45 AM

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Daan Hoogland
now we are kind of set back by this unfortunate result of Leo's work. new proposal: We start using git-flow for 4.5/5.0 exclusively. as of then we will use the workflows as describe by him [1] 4.4.x will still be released by manual RM-cherry-picking (yes volunteer dragging his feet) Leo, are you

Re: Review Request 23395: CLOUDSTACK-5663 :- Handled the NULL check for network's cidr field to avoid NPE for createNetwork and listNetwork

2014-07-24 Thread Girish Chaudhari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23395/ --- (Updated July 24, 2014, 10:45 a.m.) Review request for cloudstack, daan Hooglan

release schedule

2014-07-24 Thread Daan Hoogland
H Sebastien, With 4.4.0 out and to be announced sometime today or tomorrow, Do you still plan to bring out 4.3.1? As per the gitflow discussion we will need to bring out a 4.4.1 by hand as well. A new way of working has to start from master, as Leo convincingly described. I think we need a 4.4.1 b

Re: 4.4 use gre tunnel failed

2014-07-24 Thread Nguyen Anh Tu
Hi, Yes, OVS controller auto create OVSTunnelxxx. Please use ovs-vsctl ultility to check if OVSTunnel316 is exists. Use this guide: https://cwiki.apache.org/confluence/display/CLOUDSTACK/KVM+with+OpenVSwitch Thanks, --Tuna On Wed, Jul 23, 2014 at 3:02 PM, cloud wrote: > Hi all, When I use 4

Re: jclouds support for CloudStack

2014-07-24 Thread Nguyen Anh Tu
+1. I'm willing to contribute. --Tuna On Thu, Jul 24, 2014 at 1:05 PM, Priyanka Deepala < priyanka.deepal...@gmail.com> wrote: > count me jclouds integration for more recent API features > > Regards > Priyanka > > > On Wed, Jul 23, 2014 at 9:07 PM, Sebastien Goasguen > wrote: > > > > > On Jul

Re: [DISCUSS] Acquire New Ip from a different range on shared networks

2014-07-24 Thread Murali Reddy
On 23/07/14 4:32 AM, "Silvano Nogueira Buback" wrote: >@All >When I acquire new IP in shared network the same IP of vms are given >To the users. When you create a shared network, a range of IP address need to be given. IP addresses are allocated from this range for the VM's in the shared net

Re: GSoC update

2014-07-24 Thread Nguyen Anh Tu
Well, really great job Darren & also Ian. Congratulations!!! --Tuna On Thu, Jul 24, 2014 at 1:39 AM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Nice job! :) > > > On Wed, Jul 23, 2014 at 12:36 PM, Sebastien Goasguen > wrote: > > > Darren's work (and Ian from last year) featured on

Re: Use "OVSTunnelxxx" but not "cloudbr1" result exception

2014-07-24 Thread Nguyen Anh Tu
Hi Michael, The GRE feature on Cloudstack creates OVSTunnelxxx bridges with OpenvSwitch to serve tunnel between vms, not cloudbr. Please use ovs-vsctl utility to check if OVSTunnel460 is exist. Remember to blacklist module bridge. This might help: https://cwiki.apache.org/confluence/display/CLOUD

Re: Review Request 23856: Use local test configuration to enable testing of VPC distributed routing

2014-07-24 Thread Miguel Ferreira
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23856/ --- (Updated July 24, 2014, 10:04 a.m.) Review request for cloudstack, daan Hooglan

Re: Review Request 23808: listCapacity API missing types for certain zones

2014-07-24 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23808/#review48641 --- Ship it! commit da55aff64f1b8255daa7999b91672bd2195bb76b - Kishan

Review Request 23882: CLOUDSTACK-7125: Fixed test_blocker_bugs.py, added code to wait for snapshots to be in 'BackedUp' state

2014-07-24 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23882/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUDSTACK-7125

Re: Test Failures

2014-07-24 Thread Daan Hoogland
on the latest master? I run on a mac and it passes. I'll give it a try on windows later. On Thu, Jul 24, 2014 at 11:21 AM, Priyanka Deepala wrote: > Ya I run it several times but it is displaying the same error.I'm using > Cygwin terminal in Windows7 > > > On Thu, Jul 24, 2014 at 2:22 PM, Daan H

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Leo Simons
On Jul 24, 2014, at 10:45 AM, Daan Hoogland wrote: > Any advice on our procedure from this, Leo? Yes, to follow all the git-flow defaults [1]. * goal is for master to become stable * new develop which starts from master * create with `git flow init` * ‘abandon’ 4.4 / 4.4-forward * anything i

Re: Test Failures

2014-07-24 Thread Priyanka Deepala
Ya I run it several times but it is displaying the same error.I'm using Cygwin terminal in Windows7 On Thu, Jul 24, 2014 at 2:22 PM, Daan Hoogland wrote: > You can cut and paste it to the java stacktrace console in eclipse to > query the stack for the origin of the error. To determine whether i

Re: download template -> delete vhd

2014-07-24 Thread Tomasz Zięba
Hello, https://github.com/apache/cloudstack/blob/master/services/secondary-storage/server/src/org/apache/cloudstack/storage/template/UploadManagerImpl.java We analyzed that the problem is in this line: if (cmd.getType() == Upload.Type.VOLUME) { This line is always true because cmd.getTy

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Rajani Karuturi
Hi Daan/Erik, I am not asking about the next minor release or hot fix. But, about the process involved after cutting the release branch and before the release is votes. i.e.) making it stable and bug/blocker free. currently, we do that through commit to 4.5-forward and cherry-pick requests. Are

RE: Failed add KVM agent to CS with latest master

2014-07-24 Thread Santhosh Edukulla
Agree. I decided to split it into multiple smaller ones going ahead. Instead of revert, may be we would have identified root cause and helped to fix the issue. Santhosh From: Daan Hoogland [daan.hoogl...@gmail.com] Sent: Thursday, July 24, 2014 4:57 AM

Re: Failed add KVM agent to CS with latest master

2014-07-24 Thread Daan Hoogland
On Thu, Jul 24, 2014 at 12:12 AM, Amogh Vasekar wrote: > 4523490d44160b054de9e943f72db1d0ce06054a Another big coverity fix reverted Santhosh. Could you split this one as well. A shame if all that work is getting lost this way. -- Daan

Re: Test Failures

2014-07-24 Thread Daan Hoogland
You can cut and paste it to the java stacktrace console in eclipse to query the stack for the origin of the error. To determine whether it is an error in the test or in the tested code you would need to have your version of the checked out code. Did you run it repeatedly and bumped into the same e

Re: Review Request 23805: Added "state" parameter to the "listPublicIpAddresses" API call

2014-07-24 Thread Ilia Shakitko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23805/ --- (Updated July 24, 2014, 8:47 a.m.) Review request for cloudstack, Alena Prokhar

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Daan Hoogland
Any advice on our procedure from this, Leo? On Thu, Jul 24, 2014 at 10:39 AM, Leo Simons wrote: > On Jul 24, 2014, at 8:39 AM, Rajani Karuturi > wrote: >> here is what i propose: >> >> 1. rename 'master' to 'develop’ >> 2. branch a new 'master' from '4.4’ >> 3. branch 'RELEASE-4.5' from the dev

Re: [DISCUSS][PROPOSAL] git workflow

2014-07-24 Thread Leo Simons
On Jul 24, 2014, at 8:39 AM, Rajani Karuturi wrote: > here is what i propose: > > 1. rename 'master' to 'develop’ > 2. branch a new 'master' from '4.4’ > 3. branch 'RELEASE-4.5' from the develop > 4. merge 'RELEASE-4.5' to master once the release voting is done. I like this conceptually; I’m not

Re: Review Request 23735: Fix deployment of data center with marvin

2014-07-24 Thread Miguel Ferreira
> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/marvinLog.py, line 168 > > > > > > Make it more abstract and see if is not aware of cfg(log_cfg), i mean > > pass the logfile path, as si

Re: Review Request 23735: Fix deployment of data center with marvin

2014-07-24 Thread Miguel Ferreira
> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/marvinLog.py, line 168 > > > > > > Make it more abstract and see if is not aware of cfg(log_cfg), i mean > > pass the logfile path, as si

Review Request 23879: CLOUDSTACK-7127: Fix for addRegion failure, avoiding regionid 1 while creating new region through test case

2014-07-24 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23879/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUDSTACK-7127

Re: Test Failures

2014-07-24 Thread Priyanka Deepala
ya I got the file and xml document also. The content in the file is --- Test set: com.cloud.agent.resource.virtualnetwork.VirtualRoutingResourceTest --