[VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Hi Everyone: This is a vote to release Apache Camel 2.17.0, a new minor version with ~450 improvements, fixes and new features. Release notes: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12311211&version=1268 Staging repository: https://repository.apache.org/content/rep

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Thomas Diesler
+1 from wildfly-camel > On 21 Mar 2016, at 09:52, Gregor Zurowski wrote: > > Hi Everyone: > > This is a vote to release Apache Camel 2.17.0, a new minor version > with ~450 improvements, fixes and new features. > > Release notes: > https://iss

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread johndament
I noticed that your NOTICE file says to find the LICENSE.txt in the licenses directory. There is no licenses directory and the file is found at the root of the archive. In addition, a number of your dependencies are not apache v2 licensed. I'm not sure that I see the proper level of attribution

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread johndament
Actually, nevermind on the dependencies part. The only 3P libs being shipped are slf4j and jasypt (btw, do those have to be shipped?) -- View this message in context: http://camel.465427.n5.nabble.com/VOTE-Release-Apache-Camel-2-17-0-tp5779375p5779383.html Sent from the Camel Development maili

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gerard Klijs
+1 using the staging repo the poc with secure kafka connection works. On Mon, Mar 21, 2016 at 12:32 PM johndament wrote: > Actually, nevermind on the dependencies part. The only 3P libs being > shipped > are slf4j and jasypt (btw, do those have to be shipped?) > > > > -- > View this message in

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
Hi I found a NPE bug when using osgi blueprint testing with camel-jaxb https://issues.apache.org/jira/browse/CAMEL-9730 And could likely also be a problem when running in real karaf. On Mon, Mar 21, 2016 at 9:52 AM, Gregor Zurowski wrote: > Hi Everyone: > > This is a vote to release Apache

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Grzegorz Grzybek
hmm, *could be* related to https://issues.apache.org/jira/browse/SM-2772, I'll check it. regards Grzegorz 2016-03-21 13:37 GMT+01:00 Claus Ibsen : > Hi > > I found a NPE bug when using osgi blueprint testing with camel-jaxb > https://issues.apache.org/jira/browse/CAMEL-9730 > > And could likely

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
On Mon, Mar 21, 2016 at 2:18 PM, Grzegorz Grzybek wrote: > hmm, *could be* related to https://issues.apache.org/jira/browse/SM-2772, > I'll check it. > No it was in Camel itself in some of its osgi modules. I have fixed it. > regards > Grzegorz > > 2016-03-21 13:37 GMT+01:00 Claus Ibsen : > >> H

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
Hi Okay its fixed now and pushed to master branch. I think it will cause problems for karaf users so I am voting a -1 so we can re-cut a RC. Gregor, can you re-cut from the master branch to pickup this fix and a few other minor ones? On Mon, Mar 21, 2016 at 1:37 PM, Claus Ibsen wrote: > Hi >

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
And btw this does not mean people should stop testing the current RC. We want as much feedback as possible. On Mon, Mar 21, 2016 at 2:34 PM, Claus Ibsen wrote: > Hi > > Okay its fixed now and pushed to master branch. > > I think it will cause problems for karaf users so I am voting a -1 so > we

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Hi Claus, Thanks for your feedback. I will merge your OSGi fix into the 2.17.x branch tonight and roll another RC. Any other commits that should be included in 2.17.0 that were committed on the master branch in the meantime? Thanks, Gregor On Mon, Mar 21, 2016 at 2:25 PM, Claus Ibsen wrote:

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Andrea Cosentino
Hi Gregor, Yes, thank you. This commit is a minor one: https://github.com/apache/camel/commit/a4ebd77174d43f0e23792dc03fd88530db8a29ed Thanks! -- Andrea Cosentino -- Apache Camel PMC Member Apache Karaf Committer Apache Servicemix Committer Email: ancosen1...@y

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Andrea Cosentino
Claus already backported on 2.17.x branch. He was too fast :-) -- Andrea Cosentino -- Apache Camel PMC Member Apache Karaf Committer Apache Servicemix Committer Email: ancosen1...@yahoo.com Twitter: @oscerd2 Github: oscerd On Monday, March 21, 2016 2:38 PM, And

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
@Claus: Do we want to include the following commit in 2.17.0 as well? https://github.com/apache/camel/commit/d48cfeef1aa7a4760a3c43533a558de8f6a8cff9 Thanks, Gregor On Mon, Mar 21, 2016 at 2:35 PM, Claus Ibsen wrote: > And btw this does not mean people should stop testing the current RC. > We w

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
On Mon, Mar 21, 2016 at 3:26 PM, Gregor Zurowski wrote: > @Claus: Do we want to include the following commit in 2.17.0 as well? > > https://github.com/apache/camel/commit/d48cfeef1aa7a4760a3c43533a558de8f6a8cff9 > Yes that is a good one to get in, you are welcome to merge it. > Thanks, > Gregor

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
On Mon, Mar 21, 2016 at 4:33 PM, Claus Ibsen wrote: > On Mon, Mar 21, 2016 at 3:26 PM, Gregor Zurowski > wrote: >> @Claus: Do we want to include the following commit in 2.17.0 as well? >> >> https://github.com/apache/camel/commit/d48cfeef1aa7a4760a3c43533a558de8f6a8cff9 >> > > Yes that is a good

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Hi Claus, It seems that the recent commit for CAMEL-9730 broke the OsgiSpringCamelContext class of the camel-spring module on branch-2.17.x: === [...] [INFO] [INFO] --- maven-compiler-plugin:3.5.1:compile (default-compile) @ camel-spring --- [INFO] [INFO] Changes detected - recompiling the module

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Christian Müller
Jenkins jobs are set up and scheduled. Best, Christian - Software Integration Specialist Apache Member V.P. Apache Camel | Apache Camel PMC Member | Apache Camel committer Apache Incubator PMC Member https://www.linkedin.com/pub/christian-mueller/11/551/642 On Mon, Mar 21, 2016

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Christian Müller
https://builds.apache.org/view/A-D/view/Camel/ Christian - Software Integration Specialist Apache Member V.P. Apache Camel | Apache Camel PMC Member | Apache Camel committer Apache Incubator PMC Member https://www.linkedin.com/pub/christian-mueller/11/551/642 On Mon, Mar 21, 20

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Hi Claus, I believe the issue can be fixed by changing the invocation of the OsgiTypeConverter constructor on line 48 of class OsgiSpringCamelContext from: return new OsgiTypeConverter(ctx, getInjector(), finder); to: return new OsgiTypeConverter(ctx, this, getInjector(), finder); Please let m

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Christian, thanks a lot for settings this up. Could you briefly describe the steps involved with this, so we can add it to the release guide? Thanks, Gregor On Mon, Mar 21, 2016 at 10:12 PM, Christian Müller wrote: > Jenkins jobs are set up and scheduled. > > Best, > Christian > ---

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Gregor Zurowski
Hi Claus, I applied the previously described change to OsgiSpringCamelContext (see https://github.com/apache/camel/commit/1d3c090b37e61d46cf058fa5052227a64de20845) to branches master and 2.17.x. I am currently building a new RC for 2.17.0. Thanks, Gregor On Mon, Mar 21, 2016 at 10:20 PM, Grego

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-21 Thread Claus Ibsen
Hi Gregor Ah yeah sorry for missing that, yeah it was just to pass in the CamelContext to the created type converter registry as its needed. On Mon, Mar 21, 2016 at 10:54 PM, Gregor Zurowski wrote: > Hi Claus, > > I applied the previously described change to OsgiSpringCamelContext > (see > http

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-22 Thread Gregor Zurowski
Hi Claus, Not a problem. I am still working on the release candidate and will send out a new vote once its done. Thanks, Gregor On Tue, Mar 22, 2016 at 6:59 AM, Claus Ibsen wrote: > Hi Gregor > > Ah yeah sorry for missing that, yeah it was just to pass in the > CamelContext to the created type

Re: [VOTE] Release Apache Camel 2.17.0

2016-03-22 Thread Claus Ibsen
Hi Christian Can you change the CI jobs in jenkins 1. delete camel trunk java 7 (we drop support for java 7) 2. delete camel trunk notest java 7 if there is such a job (we drop support for java7) 3. change camel trunk notest to use java 8 4. disable the jobs for 2.15.x as its becoming EOL (we can

[VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-22 Thread Gregor Zurowski
Hi Everyone: This is a vote to release Apache Camel 2.17.0, a new minor version with ~450 improvements, fixes and new features. This is the second attempt which fixes a NPE in camel-jaxb when used with CamelTestBlueprint/OSGi [1]. Release notes: https://issues.apache.org/jira/secure/ReleaseNote.

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-22 Thread Claus Ibsen
Hi I tested the RC2 with the same tests that discovered the issue with RC1 and now it passes. I also tested the RC2 with the Camel in Action 2nd ed source code. Will give the examples a test spin later. But so far looks good. On Tue, Mar 22, 2016 at 1:48 PM, Gregor Zurowski wrote: > Hi Everyone

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-22 Thread Andrea Cosentino
+1 (binding). Tested on some of my personal project. Thanks Gregor. -- Andrea Cosentino -- Apache Camel PMC Member Apache Karaf Committer Apache Servicemix Committer Email: ancosen1...@yahoo.com Twitter: @oscerd2 Github: oscerd On Tuesday, March 22, 2016 3:23 P

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-23 Thread Claus Ibsen
+1 Tested a number of examples both karaf, standalone and spring-boot and cdi. On Tue, Mar 22, 2016 at 3:22 PM, Claus Ibsen wrote: > Hi > > I tested the RC2 with the same tests that discovered the issue with > RC1 and now it passes. > I also tested the RC2 with the Camel in Action 2nd ed sourc

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-23 Thread Antonin Stefanutti
+1 Tested CDI examples (standalone, PAX CDI / Karaf, WildFly). > On 23 Mar 2016, at 08:15, Claus Ibsen wrote: > > +1 > > Tested a number of examples both karaf, standalone and spring-boot and cdi. > > > > On Tue, Mar 22, 2016 at 3:22 PM, Claus Ibsen wrote: >> Hi >> >> I tested the RC2 wit

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-23 Thread Thomas Diesler
+1 > On 22 Mar 2016, at 13:48, Gregor Zurowski wrote: > > Hi Everyone: > > This is a vote to release Apache Camel 2.17.0, a new minor version > with ~450 improvements, fixes and new features. This is the second > attempt which fixes a NPE in camel-jaxb when used with > CamelTestBlueprint/OSGi

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-24 Thread Daniel Kulp
+1 Dan > On Mar 22, 2016, at 8:48 AM, Gregor Zurowski wrote: > > Hi Everyone: > > This is a vote to release Apache Camel 2.17.0, a new minor version > with ~450 improvements, fixes and new features. This is the second > attempt which fixes a NPE in camel-jaxb when used with > CamelTestBluepri

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-24 Thread Christian Müller
+1 camel-braintree is missing the license and notice file (I have added it) camel-etcd is missing the license and notice file (I have added it) camel-ironmq is missing the license and notice file (I have added it) camel-jcache is missing the license and notice file (I have added it) Best, Christi

Re: [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-24 Thread Aki Yoshida
+1 regards, aki 2016-03-22 13:48 GMT+01:00 Gregor Zurowski : > Hi Everyone: > > This is a vote to release Apache Camel 2.17.0, a new minor version > with ~450 improvements, fixes and new features. This is the second > attempt which fixes a NPE in camel-jaxb when used with > CamelTestBlueprint/OS

[RESULT] [VOTE] Release Apache Camel 2.17.0 (Attempt #2)

2016-03-26 Thread Gregor Zurowski
This vote passes with the following results: +1 binding: 6 (Aki Yoshida, Andrea Cosentino, Christian Müller, Claus Ibsen, Daniel Kulp, Gregor Zurowski) +1 non-binding: 2 (Antonin Stefanutti, Thomas Diesler) I will publish the artifacts for Camel 2.17.0 shortly. Thanks, Gregor On Tue, Mar 22,