[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

[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

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: [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

[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: 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

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

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

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 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 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 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 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 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 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 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 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

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: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

[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

[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: 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: 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

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-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

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

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 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 #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

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 #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