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

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

Bug on delete snapshot function / CLOUDSTACK-9297

2016-03-02 Thread Özhan Rüzgar Karaman
Hi; 2 weeks ago we noticed a bug on delete snapshot function/api for ACS 4.8. We have created a jira ticket for this bug. https://issues.apache.org/jira/browse/CLOUDSTACK-9297 issue looks like its similar bug which was mentioned on CLOUDSTACK-8845 . But because its on other function we have creat

Re: Bug on delete snapshot function / CLOUDSTACK-9297

2016-03-02 Thread Wei ZHOU
Hi Ozhan, please check the comments of the ticket. -Wei 2016-03-02 10:22 GMT+01:00 Özhan Rüzgar Karaman : > Hi; > 2 weeks ago we noticed a bug on delete snapshot function/api for ACS 4.8. > We have created a jira ticket for this bug. > > https://issues.apache.org/jira/browse/CLOUDSTACK-9297 > >

Build failed in Jenkins: build-systemvm64-master #765

2016-03-02 Thread jenkins
See -- [...truncated 3395 lines...] 100% [Packages 28.5 MB] 100% [Working] 100% [Packages 1,777 kB] 100% [Working]

[GitHub] cloudstack-docs-admin pull request: Be explicit in regards to Open...

2016-03-02 Thread ntavares
GitHub user ntavares opened a pull request: https://github.com/apache/cloudstack-docs-admin/pull/35 Be explicit in regards to OpenSwan ipsec.secrets rule. According to: http://linux.die.net/man/5/ipsec.secrets A preshared secret is most conveniently represented as a sequ

Re: PR validation using proposed CI.

2016-03-02 Thread Bharat Kumar
Hi Will, I was not having email access since few days, so could’ t reply in time. Yes, The idea is to run this on each PR. Once i am done with this, it should be easy to replicate this locally if needed. Thanks, Bharat. > On 27-Feb-2016, at 2:47 AM, Will Stevens wrote: > > I am also intere

Re: Bug on delete snapshot function / CLOUDSTACK-9297

2016-03-02 Thread Özhan Rüzgar Karaman
Hi Wei; We will follow up the ticket with you and Mike's guidance. Thanks Özhan On Wed, Mar 2, 2016 at 11:35 AM, Wei ZHOU wrote: > Hi Ozhan, > > please check the comments of the ticket. > > -Wei > > 2016-03-02 10:22 GMT+01:00 Özhan Rüzgar Karaman >: > > > Hi; > > 2 weeks ago we noticed a bug o

[GitHub] cloudstack pull request: ADD be explicit about the underlying limi...

2016-03-02 Thread ntavares
GitHub user ntavares opened a pull request: https://github.com/apache/cloudstack/pull/1426 ADD be explicit about the underlying limitation - OpenSwan According to: http://linux.die.net/man/5/ipsec.secrets A preshared secret is most conveniently represented as a sequence

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

2016-03-02 Thread jenkins
See -- [...truncated 28679 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: findbugs: new found old instance of DM_DE...

2016-03-02 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1427 findbugs: new found old instance of DM_DEFAULT_ENCODING fixed this quick fix to silence the findbugs which seems to think this is a new issue You can merge this pull request into

Re: Important Pending Items

2016-03-02 Thread Will Stevens
Thanks Sanjeev, I did not realize that the whole simulator was tested. Ok that is good... :) *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Wed, Mar 2, 2016 at 12:15 AM, Sanjeev N wrote:

[GitHub] cloudstack pull request: CLOUDSTACK-9164: Prevent firefox's quick ...

2016-03-02 Thread footplus
Github user footplus commented on the pull request: https://github.com/apache/cloudstack/pull/1271#issuecomment-191264802 Hello, I was able to confirm this patch works on CloudStack 4.8.0. The only thing is to take care of removing the systemvm-4.8.0.iso from the secondary storage, an

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

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

4.9 Release Management

2016-03-02 Thread Will Stevens
Hello Everyone, I have mentioned this in other related threads, but I wanted to make an official thread on the topic. I am nominating myself as the release manager for 4.9. Please feel free to discuss if you have comments or concerns. I will not be working alone, I will be assisted by Koushik Da

[GitHub] cloudstack pull request: [CLOUDSTACK-8973] Fix create template fro...

2016-03-02 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1424#discussion_r54762197 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -1388,7 +1388,7 @@ public TemplateResponse createTemplateUpdateResponse(ResponseView view,

[GitHub] cloudstack pull request: [CLOUDSTACK-8973] Fix create template fro...

2016-03-02 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1424#discussion_r54762401 --- Diff: server/src/com/cloud/template/TemplateManagerImpl.java --- @@ -275,6 +277,8 @@ private StorageCacheManager cacheMgr; @Inject

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

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

Fwd: Build failed in Jenkins: build-master-slowbuild #3368

2016-03-02 Thread Daan Hoogland
People, these were irritating me. I wrote a quick fix [1]. Anyone feels like running a test on it? [1] https://github.com/apache/cloudstack/pull/1427 -- Forwarded message -- From: Date: Wed, Mar 2, 2016 at 4:09 PM Subject: Build failed in Jenkins: build-master-slowbuild #3368 To

RE: Build failed in Jenkins: build-master-slowbuild #3368

2016-03-02 Thread Raja Pullela
Daan, I will try to test this out.. -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Wednesday, March 2, 2016 11:57 PM To: dev Subject: Fwd: Build failed in Jenkins: build-master-slowbuild #3368 People, these were irritating me. I wrote a quick fix [1]. An

Re: Build failed in Jenkins: build-master-slowbuild #3368

2016-03-02 Thread Daan Hoogland
thanks Raja On Wed, Mar 2, 2016 at 7:48 PM, Raja Pullela wrote: > Daan, I will try to test this out.. > > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Wednesday, March 2, 2016 11:57 PM > To: dev > Subject: Fwd: Build failed in Jenkins: build-master-s

[GitHub] cloudstack pull request: findbugs: new found old instance of DM_DE...

2016-03-02 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1427#issuecomment-191389512 It looks like your IDE had a field day with this class. So other than the reordering of imports, adding `final` to a ton of variables and removing some unneeded brac

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-03-02 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-191393797 @rafaelweingartner @pedro-martins thanks for your comments! I deleted the constructor and used setters instead and not included the Builder pattern, but if you wan

RE: 4.9 Release Management

2016-03-02 Thread Samir Agarwal
Kudos Will! I had received many private notes wondering if Accelerite will continue to play a strong role in contributions to the community; here is your proof! We wanted to take on the biggest pain points in the community, and see how we can make positive contributions. Koushik Das will work a

RE: 4.9 Release Management

2016-03-02 Thread Paul Angus
Thanks for taking up the baton Will et al. Just for clarity - given that getting CI 'done' will take some time (mainly because of the state of the Marvin Tests and Marvin output) - are 'we' now dropping the monthly releases? [ShapeBlue] Paul Angus VP Technology ,

[GitHub] cloudstack pull request: findbugs: new found old instance of DM_DE...

2016-03-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1427#issuecomment-19145 :) @swill it is the only part I typed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

Re: 4.9 Release Management

2016-03-02 Thread Will Stevens
I don't think I can answer that at this time. My personal opinion is that CI is more important than the release timeline, but others may not share this opinion (as it is only my opinion). I am full steam ahead trying to review what we have in place and the work that different parties have done o

[GitHub] cloudstack pull request: findbugs: new found old instance of DM_DE...

2016-03-02 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1427#issuecomment-191424588 Haha. Fair enough. I will let Raja test, but otherwise it LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

Re: 4.9 Release Management

2016-03-02 Thread Paul Angus
That's good enough for me (tm). you're figuring out what you need to figure out. I know the feeling. :) We probably need to have a concensus around release cadence and stick to it for a while. Otherwise we're going to lose some credibility as project. From: Will Stevens Sent: 2 Mar 2016 20

Re: 4.9 Release Management

2016-03-02 Thread Simon Weller
Will, You have our support! Let us know what we can do to help. - Si From: Paul Angus Sent: Wednesday, March 2, 2016 3:09 PM To: dev@cloudstack.apache.org Subject: Re: 4.9 Release Management That's good enough for me (tm). you're figuring out what you

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

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

Re: 4.9 Release Management

2016-03-02 Thread Will Stevens
Yes, I understand and agree with you. On the flip side, the lack of CI has been a thorn in our side for a long time and the increased cadence only adds to this problem, so I am trying to walk a thin line here. Personally, if CI is the only thing I can contribute to ACS as the 4.9 RM, I will be ha

Re: 4.9 Release Management

2016-03-02 Thread Will Stevens
Thanks Simon. :) *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Wed, Mar 2, 2016 at 4:14 PM, Simon Weller wrote: > Will, > > You have our support! > > Let us know what we can do to help.

Re: 4.9 Release Management

2016-03-02 Thread Sebastien Goasguen
> On Mar 2, 2016, at 10:21 PM, Will Stevens wrote: > > Yes, I understand and agree with you. On the flip side, the lack of CI has > been a thorn in our side for a long time and the increased cadence only > adds to this problem, so I am trying to walk a thin line here. > > Personally, if CI is

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-03-02 Thread pedro-martins
Github user pedro-martins commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1425#discussion_r54797480 --- Diff: server/test/com/cloud/api/query/dao/TemplateJoinDaoImplTest.java --- @@ -0,0 +1,56 @@ +package com.cloud.api.query.dao; + +im

Re: 4.9 Release Management

2016-03-02 Thread Ahmad Emneina
4.9 should only be released if it passes the CI tests... pardon the pun. On Wed, Mar 2, 2016 at 1:21 PM, Will Stevens wrote: > Yes, I understand and agree with you. On the flip side, the lack of CI has > been a thorn in our side for a long time and the increased cadence only > adds to this prob

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-03-02 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-191455410 @nvazquez I have reviewed your code. Class “VolumeJoinDaoImpl” at line 46, why did you change the get Logger method to use “VolumeJoinDaoImplTest

Re: Important Pending Items

2016-03-02 Thread Will Stevens
I would love that. Thank you. I have done a very high level review of the code and I like what I see. Are my current assumptions correct? - It uses KVM as its base hypervisor and all environments are spun up and tested on it? - It currently supports testing KVM and Xen? - It currently supports

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

2016-03-02 Thread jenkins
See -- [...truncated 824 lines...] at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155) at org.apache.maven.surefire.booter.ForkedBooter

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

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

Re: 4.9 Release Management

2016-03-02 Thread Koushik Das
Thanks Will for initiating the discussion on 4.9 release management. As already mentioned, I along with Patrick will be helping out Will with the release process. Having a reliable CI system is very important going forward. There are discussions already happening in the list related to it. I ag

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

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