[camel-k] Diff for: [GitHub] nicolaferraro merged pull request #329: chore: refactor source inspector

2019-01-15 Thread GitBox
diff --git a/pkg/metadata/dependencies.go b/pkg/metadata/dependencies.go deleted file mode 100644 index f70efdd1.. --- a/pkg/metadata/dependencies.go +++ /dev/null @@ -1,88 +0,0 @@ -/* -Licensed to the Apache Software Foundation (ASF) under one or more -contributor license agreements. See

[camel] Diff for: [GitHub] asfgit closed pull request #2708: CAMEL-13054: Olingo4Endpoint - avoid swallowing consumer options

2019-01-15 Thread GitBox
diff --git a/components/camel-olingo4/camel-olingo4-component/src/main/java/org/apache/camel/component/olingo4/Olingo4Endpoint.java b/components/camel-olingo4/camel-olingo4-component/src/main/java/org/apache/camel/component/olingo4/Olingo4Endpoint.java index dfd9e93738f..c1ccfb4b413 100644 ---

[camel-k] Diff for: [GitHub] lburgazzoli closed pull request #325: Release some predefined images for Knative

2019-01-13 Thread GitBox
diff --git a/.gitignore b/.gitignore index e43f8299..6361983f 100644 --- a/.gitignore +++ b/.gitignore @@ -1,6 +1,7 @@ # Binary files /camel-k /kamel +/publisher # Released Packages *.tar.gz diff --git a/cmd/util/publisher/publisher.go b/cmd/util/publisher/publisher.go new file mode 100644

[GitHub] [camel-quarkus] gnodet opened a new pull request #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
gnodet opened a new pull request #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342061236 ## File path: build/eclipse-formatter-config.xml ## @@ -262,5 +262,6 @@ + Review comment:

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342058383 ## File path: build/eclipse-formatter-config.xml ## @@ -0,0 +1,264 @@ + + + + + + + + + +

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342054951 ## File path: build/eclipse-formatter-config.xml ## @@ -0,0 +1,264 @@ + + + + + + Review comment: All these

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342052026 ## File path: build/eclipse-formatter-config.xml ## @@ -0,0 +1,264 @@ + + + + + + + + + +

[GitHub] [camel-quarkus] gastaldi commented on issue #372: Upgrade to Quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi commented on issue #372: Upgrade to Quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/issues/372#issuecomment-549463256 This is the stacktrace seen in quarkus-platform: ```java Nov 04, 2019 12:57:57 PM io.quarkus.vertx.http.runtime.QuarkusErrorHandler handle

[GitHub] [camel-quarkus] gnodet opened a new pull request #375: Reuse camel package maven plugin to avoid duplicating code, fixes #336

2019-11-04 Thread GitBox
gnodet opened a new pull request #375: Reuse camel package maven plugin to avoid duplicating code, fixes #336 URL: https://github.com/apache/camel-quarkus/pull/375 This is an automated message from the Apache Git Service.

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342163963 ## File path: examples/timer-log-xml/src/main/java/org/acme/timer/package-info.java ## @@ -1,18 +1,14 @@ /** - * Licensed to the

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
ppalaga commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342168028 ## File path: examples/rest-json/src/main/java/org/acme/rest/json/Fruit.java ## @@ -58,7 +58,7 @@ public boolean equals(Object obj) {

[GitHub] [camel-quarkus] gastaldi edited a comment on issue #372: Upgrade to Quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi edited a comment on issue #372: Upgrade to Quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/issues/372#issuecomment-549364017 @gnodet it looks like it. The tests run fine for me too when bumping the quarkus version in this repository, but fails in the

[GitHub] [camel-quarkus] ppalaga commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a…

2019-11-04 Thread GitBox
ppalaga commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a… URL: https://github.com/apache/camel-quarkus/pull/370#issuecomment-549429222 e89d48e goes back to

[GitHub] [camel-quarkus] gastaldi opened a new issue #372: Upgrade to Quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi opened a new issue #372: Upgrade to Quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/issues/372 Right now simply bumping the version in the platform BOM in https://github.com/quarkusio/quarkus-platform fails on the MicrProfileMetricsTest: ```bash [ERROR]

[GitHub] [camel-quarkus] lburgazzoli commented on issue #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
lburgazzoli commented on issue #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373#issuecomment-549408850 ok to test This is an automated message from the Apache Git Service. To respond to

[GitHub] [camel-k] apupier opened a new pull request #1042: Fix typo in doc

2019-11-04 Thread GitBox
apupier opened a new pull request #1042: Fix typo in doc URL: https://github.com/apache/camel-k/pull/1042 Signed-off-by: Aurélien Pupier **Release Note** ```release-note NONE ```

[GitHub] [camel-quarkus] gastaldi commented on issue #372: Upgrade to Quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi commented on issue #372: Upgrade to Quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/issues/372#issuecomment-549364017 @gnodet it looks like it. The tests run fine for me too when bumping the quarkus version in this repository, but fails in the

[GitHub] [camel-quarkus] gnodet commented on issue #372: Upgrade to Quarkus 0.28.0

2019-11-04 Thread GitBox
gnodet commented on issue #372: Upgrade to Quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/issues/372#issuecomment-549360830 @gastaldi are you sure this isn't a transient issue ? the tests do run fine for me with #373

[GitHub] [camel-quarkus] asf-ci commented on issue #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
asf-ci commented on issue #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373#issuecomment-549457804 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/381/

[GitHub] [camel-k] astefanutti closed issue #990: support for quarkus in JVM mode

2019-11-04 Thread GitBox
astefanutti closed issue #990: support for quarkus in JVM mode URL: https://github.com/apache/camel-k/issues/990 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-k] astefanutti merged pull request #1013: feat(quarkus): Initial Quarkus support

2019-11-04 Thread GitBox
astefanutti merged pull request #1013: feat(quarkus): Initial Quarkus support URL: https://github.com/apache/camel-k/pull/1013 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [camel-quarkus] lburgazzoli commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a…

2019-11-04 Thread GitBox
lburgazzoli commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a… URL: https://github.com/apache/camel-quarkus/pull/370#issuecomment-549412933 @ppalaga one of the actions is failing

[GitHub] [camel-quarkus] ppalaga commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a…

2019-11-04 Thread GitBox
ppalaga commented on issue #370: Improve the Github workflows: use -ntp, check licenses, better step a… URL: https://github.com/apache/camel-quarkus/pull/370#issuecomment-549427171 > @ppalaga one of the actions is failing Interesting. Looks like the Quarkus tooling depends on a

[GitHub] [camel-quarkus] gnodet commented on a change in pull request #371: Issue 337

2019-11-04 Thread GitBox
gnodet commented on a change in pull request #371: Issue 337 URL: https://github.com/apache/camel-quarkus/pull/371#discussion_r342145196 ## File path: build/eclipse-formatter-config.xml ## @@ -0,0 +1,264 @@ + + + + + + + + + +

[GitHub] [camel-quarkus] ppalaga commented on issue #348: Fix #220 platform-http component should return 204 for success and no…

2019-10-30 Thread GitBox
ppalaga commented on issue #348: Fix #220 platform-http component should return 204 for success and no… URL: https://github.com/apache/camel-quarkus/pull/348#issuecomment-547795777 bf76e48 replaced `LOG.debugf(e, "Could not handle '%s'", path)` with

[GitHub] [camel-quarkus] lburgazzoli commented on issue #344: WIP: Move to Quarkus 0.27+ yaml descriptor

2019-10-29 Thread GitBox
lburgazzoli commented on issue #344: WIP: Move to Quarkus 0.27+ yaml descriptor URL: https://github.com/apache/camel-quarkus/pull/344#issuecomment-547743272 cherry picking the commits here into https://github.com/apache/camel-quarkus/pull/350 so I can work also on upgrading the validation

[GitHub] [camel-quarkus] asf-ci commented on issue #348: Fix #220 platform-http component should return 204 for success and no…

2019-10-30 Thread GitBox
asf-ci commented on issue #348: Fix #220 platform-http component should return 204 for success and no… URL: https://github.com/apache/camel-quarkus/pull/348#issuecomment-547834984 Refer to this link for build results (access rights to CI server needed):

[GitHub] [camel-quarkus] lburgazzoli closed issue #220: platform-http component should return 204 for success and no body

2019-10-30 Thread GitBox
lburgazzoli closed issue #220: platform-http component should return 204 for success and no body URL: https://github.com/apache/camel-quarkus/issues/220 This is an automated message from the Apache Git Service. To respond

[GitHub] [camel-quarkus] lburgazzoli closed issue #299: Use Camel's async processor in platform-http

2019-10-30 Thread GitBox
lburgazzoli closed issue #299: Use Camel's async processor in platform-http URL: https://github.com/apache/camel-quarkus/issues/299 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [camel-quarkus] lburgazzoli merged pull request #348: Fix #220 platform-http component should return 204 for success and no…

2019-10-30 Thread GitBox
lburgazzoli merged pull request #348: Fix #220 platform-http component should return 204 for success and no… URL: https://github.com/apache/camel-quarkus/pull/348 This is an automated message from the Apache Git Service. To

[GitHub] [camel-quarkus] ppalaga opened a new issue #352: Re-enable the FHIR tests once the FHIR testing service is back again

2019-10-30 Thread GitBox
ppalaga opened a new issue #352: Re-enable the FHIR tests once the FHIR testing service is back again URL: https://github.com/apache/camel-quarkus/issues/352 Set `quarkus.camel.fhir.enable-dstu3=true` in

[GitHub] [camel-quarkus] davsclaus commented on a change in pull request #348: Fix #220 platform-http component should return 204 for success and no…

2019-10-30 Thread GitBox
davsclaus commented on a change in pull request #348: Fix #220 platform-http component should return 204 for success and no… URL: https://github.com/apache/camel-quarkus/pull/348#discussion_r340464216 ## File path:

[GitHub] [camel-k] dylanenabled opened a new issue #1030: Way to set a serviceaccount for the integration pod?

2019-10-30 Thread GitBox
dylanenabled opened a new issue #1030: Way to set a serviceaccount for the integration pod? URL: https://github.com/apache/camel-k/issues/1030 I'm using AWS EKS to host a kubernetes cluster and I want my camelk integration to connect to an AWS SQS queue to receive messages. AWS

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

2019-10-30 Thread GitBox
ppalaga commented on issue #136: @ConfigProperty not injected in a RouteBuilder URL: https://github.com/apache/camel-quarkus/issues/136#issuecomment-547844788 I have primitive PoC how this could work. Plz. assign to me.

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #348: Fix #220 platform-http component should return 204 for success and no…

2019-10-30 Thread GitBox
ppalaga commented on a change in pull request #348: Fix #220 platform-http component should return 204 for success and no… URL: https://github.com/apache/camel-quarkus/pull/348#discussion_r340461185 ## File path:

[GitHub] [camel-k] dylanenabled opened a new issue #1031: Support for AWS ECR in kaniko builder

2019-10-30 Thread GitBox
dylanenabled opened a new issue #1031: Support for AWS ECR in kaniko builder URL: https://github.com/apache/camel-k/issues/1031 Currently the kaniko builder only supports pushing images to Google. I'd like it if kaniko could push builds to AWS ECR. It looks like this is fairly

[GitHub] [camel-k] hguerrero commented on issue #563: Inject CamelContext into Java DSL RouteBuilder

2019-11-01 Thread GitBox
hguerrero commented on issue #563: Inject CamelContext into Java DSL RouteBuilder URL: https://github.com/apache/camel-k/issues/563#issuecomment-548928318 @davsclaus @lburgazzoli can you add an example of how to configure the context with a connection factory using Java DSL?

[GitHub] [camel-k] pabloxtiyo opened a new issue #1039: Camel K Operator is not reading the Telegram endpoint API-Key properly.

2019-11-02 Thread GitBox
pabloxtiyo opened a new issue #1039: Camel K Operator is not reading the Telegram endpoint API-Key properly. URL: https://github.com/apache/camel-k/issues/1039 Hi, im running Camel-K last version over GKE and when I try to use the Telegram Endpoint I'm getting this issue: Caused

[GitHub] [camel-quarkus] asf-ci commented on issue #374: chore(deps): remove jetty-client from suport/common

2019-11-04 Thread GitBox
asf-ci commented on issue #374: chore(deps): remove jetty-client from suport/common URL: https://github.com/apache/camel-quarkus/pull/374#issuecomment-549550377 Refer to this link for build results (access rights to CI server needed):

[GitHub] [camel-quarkus] gastaldi commented on issue #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi commented on issue #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373#issuecomment-549479921 Better update to 0.28.1 (it should be available soon) This is an automated message from

[GitHub] [camel-quarkus] gastaldi commented on issue #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
gastaldi commented on issue #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373#issuecomment-549479646 I can confirm that the integration tests in quarkus-platform pass with this PR This is

[GitHub] [camel-quarkus] lburgazzoli merged pull request #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
lburgazzoli merged pull request #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373 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 issue #373: Upgrade to quarkus 0.28.0

2019-11-04 Thread GitBox
lburgazzoli commented on issue #373: Upgrade to quarkus 0.28.0 URL: https://github.com/apache/camel-quarkus/pull/373#issuecomment-549512812 @gastaldi I suspect it may be a problem of bom imported in various location This is

[GitHub] [camel-quarkus] lburgazzoli commented on issue #374: chore(deps): remove jetty-client from suport/common

2019-11-04 Thread GitBox
lburgazzoli commented on issue #374: chore(deps): remove jetty-client from suport/common URL: https://github.com/apache/camel-quarkus/pull/374#issuecomment-549512198 ok to test This is an automated message from the Apache

[GitHub] [camel-quarkus] lburgazzoli closed issue #372: Upgrade to Quarkus 0.28.1

2019-11-04 Thread GitBox
lburgazzoli closed issue #372: Upgrade to Quarkus 0.28.1 URL: https://github.com/apache/camel-quarkus/issues/372 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-quarkus] lburgazzoli opened a new pull request #367: Add description to extension matedata

2019-11-03 Thread GitBox
lburgazzoli opened a new pull request #367: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/pull/367 Fixes #360 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] lburgazzoli commented on issue #367: Add description to extension matedata

2019-11-03 Thread GitBox
lburgazzoli commented on issue #367: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/pull/367#issuecomment-549123227 ok to test This is an automated message from the Apache Git Service. To

[GitHub] [camel-quarkus] lburgazzoli closed issue #347: Add category to extension descriptors

2019-11-03 Thread GitBox
lburgazzoli closed issue #347: Add category to extension descriptors URL: https://github.com/apache/camel-quarkus/issues/347 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [camel-quarkus] lburgazzoli commented on issue #347: Add category to extension descriptors

2019-11-03 Thread GitBox
lburgazzoli commented on issue #347: Add category to extension descriptors URL: https://github.com/apache/camel-quarkus/issues/347#issuecomment-549122205 Done with https://github.com/apache/camel-quarkus/pull/350 This is an

[GitHub] [camel-quarkus] lburgazzoli merged pull request #367: Add description to extension matedata

2019-11-03 Thread GitBox
lburgazzoli merged pull request #367: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/pull/367 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [camel-quarkus] lburgazzoli closed issue #360: Add description to extension matedata

2019-11-03 Thread GitBox
lburgazzoli closed issue #360: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/issues/360 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [camel-quarkus] ppalaga opened a new pull request #366: Set camel-quarkus-last-release: 0.3.1 in site.yml

2019-11-03 Thread GitBox
ppalaga opened a new pull request #366: Set camel-quarkus-last-release: 0.3.1 in site.yml URL: https://github.com/apache/camel-quarkus/pull/366 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] asf-ci commented on issue #367: Add description to extension matedata

2019-11-03 Thread GitBox
asf-ci commented on issue #367: Add description to extension matedata URL: https://github.com/apache/camel-quarkus/pull/367#issuecomment-549130764 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/379/

[GitHub] [camel-quarkus] lburgazzoli merged pull request #366: Set camel-quarkus-last-release: 0.3.1 in site.yml

2019-11-03 Thread GitBox
lburgazzoli merged pull request #366: Set camel-quarkus-last-release: 0.3.1 in site.yml URL: https://github.com/apache/camel-quarkus/pull/366 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] lburgazzoli commented on issue #352: Re-enable the FHIR tests once the FHIR testing service is back again

2019-11-03 Thread GitBox
lburgazzoli commented on issue #352: Re-enable the FHIR tests once the FHIR testing service is back again URL: https://github.com/apache/camel-quarkus/issues/352#issuecomment-549122675 May be better to use something like: https://hapifhir.io/doc_server_tester.html

[GitHub] [camel-quarkus] lburgazzoli closed pull request #368: DO NOT MERGE

2019-11-03 Thread GitBox
lburgazzoli closed pull request #368: DO NOT MERGE URL: https://github.com/apache/camel-quarkus/pull/368 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [camel-quarkus] lburgazzoli commented on issue #143: Create a Camel AMQP extension

2019-11-03 Thread GitBox
lburgazzoli commented on issue #143: Create a Camel AMQP extension URL: https://github.com/apache/camel-quarkus/issues/143#issuecomment-549133527 @zhfeng I' not very sure about the need of wait for https://github.com/quarkusio/quarkus/issues/3436 as camel-jms uses quite a log of Spring

[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #143: Create a Camel AMQP extension

2019-11-03 Thread GitBox
lburgazzoli edited a comment on issue #143: Create a Camel AMQP extension URL: https://github.com/apache/camel-quarkus/issues/143#issuecomment-549133527 @zhfeng I' not very sure about the need of wait for https://github.com/quarkusio/quarkus/issues/3436 as camel-jms uses quite a log of

[GitHub] [camel-quarkus] lburgazzoli commented on issue #369: Use capabilities instead of feature names

2019-11-03 Thread GitBox
lburgazzoli commented on issue #369: Use capabilities instead of feature names URL: https://github.com/apache/camel-quarkus/pull/369#issuecomment-549156019 ok to test This is an automated message from the Apache Git Service.

[GitHub] [camel-quarkus] asf-ci commented on issue #369: Use capabilities instead of feature names

2019-11-03 Thread GitBox
asf-ci commented on issue #369: Use capabilities instead of feature names URL: https://github.com/apache/camel-quarkus/pull/369#issuecomment-549164952 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/380/

[GitHub] [camel-quarkus] lburgazzoli merged pull request #369: Use capabilities instead of feature names

2019-11-03 Thread GitBox
lburgazzoli merged pull request #369: Use capabilities instead of feature names URL: https://github.com/apache/camel-quarkus/pull/369 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [camel-quarkus] lburgazzoli commented on issue #289: Run CI with Java 11

2019-11-03 Thread GitBox
lburgazzoli commented on issue #289: Run CI with Java 11 URL: https://github.com/apache/camel-quarkus/issues/289#issuecomment-549180568 We have not a GitHub Action that builds the project in JVM mode against Java 11 This is

[GitHub] [camel-quarkus] zhfeng commented on issue #143: Create a Camel AMQP extension

2019-11-03 Thread GitBox
zhfeng commented on issue #143: Create a Camel AMQP extension URL: https://github.com/apache/camel-quarkus/issues/143#issuecomment-549199908 OK, I will take a look at the camel-sjms This is an automated message from the

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-10 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344565753 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-10 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344565753 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-551896685 I think we really need to get rid of the double discovery as having two identical builder is confusing and

[GitHub] [camel-quarkus] lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
lburgazzoli edited a comment on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-551896685 I think we really need to get rid of the double discovery as having two identical builder is confusing

[GitHub] [camel-quarkus] lburgazzoli opened a new pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
lburgazzoli opened a new pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392 Fixes #389 This is an automated

[GitHub] [camel-quarkus] lburgazzoli merged pull request #390: Add SnakeYAML extension

2019-11-11 Thread GitBox
lburgazzoli merged pull request #390: Add SnakeYAML extension URL: https://github.com/apache/camel-quarkus/pull/390 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-quarkus] davsclaus commented on issue #391: camel-validator component

2019-11-11 Thread GitBox
davsclaus commented on issue #391: camel-validator component URL: https://github.com/apache/camel-quarkus/issues/391#issuecomment-552384606 Just mind you may need to add the new extensions to some pom files in the poms folder. The create extension tool dont get around doing all that 100%.

[GitHub] [camel-quarkus] asf-ci commented on issue #390: Add SnakeYAML extension

2019-11-11 Thread GitBox
asf-ci commented on issue #390: Add SnakeYAML extension URL: https://github.com/apache/camel-quarkus/pull/390#issuecomment-552423909 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/camel-quarkus-pr/385/

[GitHub] [camel-quarkus] lburgazzoli commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
lburgazzoli commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#issuecomment-552423722 The default type registry does it's own class-path scanning and does not offer an hook

[GitHub] [camel-quarkus] lburgazzoli closed issue #338: Create a Camel SnakeYAML extension

2019-11-11 Thread GitBox
lburgazzoli closed issue #338: Create a Camel SnakeYAML extension URL: https://github.com/apache/camel-quarkus/issues/338 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 issue #390: Add SnakeYAML extension

2019-11-11 Thread GitBox
lburgazzoli commented on issue #390: Add SnakeYAML extension URL: https://github.com/apache/camel-quarkus/pull/390#issuecomment-552378989 ok to test This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-11 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344665013 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
lburgazzoli commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#issuecomment-552409366 ok to test This is an

[GitHub] [camel-quarkus] ppalaga commented on issue #389: [code formatter] multi line TODOs are not supported

2019-11-11 Thread GitBox
ppalaga commented on issue #389: [code formatter] multi line TODOs are not supported URL: https://github.com/apache/camel-quarkus/issues/389#issuecomment-552353847 In Quarkus it works the way you describe and they have the following two settings that might be what we are missing:

[GitHub] [camel-quarkus] ploef opened a new issue #391: camel-validator component

2019-11-11 Thread GitBox
ploef opened a new issue #391: camel-validator component URL: https://github.com/apache/camel-quarkus/issues/391 Hi, I'm converting some of my Spring Boot services to Quarkus and see that the camel-validation-component is missing. I will try to create an extension for it.

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-11 Thread GitBox
ppalaga commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344661828 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-11 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344665013 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
lburgazzoli commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#discussion_r344684441 ## File path:

[GitHub] [camel-quarkus] ploef opened a new pull request #393: added Camel validation extension

2019-11-11 Thread GitBox
ploef opened a new pull request #393: added Camel validation extension URL: https://github.com/apache/camel-quarkus/pull/393 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [camel-quarkus] jamesnetherton opened a new pull request #390: Add SnakeYAML extension

2019-11-11 Thread GitBox
jamesnetherton opened a new pull request #390: Add SnakeYAML extension URL: https://github.com/apache/camel-quarkus/pull/390 fixes #338 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-11 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344665013 ## File path:

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
ppalaga commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#discussion_r344679240 ## File path:

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
ppalaga commented on a change in pull request #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#discussion_r344677133 ## File path:

[GitHub] [camel-quarkus] lburgazzoli opened a new issue #389: [code formatter] multi line TODOs are not supported

2019-11-11 Thread GitBox
lburgazzoli opened a new issue #389: [code formatter] multi line TODOs are not supported URL: https://github.com/apache/camel-quarkus/issues/389 Assuming I have a TODO like: ```java // TODO: blablabla // blablabla ``` This is then re-formatted as:

[GitHub] [camel-quarkus] ppalaga commented on issue #391: camel-validator component

2019-11-11 Thread GitBox
ppalaga commented on issue #391: camel-validator component URL: https://github.com/apache/camel-quarkus/issues/391#issuecomment-552383469 Nice to hear that! Do not hesitate to ask here if you have specific questions. You might want to check

[GitHub] [camel-quarkus] lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-552000542 does this means that route builder classes are now instantiated at runtime only ?

[GitHub] [camel-quarkus] ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-552003860 > does this means that route builder classes are now instantiated at runtime only ? Yes, the constructors

[GitHub] [camel-quarkus] lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
lburgazzoli commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344382003 ## File path:

[GitHub] [camel-quarkus] lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
lburgazzoli commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-552005268 This is IMHO bad as we are de optimizing the app

[GitHub] [camel-quarkus] ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
ppalaga commented on issue #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#issuecomment-551999282 b14487b is the most elegant fix I was able to find. All RoutesBuilders are instantiated by Arc and queried via

[GitHub] [camel-quarkus] davsclaus closed issue #386: @Inject CamelContext on RouteBuilder seems not working

2019-11-08 Thread GitBox
davsclaus closed issue #386: @Inject CamelContext on RouteBuilder seems not working URL: https://github.com/apache/camel-quarkus/issues/386 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] davsclaus commented on issue #386: @Inject CamelContext on RouteBuilder seems not working

2019-11-08 Thread GitBox
davsclaus commented on issue #386: @Inject CamelContext on RouteBuilder seems not working URL: https://github.com/apache/camel-quarkus/issues/386#issuecomment-551997566 See ticket #136 This is an automated message from the

[GitHub] [camel-quarkus] davsclaus closed issue #384: camel-quarkus-microprofile-metrics - Does not work

2019-11-08 Thread GitBox
davsclaus closed issue #384: camel-quarkus-microprofile-metrics - Does not work URL: https://github.com/apache/camel-quarkus/issues/384 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders

2019-11-08 Thread GitBox
ppalaga commented on a change in pull request #358: Fix #136 @ConfigProperty and @Inject do not work in RouteBuilders URL: https://github.com/apache/camel-quarkus/pull/358#discussion_r344385789 ## File path:

[GitHub] [camel-quarkus] davsclaus commented on issue #384: camel-quarkus-microprofile-metrics - Does not work

2019-11-08 Thread GitBox
davsclaus commented on issue #384: camel-quarkus-microprofile-metrics - Does not work URL: https://github.com/apache/camel-quarkus/issues/384#issuecomment-551997208 See ticket #319 This is an automated message from the

[GitHub] [camel-quarkus] asf-ci commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner

2019-11-11 Thread GitBox
asf-ci commented on issue #392: Perform TypeConverterLoader discovery using jandex instead of camel's class path scanner URL: https://github.com/apache/camel-quarkus/pull/392#issuecomment-552460424 Refer to this link for build results (access rights to CI server needed):

  1   2   3   4   5   6   7   8   9   10   >