[GitHub] [camel-kafka-connector] tadayosi commented on issue #924: Camel-Netty-* connectors are not working

2021-02-18 Thread GitBox
tadayosi commented on issue #924: URL: https://github.com/apache/camel-kafka-connector/issues/924#issuecomment-781898276 I'll work on #1036 next, so let's see if I can catch this issue there, too. This is an automated messag

[GitHub] [camel-kafka-connector] oscerd commented on issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-18 Thread GitBox
oscerd commented on issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030#issuecomment-781899042 I guess zip is supported too https://github.com/strimzi/strimzi-kafka-operator/blob/master/install/cluster-operator/041-Crd-kafkaconnect.yaml#L1908 ---

[GitHub] [camel-kafka-connector] tadayosi commented on issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-18 Thread GitBox
tadayosi commented on issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030#issuecomment-781899719 Ah cool. Then I'm fine with zip, too. This is an automated message from the Apache Git Service. To re

[GitHub] [camel-kafka-connector] oscerd commented on issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-18 Thread GitBox
oscerd commented on issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030#issuecomment-781900034 I'm asking for confirmation, I have the PR ready for this :-) This is an automated message from the Apa

[GitHub] [camel-quarkus] ppalaga commented on pull request #2271: Azure EventHubs test fixup

2021-02-18 Thread GitBox
ppalaga commented on pull request #2271: URL: https://github.com/apache/camel-quarkus/pull/2271#issuecomment-781901102 > @ppalaga I've tried the script and it failed with `Insufficient privileges to complete the operation.` after `az role assignment create` Maybe because you are not

[GitHub] [camel-quarkus] ppalaga commented on pull request #2254: Nitrite native support #1298

2021-02-18 Thread GitBox
ppalaga commented on pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#issuecomment-781902647 @JiriOndrusek could you please check the test failures? This is an automated message from the Apache Git S

[GitHub] [camel-k] astefanutti commented on a change in pull request #2040: feat(trait): toleration trait

2021-02-19 Thread GitBox
astefanutti commented on a change in pull request #2040: URL: https://github.com/apache/camel-k/pull/2040#discussion_r578995319 ## File path: pkg/trait/toleration.go ## @@ -0,0 +1,154 @@ +/* +Licensed to the Apache Software Foundation (ASF) under one or more +contributor licens

[GitHub] [camel-kafka-connector] oscerd opened a new issue #1037: Add a regen bot to gh action like Camel

2021-02-19 Thread GitBox
oscerd opened a new issue #1037: URL: https://github.com/apache/camel-kafka-connector/issues/1037 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [camel-kafka-connector] oscerd commented on issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd commented on issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030#issuecomment-781920989 @scholzj provided some feedback, zip will be supported from the next release, so since we are planning to release this weekend, I'll change the PR to only generate tar.

[GitHub] [camel-kafka-connector] oscerd edited a comment on issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd edited a comment on issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030#issuecomment-781430403 We'll remove the .zip one. This is an automated message from the Apache Git Service. To respond

[GitHub] [camel-k] bouskaJ opened a new pull request #2054: Wait for operator to start before scaling it to zero

2021-02-19 Thread GitBox
bouskaJ opened a new pull request #2054: URL: https://github.com/apache/camel-k/pull/2054 **Release Note** ```release-note NONE ``` This is an automated message from the Apache Git Ser

[GitHub] [camel-quarkus] llowinge commented on pull request #2271: Azure EventHubs test fixup

2021-02-19 Thread GitBox
llowinge commented on pull request #2271: URL: https://github.com/apache/camel-quarkus/pull/2271#issuecomment-781928659 @ppalaga Yes, probably. It makes the script unusable for me :( However the fix in the test is working with my previously created credentials. Thanks a lot ! ---

[GitHub] [camel-k] johnpoth opened a new pull request #2055: feat: add command completion for connect flag

2021-02-19 Thread GitBox
johnpoth opened a new pull request #2055: URL: https://github.com/apache/camel-k/pull/2055 **Release Note** ```release-note NONE ``` Thanks ! This is an automated message from the A

[GitHub] [camel-quarkus] JiriOndrusek commented on pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#issuecomment-781934978 @ppalaga I'll look into it This is an automated message from the Apache Git Service. To respond to

[GitHub] [camel-quarkus] JiriOndrusek commented on a change in pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on a change in pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#discussion_r579028288 ## File path: integration-tests/nitrite/src/test/java/org/apache/camel/quarkus/component/nitrite/it/NitriteTest.java ## @@ -64,12 +98,15 @@ pu

[GitHub] [camel-quarkus] JiriOndrusek commented on pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#issuecomment-781939717 @ppalaga Failures will go away, once we switch quarkus to 1.13 (requires change quarkusio/quarkus#15043)

[GitHub] [camel-k] squakez opened a new pull request #2056: chore(cmd): skip operator compatibility warning

2021-02-19 Thread GitBox
squakez opened a new pull request #2056: URL: https://github.com/apache/camel-k/pull/2056 Fix #2051 **Release Note** ```release-note NONE ``` This is an automated message from t

[GitHub] [camel-quarkus] JiriOndrusek commented on a change in pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on a change in pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#discussion_r579043850 ## File path: integration-tests/nitrite/src/test/java/org/apache/camel/quarkus/component/nitrite/it/NitriteTest.java ## @@ -64,12 +98,15 @@ pu

[GitHub] [camel-quarkus] JiriOndrusek commented on a change in pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on a change in pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#discussion_r579043967 ## File path: integration-tests/nitrite/src/test/java/org/apache/camel/quarkus/component/nitrite/it/NitriteTest.java ## @@ -78,28 +115,42 @@ p

[GitHub] [camel-quarkus] JiriOndrusek commented on pull request #2254: Nitrite native support #1298

2021-02-19 Thread GitBox
JiriOndrusek commented on pull request #2254: URL: https://github.com/apache/camel-quarkus/pull/2254#issuecomment-781962853 Last comment is wrong, test should succeed also in 1.12.final. I've fixed it. This is an automated me

[GitHub] [camel-k] astefanutti merged pull request #2054: Wait for operator to start before scaling it to zero

2021-02-19 Thread GitBox
astefanutti merged pull request #2054: URL: https://github.com/apache/camel-k/pull/2054 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [camel-k] astefanutti merged pull request #2053: chore(test): Add integration scaling e2e tests

2021-02-19 Thread GitBox
astefanutti merged pull request #2053: URL: https://github.com/apache/camel-k/pull/2053 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [camel-k] astefanutti commented on a change in pull request #2040: feat(trait): toleration trait

2021-02-19 Thread GitBox
astefanutti commented on a change in pull request #2040: URL: https://github.com/apache/camel-k/pull/2040#discussion_r579081890 ## File path: pkg/trait/toleration.go ## @@ -0,0 +1,156 @@ +/* +Licensed to the Apache Software Foundation (ASF) under one or more +contributor licens

[GitHub] [camel-k] astefanutti closed issue #2051: [cmd] Warning on launching operator

2021-02-19 Thread GitBox
astefanutti closed issue #2051: URL: https://github.com/apache/camel-k/issues/2051 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

[GitHub] [camel-k] astefanutti merged pull request #2056: chore(cmd): skip operator compatibility warning

2021-02-19 Thread GitBox
astefanutti merged pull request #2056: URL: https://github.com/apache/camel-k/pull/2056 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [camel-k] astefanutti commented on pull request #2056: chore(cmd): skip operator compatibility warning

2021-02-19 Thread GitBox
astefanutti commented on pull request #2056: URL: https://github.com/apache/camel-k/pull/2056#issuecomment-781991096 Thanks. This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [camel-k] squakez commented on a change in pull request #2040: feat(trait): toleration trait

2021-02-19 Thread GitBox
squakez commented on a change in pull request #2040: URL: https://github.com/apache/camel-k/pull/2040#discussion_r579089188 ## File path: pkg/trait/toleration.go ## @@ -0,0 +1,156 @@ +/* +Licensed to the Apache Software Foundation (ASF) under one or more +contributor license ag

[GitHub] [camel-k-runtime] lburgazzoli opened a new pull request #633: Upgrade to camel-quarkus 1.7.0 (camel 3.8.x, quarkus 1.12.x)

2021-02-19 Thread GitBox
lburgazzoli opened a new pull request #633: URL: https://github.com/apache/camel-k-runtime/pull/633 **Release Note** ```release-note NONE ``` This is an automated message from the Apache G

[GitHub] [camel-kafka-connector] orpiske merged pull request #1028: Refactor the CXF servers into a service for simpler reuse

2021-02-19 Thread GitBox
orpiske merged pull request #1028: URL: https://github.com/apache/camel-kafka-connector/pull/1028 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [camel-kafka-connector] orpiske commented on pull request #1028: Refactor the CXF servers into a service for simpler reuse

2021-02-19 Thread GitBox
orpiske commented on pull request #1028: URL: https://github.com/apache/camel-kafka-connector/pull/1028#issuecomment-781997671 Problems were fixed. Merged. This is an automated message from the Apache Git Service. To respond

[GitHub] [camel-k] astefanutti opened a new pull request #2057: test: Add PodDisruptionBudget trait e2e tests

2021-02-19 Thread GitBox
astefanutti opened a new pull request #2057: URL: https://github.com/apache/camel-k/pull/2057 **Release Note** ```release-note NONE ``` This is an automated message from the Apache Git Service. To respond to the

[GitHub] [camel-quarkus] ppalaga opened a new issue #2273: Prefer SyntheticBeanBuildItem to initializing bean producers via volatile fields

2021-02-19 Thread GitBox
ppalaga opened a new issue #2273: URL: https://github.com/apache/camel-quarkus/issues/2273 There is couple of situations where we use the banned pattern: * AvroDataFormatProducer * FhirContextProducers * ReactiveStreamsProducers * CamelProducers * CamelMainProducers

[GitHub] [camel-k] astefanutti merged pull request #2055: feat: add command completion for connect flag

2021-02-19 Thread GitBox
astefanutti merged pull request #2055: URL: https://github.com/apache/camel-k/pull/2055 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [camel-quarkus] ppalaga opened a new pull request #2274: Next is 1.8.0

2021-02-19 Thread GitBox
ppalaga opened a new pull request #2274: URL: https://github.com/apache/camel-quarkus/pull/2274 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [camel-k] nicolaferraro opened a new pull request #2058: Implement a lightning-fast global strategy

2021-02-19 Thread GitBox
nicolaferraro opened a new pull request #2058: URL: https://github.com/apache/camel-k/pull/2058 Fix #1799 This is the best strategy I could think about, and it comes with no copies or redundant information. Basically, the behavior of Camel K is now the following when r

[GitHub] [camel-quarkus] zhfeng commented on issue #1918: Revisit transaction support

2021-02-19 Thread GitBox
zhfeng commented on issue #1918: URL: https://github.com/apache/camel-quarkus/issues/1918#issuecomment-782035392 @jamesnetherton can you take a review of the PR ? This is an automated message from the Apache Git Service. To r

[GitHub] [camel-k] lburgazzoli commented on pull request #2058: Implement a lightning-fast global strategy

2021-02-19 Thread GitBox
lburgazzoli commented on pull request #2058: URL: https://github.com/apache/camel-k/pull/2058#issuecomment-782039979 How does this strategy handle the case where you programmatic create an IntegrationKit ? This is an automat

[GitHub] [camel-k-runtime] lburgazzoli merged pull request #629: build(deps): bump testcontainers from 1.15.1 to 1.15.2

2021-02-19 Thread GitBox
lburgazzoli merged pull request #629: URL: https://github.com/apache/camel-k-runtime/pull/629 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [camel-kafka-connector] orpiske merged pull request #1038: Abstracted the CXF service client setup from the CXF test code

2021-02-19 Thread GitBox
orpiske merged pull request #1038: URL: https://github.com/apache/camel-kafka-connector/pull/1038 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [camel-k] astefanutti commented on pull request #2058: Implement a lightning-fast global strategy

2021-02-19 Thread GitBox
astefanutti commented on pull request #2058: URL: https://github.com/apache/camel-k/pull/2058#issuecomment-782051468 Woot! For future references, next steps towards the perfect build system is #592, and possibly #1784. This

[GitHub] [camel-k] astefanutti commented on issue #681: Build controller service / operator

2021-02-19 Thread GitBox
astefanutti commented on issue #681: URL: https://github.com/apache/camel-k/issues/681#issuecomment-782052024 Closing in favor of #1799. This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [camel-k] astefanutti closed issue #681: Build controller service / operator

2021-02-19 Thread GitBox
astefanutti closed issue #681: URL: https://github.com/apache/camel-k/issues/681 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [camel-k] astefanutti merged pull request #2040: feat(trait): toleration trait

2021-02-19 Thread GitBox
astefanutti merged pull request #2040: URL: https://github.com/apache/camel-k/pull/2040 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [camel-k] astefanutti commented on pull request #2040: feat(trait): toleration trait

2021-02-19 Thread GitBox
astefanutti commented on pull request #2040: URL: https://github.com/apache/camel-k/pull/2040#issuecomment-782056732 Thanks! This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [camel-k] nicolaferraro commented on pull request #2058: Implement a lightning-fast global strategy

2021-02-19 Thread GitBox
nicolaferraro commented on pull request #2058: URL: https://github.com/apache/camel-k/pull/2058#issuecomment-782080621 > How does this strategy handle the case where you programmatic create an IntegrationKit ? Well, no problems if the integrationKit is co-located with the integration

[GitHub] [camel-kafka-connector] oscerd opened a new pull request #1039: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd opened a new pull request #1039: URL: https://github.com/apache/camel-kafka-connector/pull/1039 Fixes #1030 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-k] nicolaferraro merged pull request #2048: Add local build command replacing local create

2021-02-19 Thread GitBox
nicolaferraro merged pull request #2048: URL: https://github.com/apache/camel-k/pull/2048 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [camel-k] nicolaferraro closed issue #2043: kamelet-binding: support for processing steps

2021-02-19 Thread GitBox
nicolaferraro closed issue #2043: URL: https://github.com/apache/camel-k/issues/2043 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [camel-k] nicolaferraro merged pull request #2044: Fix #2043: add processing steps to KameletBinding

2021-02-19 Thread GitBox
nicolaferraro merged pull request #2044: URL: https://github.com/apache/camel-k/pull/2044 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [camel-k] lburgazzoli commented on pull request #2058: Implement a lightning-fast global strategy

2021-02-19 Thread GitBox
lburgazzoli commented on pull request #2058: URL: https://github.com/apache/camel-k/pull/2058#issuecomment-782087407 > > How does this strategy handle the case where you programmatic create an IntegrationKit ? > > Well, no problems if the integrationKit is co-located with the integra

[GitHub] [camel-kamelets] nicolaferraro opened a new pull request #14: Add a webhook Kamelet

2021-02-19 Thread GitBox
nicolaferraro opened a new pull request #14: URL: https://github.com/apache/camel-kamelets/pull/14 And fix the sqs icon This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [camel-k-runtime] nicolaferraro opened a new issue #634: HTTP error (can't write to /deployments/file-uploads) when not setting content-type

2021-02-19 Thread GitBox
nicolaferraro opened a new issue #634: URL: https://github.com/apache/camel-k-runtime/issues/634 I've just added a webhook-source Kamelet in https://github.com/apache/camel-kamelets/pull/14. When bound with the following binding: ```yaml apiVersion: camel.apache.org/v1alpha

[GitHub] [camel-k-runtime] lburgazzoli commented on issue #634: HTTP error (can't write to /deployments/file-uploads) when not setting content-type

2021-02-19 Thread GitBox
lburgazzoli commented on issue #634: URL: https://github.com/apache/camel-k-runtime/issues/634#issuecomment-782116633 @ppalaga @jamesnetherton This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [camel-quarkus] ppalaga merged pull request #2274: Next is 1.8.0

2021-02-19 Thread GitBox
ppalaga merged pull request #2274: URL: https://github.com/apache/camel-quarkus/pull/2274 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [camel-kafka-connector] oscerd opened a new pull request #1040: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd opened a new pull request #1040: URL: https://github.com/apache/camel-kafka-connector/pull/1040 Related to #1030 This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [camel-kafka-connector] oscerd closed issue #1030: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd closed issue #1030: URL: https://github.com/apache/camel-kafka-connector/issues/1030 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [camel-kafka-connector] oscerd merged pull request #1039: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd merged pull request #1039: URL: https://github.com/apache/camel-kafka-connector/pull/1039 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [camel] k-jamroz opened a new pull request #5103: Fixed some camel-netty and camel-netty-http tests failing on Windows

2021-02-19 Thread GitBox
k-jamroz opened a new pull request #5103: URL: https://github.com/apache/camel/pull/5103 On Windows when the port is not opened one gets connection timeout instead of connection refused (may depend on firewall configuration). Made `NettyHttpRedeliveryTest` to work as intended in this case.

[GitHub] [camel-kafka-connector] oscerd merged pull request #1040: Generate just one connector package: .zip or .tar.gz

2021-02-19 Thread GitBox
oscerd merged pull request #1040: URL: https://github.com/apache/camel-kafka-connector/pull/1040 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [camel-kamelets] nicolaferraro merged pull request #14: Add a webhook Kamelet

2021-02-19 Thread GitBox
nicolaferraro merged pull request #14: URL: https://github.com/apache/camel-kamelets/pull/14 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [camel-kafka-connector] orpiske opened a new issue #1041: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
orpiske opened a new issue #1041: URL: https://github.com/apache/camel-kafka-connector/issues/1041 It looks like the connectors list on the website is broken. ![Screenshot_2021-02-19 Supported connectors and documentation Apache Camel](https://user-images.githubusercontent.com/185156

[GitHub] [camel-kafka-connector] oscerd commented on issue #1041: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd commented on issue #1041: URL: https://github.com/apache/camel-kafka-connector/issues/1041#issuecomment-782217788 I'll check this should be related to #1030 This is an automated message from the Apache Git Service. T

[GitHub] [camel-kafka-connector] oscerd opened a new pull request #1042: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd opened a new pull request #1042: URL: https://github.com/apache/camel-kafka-connector/pull/1042 Fixes #1041 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [camel-kafka-connector] oscerd closed issue #1041: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd closed issue #1041: URL: https://github.com/apache/camel-kafka-connector/issues/1041 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above t

[GitHub] [camel-kafka-connector] oscerd merged pull request #1042: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd merged pull request #1042: URL: https://github.com/apache/camel-kafka-connector/pull/1042 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [camel-kafka-connector] oscerd opened a new pull request #1043: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd opened a new pull request #1043: URL: https://github.com/apache/camel-kafka-connector/pull/1043 Related to #1041 This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [camel-kafka-connector] oscerd merged pull request #1043: Website: connectors' list formatting problems

2021-02-19 Thread GitBox
oscerd merged pull request #1043: URL: https://github.com/apache/camel-kafka-connector/pull/1043 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [camel-kafka-connector] ffang opened a new issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
ffang opened a new issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044 Source connectors basically will start a consumer endpoint "from uri", but currently no way to add a processor class after from. The processor is really flexible and can do pretty much e

[GitHub] [camel-kafka-connector] ffang opened a new pull request #1045: add processor configuration

2021-02-19 Thread GitBox
ffang opened a new pull request #1045: URL: https://github.com/apache/camel-kafka-connector/pull/1045 (cherry picked from commit 38b441c776272b02f531194479038c4f41e61b54) This is an automated message from the Apache Git Servi

[GitHub] [camel-kafka-connector] ffang commented on issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
ffang commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782285288 Open a PR here https://github.com/apache/camel-kafka-connector/pull/1045 This is an automated message

[GitHub] [camel-kafka-connector] oscerd commented on issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
oscerd commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782285478 This is out of the roadmap. In camel-kafka-connector there is no need for processors. If a user need a processor then he should use camel directly. Camel is the engine.

[GitHub] [camel-kafka-connector] ffang commented on issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
ffang commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782289661 Hi @oscerd , I may miss something, but could you please elaborate how to use camel directly? Because if I read the CKC code correctly, when initialising a sourc

[GitHub] [camel-kafka-connector] oscerd commented on issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
oscerd commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782292439 Basically the idea of ckc is avoid exposing Camel details and creating a tiny abstraction layers between kafka connect and camel. To use a processor people should know

[GitHub] [camel-kafka-connector] orpiske commented on issue #1044: add processor configuration for source connectors

2021-02-19 Thread GitBox
orpiske commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782299193 My understanding is in line w/ @oscerd's explanation. Personally, I'd prefer the project to abstract away some Camel features and patterns in order to offer a *

[GitHub] [camel-quarkus] github-actions[bot] opened a new issue #2275: [CI] - Quarkus Master Branch Build Failure

2021-02-19 Thread GitBox
github-actions[bot] opened a new issue #2275: URL: https://github.com/apache/camel-quarkus/issues/2275 The [quarkus-master](https://github.com/apache/camel-quarkus/tree/quarkus-master) branch build is failing: * Build ID: 583268750-303-cdc1570e-a730-491f-a8a2-6044c5063a40 * Camel

[GitHub] [camel] lburgazzoli merged pull request #5104: core: improve expression definition to ease the integration with DSLs

2021-02-20 Thread GitBox
lburgazzoli merged pull request #5104: URL: https://github.com/apache/camel/pull/5104 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

[GitHub] [camel-kafka-connector] oscerd commented on issue #1044: add processor configuration for source connectors

2021-02-20 Thread GitBox
oscerd commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782672831 If you need to do something like manipulate the message while moving from source to kafka, you should use an SMT or a converter (kafka concepts), introducing processors

[GitHub] [camel] davsclaus merged pull request #5103: Fixed some camel-netty and camel-netty-http tests failing on Windows

2021-02-20 Thread GitBox
davsclaus merged pull request #5103: URL: https://github.com/apache/camel/pull/5103 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [camel-kafka-connector] oscerd edited a comment on issue #1044: add processor configuration for source connectors

2021-02-20 Thread GitBox
oscerd edited a comment on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782292439 Basically the idea of ckc is avoid exposing Camel details and creating a tiny abstraction layers between kafka connect and camel. To use a processor people shoul

[GitHub] [camel] jameshilliard opened a new pull request #5106: Mark NatsEndpoint as supporting MultipleConsumers.

2021-02-20 Thread GitBox
jameshilliard opened a new pull request #5106: URL: https://github.com/apache/camel/pull/5106 Mark `NatsEndpoint` as supporting `MultipleConsumers`. Per nats documentation: NATS implements a publish-subscribe message distribution model for one-to-many communication.

[GitHub] [camel] jameshilliard commented on pull request #5106: Mark NatsEndpoint as supporting MultipleConsumers.

2021-02-20 Thread GitBox
jameshilliard commented on pull request #5106: URL: https://github.com/apache/camel/pull/5106#issuecomment-782710401 > This is not obvious and it doesn't depend from Nats, but from camel. Isn't it effectively a matter of whether or not it is safe to make multiple subscriptions from t

[GitHub] [camel] oscerd commented on pull request #5106: Mark NatsEndpoint as supporting MultipleConsumers.

2021-02-20 Thread GitBox
oscerd commented on pull request #5106: URL: https://github.com/apache/camel/pull/5106#issuecomment-782712169 Ah yeah, the NatsConsumer is using the executorService, so this is correct. My fault Sorry. This is an automated

[GitHub] [camel] oscerd merged pull request #5106: Mark NatsEndpoint as supporting MultipleConsumers.

2021-02-20 Thread GitBox
oscerd merged pull request #5106: URL: https://github.com/apache/camel/pull/5106 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [camel-k-runtime] davsclaus commented on issue #632: Functions to escape Kamelet parameters

2021-02-20 Thread GitBox
davsclaus commented on issue #632: URL: https://github.com/apache/camel-k-runtime/issues/632#issuecomment-782718978 I created a ticket about this problem as its a general camel-core problem https://issues.apache.org/jira/browse/CAMEL-16242 ---

[GitHub] [camel-k-runtime] davsclaus commented on pull request #633: Upgrade to camel-quarkus 1.7.0 (camel 3.8.x, quarkus 1.12.x)

2021-02-20 Thread GitBox
davsclaus commented on pull request #633: URL: https://github.com/apache/camel-k-runtime/pull/633#issuecomment-782719090 Oh you have been busy and productive This is an automated message from the Apache Git Service. To respon

[GitHub] [camel] jameshilliard opened a new pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-20 Thread GitBox
jameshilliard opened a new pull request #5107: URL: https://github.com/apache/camel/pull/5107 Fixes `NullPointerException` This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel] jameshilliard opened a new pull request #5108: Kamelet: remove null check for unused tid variable.

2021-02-20 Thread GitBox
jameshilliard opened a new pull request #5108: URL: https://github.com/apache/camel/pull/5108 This doesn't seem to be used. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel] davsclaus commented on pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-20 Thread GitBox
davsclaus commented on pull request #5107: URL: https://github.com/apache/camel/pull/5107#issuecomment-782742524 I wonder what example / use-case you had that could cause this NPE? I am asking because a camel route should always have 1 input. -

[GitHub] [camel] jameshilliard commented on pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-20 Thread GitBox
jameshilliard commented on pull request #5107: URL: https://github.com/apache/camel/pull/5107#issuecomment-782743758 > I wonder what example / use-case you had that could cause this NPE? I was messing around with route templates and hit this error, here's the reproducer: ```java

[GitHub] [camel] lburgazzoli merged pull request #5105: Improve expression definitions

2021-02-20 Thread GitBox
lburgazzoli merged pull request #5105: URL: https://github.com/apache/camel/pull/5105 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

[GitHub] [camel-quarkus] github-actions[bot] commented on issue #2275: [CI] - Quarkus Master Branch Build Failure

2021-02-20 Thread GitBox
github-actions[bot] commented on issue #2275: URL: https://github.com/apache/camel-quarkus/issues/2275#issuecomment-782783673 The [quarkus-master](https://github.com/apache/camel-quarkus/tree/quarkus-master) branch build is still failing: * Build ID: 585493900-304-1165df38-42d6-4d21

[GitHub] [camel-kafka-connector] tadayosi commented on issue #1044: add processor configuration for source connectors

2021-02-20 Thread GitBox
tadayosi commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782804218 I think it's a great discussion. Personally what @oscerd and @orpiske explained really makes sense: CKC is the complehensive collection of generic Kafka connectors ab

[GitHub] [camel] davsclaus commented on pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-21 Thread GitBox
davsclaus commented on pull request #5107: URL: https://github.com/apache/camel/pull/5107#issuecomment-782817886 Okay but as an end user you should not call `.asRouteDefinition()` its for internal use This is an automated me

[GitHub] [camel] davsclaus commented on pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-21 Thread GitBox
davsclaus commented on pull request #5107: URL: https://github.com/apache/camel/pull/5107#issuecomment-782817966 Instead of avoiding the NPE then you can update the PR to check that input is not null and report an error if so. --

[GitHub] [camel] jameshilliard commented on pull request #5107: Kamelet: Don't call getEndpointUri() on null input.

2021-02-21 Thread GitBox
jameshilliard commented on pull request #5107: URL: https://github.com/apache/camel/pull/5107#issuecomment-782818414 > Okay but as an end user you should not call `.asRouteDefinition()` its for internal use hmm, shouldn't it not be public if that is the case? --

[GitHub] [camel-kafka-connector] oscerd commented on issue #1044: add processor configuration for source connectors

2021-02-21 Thread GitBox
oscerd commented on issue #1044: URL: https://github.com/apache/camel-kafka-connector/issues/1044#issuecomment-782874796 I agree on 1. and 2., I disagree a bit on .3, because the connectors work exactly as the camel component behaves and they are generated directly from the components. So

[GitHub] [camel-quarkus] github-actions[bot] commented on issue #2275: [CI] - Quarkus Master Branch Build Failure

2021-02-21 Thread GitBox
github-actions[bot] commented on issue #2275: URL: https://github.com/apache/camel-quarkus/issues/2275#issuecomment-782995912 The [quarkus-master](https://github.com/apache/camel-quarkus/tree/quarkus-master) branch build is still failing: * Build ID: 587739592-305-8c451211-4525-4d5f

[GitHub] [camel-quarkus] jamesnetherton commented on a change in pull request #2272: Fix #1918 to add a integration test with camel-jdbc and camel-jms

2021-02-21 Thread GitBox
jamesnetherton commented on a change in pull request #2272: URL: https://github.com/apache/camel-quarkus/pull/2272#discussion_r580026798 ## File path: integration-tests/jta/pom.xml ## @@ -108,6 +142,7 @@ native + --re

[GitHub] [camel-k] bouskaJ opened a new pull request #2059: Move trait test to separate directory

2021-02-21 Thread GitBox
bouskaJ opened a new pull request #2059: URL: https://github.com/apache/camel-k/pull/2059 **Release Note** ```release-note NONE ``` This is an automated message from the Apache Git Ser

[GitHub] [camel-k] astefanutti opened a new pull request #2060: fix(api): Remove deprecated deepcopy-gen markers

2021-02-22 Thread GitBox
astefanutti opened a new pull request #2060: URL: https://github.com/apache/camel-k/pull/2060 **Release Note** ```release-note NONE ``` This is an automated message from the Apache Git Service. To respond to the

<    3   4   5   6   7   8   9   10   11   12   >