Re: [camel] branch master updated: fix the compilation error because of the missing DefinitionPropertiesPlaceholderProviderHelper class being generated

2019-08-15 Thread Claus Ibsen
I just did a fresh rebuild with Java 11 and it worked fine git clean -dXf rm -rf org/apache/camel in my m2 repo mvn clean install -P fastinstall On Thu, Aug 15, 2019 at 8:51 AM Claus Ibsen wrote: > > Hi > > I am on java 8 and same maven 3.6.1 version > > On Thu, Aug 15, 2019 at 8:35 AM Babak

[GitHub] [camel-quarkus] ppalaga opened a new issue #136: @ConfigProperty not injected in a RouteBuilder

2019-08-15 Thread GitBox
ppalaga opened a new issue #136: @ConfigProperty not injected in a RouteBuilder URL: https://github.com/apache/camel-quarkus/issues/136 Probably caused by how we instantiate RouteBuilders. Stumbled across this when adding a tests to the twitter test project where we have this

[GitHub] [camel-quarkus] asf-ci commented on issue #135: Upgrade to Quarkus 0.21.0

2019-08-15 Thread GitBox
asf-ci commented on issue #135: Upgrade to Quarkus 0.21.0 URL: https://github.com/apache/camel-quarkus/pull/135#issuecomment-521609977 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/89/

Re: Apache Camel 3.0 RC1

2019-08-15 Thread Claus Ibsen
On Thu, Aug 15, 2019 at 1:45 PM Gregor Zurowski wrote: > > Yes, I would be available for cutting the release candidate, but it would > be not before the weekend of August 24. Does that work for everyone? > Yeah sure, we will just get more goodies into the RC1. > Thanks, > Gregor > > On Sat, Aug

[GitHub] [camel-quarkus] lburgazzoli merged pull request #135: Upgrade to Quarkus 0.21.0

2019-08-15 Thread GitBox
lburgazzoli merged pull request #135: Upgrade to Quarkus 0.21.0 URL: https://github.com/apache/camel-quarkus/pull/135 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-quarkus] asf-ci commented on issue #137: Upgrade to Quarkus 0.21.1

2019-08-15 Thread GitBox
asf-ci commented on issue #137: Upgrade to Quarkus 0.21.1 URL: https://github.com/apache/camel-quarkus/pull/137#issuecomment-521674007 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/90/

[GitHub] [camel-quarkus] lburgazzoli commented on issue #136: @ConfigProperty not injected in a RouteBuilder

2019-08-15 Thread GitBox
lburgazzoli commented on issue #136: @ConfigProperty not injected in a RouteBuilder URL: https://github.com/apache/camel-quarkus/issues/136#issuecomment-521671960 you can also use properties placeholders in the uri definition with {{...}} On Thu, 15 Aug 2019 at 16:08, Peter Palaga

Re: Apache Camel 3.0 RC1

2019-08-15 Thread Andrea Cosentino
+1 for me Il gio 15 ago 2019, 13:45 Gregor Zurowski ha scritto: > Yes, I would be available for cutting the release candidate, but it would > be not before the weekend of August 24. Does that work for everyone? > > Thanks, > Gregor > > On Sat, Aug 10, 2019, 2:33 PM Andrea Cosentino wrote: > >

Re: Apache Camel 3.0 RC1

2019-08-15 Thread Gregor Zurowski
Yes, I would be available for cutting the release candidate, but it would be not before the weekend of August 24. Does that work for everyone? Thanks, Gregor On Sat, Aug 10, 2019, 2:33 PM Andrea Cosentino wrote: > +1 for release candidate 1. > > Gregor are you able to cut the RC eventually? >

[GitHub] [camel-quarkus] ppalaga opened a new pull request #137: Upgrade to Quarkus 0.21.1

2019-08-15 Thread GitBox
ppalaga opened a new pull request #137: Upgrade to Quarkus 0.21.1 URL: https://github.com/apache/camel-quarkus/pull/137 This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [camel] branch master updated: fix the compilation error because of the missing DefinitionPropertiesPlaceholderProviderHelper class being generated

2019-08-15 Thread Babak Vahdat
Thanks Claus for fixing it! Those mongodb changes were missing in my PR as I somehow messed it up while merging them from my fork. Is there a documentation about how to merge a PR provided by the GitHub mirror repo into the Apache gitbox repo? I did it as the following which I am not sure if

Re: [camel] branch master updated: fix the compilation error because of the missing DefinitionPropertiesPlaceholderProviderHelper class being generated

2019-08-15 Thread Claus Ibsen
Hi I am on java 8 and same maven 3.6.1 version On Thu, Aug 15, 2019 at 8:35 AM Babak Vahdat wrote: > > Thanks Claus for fixing it! > > Those mongodb changes were missing in my PR as I somehow messed it up while > merging them from my fork. Is there a documentation about how to merge a PR >

[GitHub] [camel-website] zregvart merged pull request #73: Redesign

2019-08-15 Thread GitBox
zregvart merged pull request #73: Redesign URL: https://github.com/apache/camel-website/pull/73 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [camel-quarkus] ppalaga opened a new pull request #135: Upgrade to Quarkus 0.21.0

2019-08-15 Thread GitBox
ppalaga opened a new pull request #135: Upgrade to Quarkus 0.21.0 URL: https://github.com/apache/camel-quarkus/pull/135 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#discussion_r314431942 ## File path:

[GitHub] [camel-website] Nayananga commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
Nayananga commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314431956 ## File path: layouts/_default/list.algolia.json ## @@ -0,0 +1,54 @@ +{{/* Generates a valid

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#discussion_r314431968 ## File path:

[GitHub] [camel-quarkus] ppalaga commented on issue #136: @ConfigProperty not injected in a RouteBuilder

2019-08-15 Thread GitBox
ppalaga commented on issue #136: @ConfigProperty not injected in a RouteBuilder URL: https://github.com/apache/camel-quarkus/issues/136#issuecomment-521689557 Thanks, `{{camel.component.twitter-directmessage.user}}` works. `@ConfigProperty` and `@Inject` do not work because we do

[GitHub] [camel-quarkus] asf-ci commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
asf-ci commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521706070 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/91/

[GitHub] [camel-quarkus] lburgazzoli commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521740717 > > > We may also think about using `camel-mock` > > Not sure how? Injecting to the test class works only JVM mode.

[GitHub] [camel-website] zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314410102 ## File path: static/css/search.css ## @@ -0,0 +1,28 @@ +h1 { Review comment: This file

[GitHub] [camel-website] zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314408603 ## File path: .env ## @@ -0,0 +1,5 @@ +ALGOLIA_APP_ID={{ YOUR_APP_ID }} +SEARCH_ONLY_API_KEY={{

[GitHub] [camel-website] zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314409914 ## File path: layouts/partials/header.html ## @@ -27,6 +27,10 @@ {{ end }} +

[GitHub] [camel-website] zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314409135 ## File path: layouts/_default/list.algolia.json ## @@ -0,0 +1,54 @@ +{{/* Generates a valid

[GitHub] [camel-website] zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314412143 ## File path: layouts/partials/footer.html ## @@ -42,19 +42,11 @@ - -

[GitHub] [camel-website] zregvart commented on issue #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on issue #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#issuecomment-521724187 The build is currently failing with: ``` [2019-08-15T16:38:45.078Z] $ antora --require ./menu.js --clean --fetch site.yml

[GitHub] [camel-quarkus] lburgazzoli commented on issue #136: @ConfigProperty not injected in a RouteBuilder

2019-08-15 Thread GitBox
lburgazzoli commented on issue #136: @ConfigProperty not injected in a RouteBuilder URL: https://github.com/apache/camel-quarkus/issues/136#issuecomment-521695936 there’s a bean post processor in camel that usually take cares of doing injection so we perhaps need to implement a

[GitHub] [camel-website] Nayananga opened a new pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
Nayananga opened a new pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74 I need to set CSS for search and each search result should able to link to the corresponding page when clicked. I would like to know is this

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga commented on a change in pull request #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#discussion_r314426759 ## File path: integration-tests/twitter/src/main/java/org/apache/camel/quarkus/component/twitter/CamelRoute.java

[GitHub] [camel-quarkus] lburgazzoli merged pull request #137: Upgrade to Quarkus 0.21.1

2019-08-15 Thread GitBox
lburgazzoli merged pull request #137: Upgrade to Quarkus 0.21.1 URL: https://github.com/apache/camel-quarkus/pull/137 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-quarkus] ppalaga opened a new pull request #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga opened a new pull request #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138 The tests are run conditionally using `@EnabledIfEnvironmentVariable(named = "TWITTER_CONSUMER_KEY", matches = "[a-zA-Z0-9]+")` so they won't fail if the

[GitHub] [camel-quarkus] lburgazzoli commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521711918 I think this can be greatly simplified by using consumer/producer templates instead i.e. putting thing on a queue and polling. I'd also

[GitHub] [camel-website] zregvart commented on issue #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
zregvart commented on issue #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#issuecomment-521719394 Thanks for this. Can you move the bits that refactor the microdata/seo into a separate PR, from what I understand it's not a

[GitHub] [camel-quarkus] ppalaga commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521739084 > I think this can be greatly simplified by using consumer/producer templates instead i.e. putting thing on a queue and polling.

[GitHub] [camel-quarkus] ppalaga commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521739474 > No we don't have credentials. May I provide mine then? This is an

[GitHub] [camel-quarkus] ppalaga commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521710478 Anybody knows what the Jenkins message `git clean -fdx" returned status code 143:` means?

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli commented on a change in pull request #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#discussion_r314405007 ## File path:

[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
lburgazzoli edited a comment on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521711918 I think this can be greatly simplified by using consumer/producer templates instead i.e. putting thing on a queue and polling.

Re: [DISCUSS] New Camel website, rump up to go live

2019-08-15 Thread Zoran Regvart
Hi Cameleers, today I merged the redesign of the front page. I will work on adding a filter box to the compnents navigation menu (CAMEL-13622[1]), Nayananga has a PR in the works for adding site search. I would still like to address the design of the news section (CAMEL-13818[2]). That would

[GitHub] [camel-quarkus] ppalaga commented on issue #138: Add tests to the twitter itest project

2019-08-15 Thread GitBox
ppalaga commented on issue #138: Add tests to the twitter itest project URL: https://github.com/apache/camel-quarkus/pull/138#issuecomment-521762314 > > > We may also think about using `camel-mock` > > > > > > Not sure how? Injecting to the test class works only JVM mode. >

[GitHub] [camel-website] Nayananga commented on a change in pull request #74: Add search for Hugo (more polishing need to do)

2019-08-15 Thread GitBox
Nayananga commented on a change in pull request #74: Add search for Hugo (more polishing need to do) URL: https://github.com/apache/camel-website/pull/74#discussion_r314474377 ## File path: .env ## @@ -0,0 +1,5 @@ +ALGOLIA_APP_ID={{ YOUR_APP_ID }} +SEARCH_ONLY_API_KEY={{