Build failed in Jenkins: build-master-slowbuild #2933

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Build failed in Jenkins: build-master-slowbuild #2932

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Build failed in Jenkins: build-master-slowbuild #2931

2016-01-07 Thread jenkins
See -- [...truncated 28733 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Re: Minor releases!

2016-01-07 Thread Marcus
A few things to note: 1. I repeated this until I felt bad about harping on it, but we were seeing new functionality slip into minors all the time. The idea that 4.4.x -> 4.4.y upgrade was safer than 4.4.x -> 4.5.0 (just made up versions as an example) is unfortunately wrong. 2. I agree that large

Build failed in Jenkins: build-master-slowbuild #2930

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

Re: Minor releases!

2016-01-07 Thread Wei ZHOU
2016-01-07 15:24 GMT+01:00 Rene Moser : > Hi Remi > > On 01/07/2016 02:28 PM, Remi Bergsma wrote: > > I simply don’t understand why you need lots and lots of minor versions. > I do understand you need a stable cloud, and that’s exactly what we’re > achieving here. > > > > We changed our way of wor

[GitHub] cloudstack pull request: CLOUDSTACK-9216: Fix DHCP lease removal l...

2016-01-07 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1318 CLOUDSTACK-9216: Fix DHCP lease removal logic on VR It now releases all DHCP leases when requested. Will post Integration test results once finished. You can merge this pull request

Build failed in Jenkins: build-master-slowbuild #2929

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9104: VM naming convention in ...

2016-01-07 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1302#discussion_r49103580 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2032,11 +2032,20 @@ int getReservedC

[GitHub] cloudstack pull request: Quota: findbug fixes

2016-01-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1289#issuecomment-169753995 Would you mind changing the comments over attributes “details” and “isDynamic” to proper java doc style? Additionally, what about changing them to

[GitHub] cloudstack pull request: Quota: findbug fixes

2016-01-07 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1289#discussion_r49101315 --- Diff: framework/quota/src/org/apache/cloudstack/quota/vo/ServiceOfferingVO.java --- @@ -310,14 +269,12 @@ public String getDeploymentPlanne

[GitHub] cloudstack pull request: [4.7] FIX Site2SiteVPN on redundant VPC

2016-01-07 Thread michaelandersen
Github user michaelandersen commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1276#discussion_r49099622 --- Diff: test/integration/smoke/test_vpc_vpn.py --- @@ -494,9 +526,411 @@ def create_natrule(self, vpc, vm, public_port, private_port, public_ip

Re: Minor releases!

2016-01-07 Thread Rene Moser
On 01/07/2016 05:04 PM, sebgoa wrote: > Yet folks (like Rene) may like a pattern of just minor and very infrequent > major. While folks like Remi want continuous deployment. > > So at the cost of sounding a bit "fatherly" we indeed need to discuss this a > bit. I mentioned it after 4.6, and com

Joining Schubert Philis

2016-01-07 Thread Boris Roman Schrijver
Hi all, As of the 1st of January I joined the Cloud Team at Schuberg Philis. Therefore I’d to cease contributing to Cloudstack in the name of PCextreme. You can reach me at bschrij...@schubergphilis.com! Boris Roman Schrijver Mission Critical Engineer +31 20 7506500 +31 20 7506500 Boeingavenue

Re: Minor releases!

2016-01-07 Thread Rene Moser
On 01/07/2016 05:27 PM, Remi Bergsma wrote: > Anyway, I cannot and don’t want to convince you. We want something different > and that is fine. What I do want to know is what others want. Because if the > majority wants what you are asking for, we should do that. It is not my decision, it is a

Re: Minor releases!

2016-01-07 Thread sebgoa
On Jan 7, 2016, at 5:27 PM, Remi Bergsma wrote: > > On 07/01/16 17:22, "Rene Moser" wrote: >> No, it is not the pace. You can do as many major as often as you want >> but if one uses this major, how long will it get minors? We have no clue. >> >> I understand your point completely while my ar

Re: Minor releases!

2016-01-07 Thread sebgoa
see inline On Jan 7, 2016, at 3:39 PM, Nux! wrote: > Hi, > > >> So, yes, monthly releases can be done and the quality is better than before. >> Actually, I think we should go much faster. Whenever a PR is merged that >> fixes >> your issue, it should be possible to deploy it right away. It’s a

Re: Minor releases!

2016-01-07 Thread Remi Bergsma
Heaven is closer than you might think. 4.7 did not require a systemvm template change, so we reused the existing one. No impact in upgrading. We need to change it only when needed, it’s that simple. A separate project and version is probably best way forward. Regards, Remi On 07/01/16 15:

Re: Minor releases!

2016-01-07 Thread Wido den Hollander
On 01/07/2016 03:53 PM, Erik Weber wrote: > On Thu, Jan 7, 2016 at 3:50 PM, Wido den Hollander wrote: > >> >> >> On 01/07/2016 03:47 PM, Nux! wrote: And that is indeed it. I'm not keen on doing point releases either. >>> Simplify upgrade paths, ditch the fact that a VR has to be upgra

Re: Minor releases!

2016-01-07 Thread Nux!
> And that is indeed it. I'm not keen on doing point releases either. > Simplify upgrade paths, ditch the fact that a VR has to be upgraded > every time, etc, etc. That'd be heaven. I hate the VR upgrades! > > We could also call 4.8 simply 4.7.2. It's just a number :) +1, perhaps switch to ver

Re: Minor releases!

2016-01-07 Thread Nux!
Hi, > So, yes, monthly releases can be done and the quality is better than before. > Actually, I think we should go much faster. Whenever a PR is merged that fixes > your issue, it should be possible to deploy it right away. It’s a change in > mindset. This actually sounds very interesting; havi

Re: Minor releases!

2016-01-07 Thread Wido den Hollander
On 01/07/2016 03:35 PM, Remi Bergsma wrote: > > We released: > 4.6.0 > 4.6.1 > 4.6.2 > 4.7.0 > (4.7.1) > (4.8.0) > > We _do_ release minor releases, and every month one with new features. > > To be clear: > Any branch (be it 4.5, 4.6, 4.7) can have as many minor releases as people > want, unt

Re: Minor releases!

2016-01-07 Thread Remi Bergsma
We released: 4.6.0 4.6.1 4.6.2 4.7.0 (4.7.1) (4.8.0) We _do_ release minor releases, and every month one with new features. To be clear: Any branch (be it 4.5, 4.6, 4.7) can have as many minor releases as people want, until the end of time. Just step up and release it. Regards, Remi >We a

Re: Minor releases!

2016-01-07 Thread Remi Bergsma
Hi René, all, I simply don’t understand why you need lots and lots of minor versions. I do understand you need a stable cloud, and that’s exactly what we’re achieving here. We changed our way of working from 4.6 on. Before that, it took _a long_ time to release new versions (be it major, minor

Build failed in Jenkins: build-master-slowbuild #2927

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9213 - As a user I want to be ...

2016-01-07 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1311#issuecomment-169631870 LGTM, verified it working properly: ![screen shot 2016-01-06 at 14 51 17](https://cloud.githubusercontent.com/assets/1630096/12169101/c3583e0a-b536-11e5

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9214 , detect w...

2016-01-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1315#issuecomment-169611566 looks good on first sight, buiding an env to test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

2016-01-07 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169609249 @wilderrodrigues as @koushik-das says, the diff for engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java does not show

Build failed in Jenkins: build-master-slowbuild #2926

2016-01-07 Thread jenkins
See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack

[GitHub] cloudstack pull request: CLOUDSTACK-9213 - As a user I want to be ...

2016-01-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1311#issuecomment-169600413 The test I missed: ``` [root@cs1 integration]# less /tmp//MarvinLogs/test_routers_network_ops_6WMMRL/results.txt Test redundant router inte

[GitHub] cloudstack pull request: CLOUDSTACK-9213 - As a user I want to be ...

2016-01-07 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1311#issuecomment-169592688 Ping @remibergsma @DaanHoogland @borisroman I'm now running the ```test_routers_network_ops``` because I noticed it was not on the list after the b