date:20220216

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from a8f30d5  Regen
 add 17477db  Fixed test after some smaller changes

No new revisions were added by this update.

Summary of changes:
 .../src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

[GitHub] [camel] JiriOndrusek edited a comment on pull request #6960: CAMEL-17658 camel-debezium: additionalProperties are trimmed



JiriOndrusek edited a comment on pull request #6960:
URL: https://github.com/apache/camel/pull/6960#issuecomment-1042651285


   The reason why it should work is fact, that it works in "non-specific 
debezium parent component (see 
https://github.com/apache/camel/blob/main/components/camel-debezium/camel-debezium-common/camel-debezium-common-component/src/test/java/org/apache/camel/component/debezium/DebeziumComponentTest.java#L48)"
   
   The difference is caused by the fact, that configurer is not used and the 
value is not trimmed. Therefore I think it should work in the same way with 
configurer.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] JiriOndrusek commented on pull request #6960: CAMEL-17658 camel-debezium: additionalProperties are trimmed



JiriOndrusek commented on pull request #6960:
URL: https://github.com/apache/camel/pull/6960#issuecomment-1042651285


   The reason why it should work is fact, that it works in "non-specific 
debezium parent component (see 
https://github.com/apache/camel/blob/main/components/camel-debezium/camel-debezium-common/camel-debezium-common-component/src/test/java/org/apache/camel/component/debezium/DebeziumComponentTest.java#L48)"
   
   The difference is caused by the fact, that thois parent component doesn't 
define `@UriParam(label = "common", prefix = "map.", multiValue = true`, so the 
**map** is not use configurer and the value is not trimmed. Therefore I think 
it should work in the same way with configurer.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] JiriOndrusek commented on pull request #6960: CAMEL-17658 camel-debezium: additionalProperties are trimmed



JiriOndrusek commented on pull request #6960:
URL: https://github.com/apache/camel/pull/6960#issuecomment-1042648448


   Yes, it is exacty the case. Value in the map is with dot (` 
"=CET"`). See 
https://github.com/apache/camel-quarkus/issues/3488.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (a4a5377 -> a8f30d5)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from a4a5377  CAMEL-17662: camel-jbang - Should setup compoenents to use 
for rest-dsl
 add e00b0ab  (chores) camel-tooling-model: remove commented code
 add dfedd1a  (chores) camel-tooling-model: replace duplicated check for 
file existence
 add d060d85  (chores) camel-tooling-model: use final when possible
 add 58a6fac  (chores) camel-csimple-maven-plugin: code cleanups
 add 49b460b  (chores) camel-api-component-maven-plugin: code cleanups
 add 7a95786  (chores) camel-eip-documentation-enricher-maven-plugin: 
removed unused exceptions
 add 5e59d49  enhance test (#6963)
 add a8f30d5  Regen

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/camel/maven/GenerateMojo.java  |   3 +-
 .../support/RouteWatcherReloadStrategyTest.java|  42 +++-
 .../org/apache/camel/tooling/model/Strings.java|  34 ---
 .../org/apache/camel/tooling/util/FileUtil.java|  13 ++-
 .../camel/maven/bom/generator/DependencySet.java   |   3 -
 .../bom/generator/ExternalBomConflictCheck.java|   3 -
 .../bom/generator/ExternalBomConflictCheckSet.java |   3 -
 .../maven/AbstractApiMethodGeneratorMojo.java  | 110 +
 .../camel/maven/ApiComponentGeneratorMojo.java |  20 
 .../org/apache/camel/maven/JavaSourceParser.java   |   4 +-
 .../camel/maven/EipDocumentationEnricherMojo.java  |   3 +-
 11 files changed, 163 insertions(+), 75 deletions(-)

[GitHub] [camel-quarkus-examples] jamesnetherton merged pull request #76: Dynamically generate native test matrix



jamesnetherton merged pull request #76:
URL: https://github.com/apache/camel-quarkus-examples/pull/76


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel-quarkus-examples] branch camel-quarkus-main updated: Dynamically generate native test matrix

2022-02-16 Thread jamesnetherton

This is an automated email from the ASF dual-hosted git repository.

jamesnetherton pushed a commit to branch camel-quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus-examples.git


The following commit(s) were added to refs/heads/camel-quarkus-main by this 
push:
 new 7fb72d4  Dynamically generate native test matrix
7fb72d4 is described below

commit 7fb72d472934797123ea521d123ae7d6739c3ff3
Author: James Netherton 
AuthorDate: Wed Feb 16 10:43:07 2022 +

Dynamically generate native test matrix
---
 .github/generate-test-groups.groovy | 43 +++
 .github/test-categories.yaml| 43 ---
 .github/workflows/ci-build.yaml | 45 +++--
 3 files changed, 66 insertions(+), 65 deletions(-)

diff --git a/.github/generate-test-groups.groovy 
b/.github/generate-test-groups.groovy
new file mode 100644
index 000..6987c2d
--- /dev/null
+++ b/.github/generate-test-groups.groovy
@@ -0,0 +1,43 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+import groovy.json.JsonOutput;
+
+final int MAX_GROUPS = 10
+final List> GROUPS = new ArrayList<>()
+int groupId = 0
+
+// Distribute example projects across a bounded set of test groups and output 
as JSON
+new File(".").eachFileRecurse { file ->
+if (file.getName() == "pom.xml" && file.text.contains("native")) {
+if (GROUPS[groupId] == null) {
+GROUPS[groupId] = [:]
+GROUPS[groupId].name = "group-${String.format("%02d", groupId + 
1)}"
+GROUPS[groupId].tests = ""
+}
+
+String separator = GROUPS[groupId].tests == "" ? "" : ","
+
+GROUPS[groupId].tests = 
"${GROUPS[groupId].tests}${separator}${file.parentFile.name}"
+
+groupId += 1;
+if (groupId == MAX_GROUPS) {
+groupId = 0
+}
+}
+}
+
+print JsonOutput.toJson(["include": GROUPS])
diff --git a/.github/test-categories.yaml b/.github/test-categories.yaml
deleted file mode 100644
index bdaec59..000
--- a/.github/test-categories.yaml
+++ /dev/null
@@ -1,43 +0,0 @@
-
-#
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements.  See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License.  You may obtain a copy of the License at
-#
-#  http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-# Association of categories and camel-quarkus-examples maven modules. Used by 
GitHub workflows.
-group-01:
-  - file-bindy-ftp
-  - file-split-log-xml
-group-02:
-  - health
-  - observability
-group-03:
-  - http-log
-  - rest-json
-group-04:
-  - aws-lambda
-  - timer-log
-  - timer-log-cdi
-group-05:
-  - timer-log-kotlin
-  - timer-log-main
-group-06:
-  - timer-log-spring
-  - timer-log-xml
-group-07:
-  - kafka
-  - kamelet-chucknorris
-group-08:
-  - jdbc-datasource
diff --git a/.github/workflows/ci-build.yaml b/.github/workflows/ci-build.yaml
index b040299..2727618 100644
--- a/.github/workflows/ci-build.yaml
+++ b/.github/workflows/ci-build.yaml
@@ -78,19 +78,6 @@ jobs:
 license:check \
 net.revelc.code.formatter:formatter-maven-plugin:validate \
 net.revelc.code:impsort-maven-plugin:check
-  - name: Validate test categories YAML
-run: |
-  VALID=true
-  for EXAMPLE in $(dirname $(grep -l "camel-quarkus-examples-" $(find 
. -name pom.xml)) | cut -f2 -d'/'); do
-if ! grep ${EXAMPLE} .github/test-categories.yaml > /dev/null; then
-  echo "Project ${EXAMPLE} is not listed in 
.github/test-categories.yaml"
-  VALID=false
-fi
-  done
-
-  if [[

[GitHub] [camel-karavan] mmuzikar commented on issue #192: Pressing Enter in "Create New Integration" dialog cancels, not confirms



mmuzikar commented on issue #192:
URL: https://github.com/apache/camel-karavan/issues/192#issuecomment-1042630324


   In the docker image `ghcr.io/apache/camel-karavan:latest`


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: Fixed test after some smaller changes

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new 17477db  Fixed test after some smaller changes
17477db is described below

commit 17477db3f83335d4ef76be07e1ad32d7eb0cd8db
Author: Claus Ibsen 
AuthorDate: Thu Feb 17 07:45:35 2022 +0100

Fixed test after some smaller changes
---
 .../src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/tooling/camel-tooling-util/src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java
 
b/tooling/camel-tooling-util/src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java
index 5384b0f..e94f542 100644
--- 
a/tooling/camel-tooling-util/src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java
+++ 
b/tooling/camel-tooling-util/src/test/java/org/apache/camel/tooling/util/PackageHelperTest.java
@@ -33,7 +33,7 @@ public class PackageHelperTest {
 @Test
 public void testFileToString() throws Exception {
 File file = ResourceUtils.getResourceAsFile("filecontent/a.txt");
-assertEquals("dk19i21)@+#(OR\n", PackageHelper.loadText(file));
+assertEquals("dk19i21)@+#(OR", PackageHelper.loadText(file));
 }
 
 @Test

[camel-quarkus] 01/02: Upgrade to Quarkus 2.8.0.CR1

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git

commit 84a0c1de40f3baf84de28e4efe11860c08804fb2
Author: James Netherton 
AuthorDate: Thu Jan 20 07:52:37 2022 +

Upgrade to Quarkus 2.8.0.CR1
---
 docs/antora.yml|  4 +--
 integration-tests/http/pom.xml |  2 +-
 .../kafka/oauth/it/container/KafkaContainer.java   | 30 +-
 integration-tests/vertx-websocket/pom.xml  |  2 +-
 pom.xml|  7 ++---
 poms/build-parent/pom.xml  |  9 +++
 6 files changed, 35 insertions(+), 19 deletions(-)

diff --git a/docs/antora.yml b/docs/antora.yml
index 1413755..1db4eff 100644
--- a/docs/antora.yml
+++ b/docs/antora.yml
@@ -31,8 +31,8 @@ asciidoc:
 target-maven-version: 3.8.4 # replace ${target-maven-version}
 camel-version: 3.15.0 # replace ${camel.version}
 camel-docs-version: 3.15.x # replace ${camel.docs.components.version}
-quarkus-version: 2.7.1.Final # replace ${quarkus.version}
-graalvm-version: 21.3.1 # replace ${graalvm.version}
+quarkus-version: 999-SNAPSHOT # replace ${quarkus.version}
+graalvm-version: 22.0.0.2 # replace ${graalvm.version}
 graalvm-docs-version: 21.3
 # attributes used in xrefs to other Antora components
 cq-camel-components: 3.15.x@components # replace 
${camel.docs.components.xref}
diff --git a/integration-tests/http/pom.xml b/integration-tests/http/pom.xml
index 0bb8a2a..793de20 100644
--- a/integration-tests/http/pom.xml
+++ b/integration-tests/http/pom.xml
@@ -61,7 +61,7 @@
 
 
 io.quarkus
-quarkus-undertow-websockets
+quarkus-websockets
 
 
 io.quarkus
diff --git 
a/integration-tests/kafka-oauth/src/test/java/org/apache/camel/quarkus/kafka/oauth/it/container/KafkaContainer.java
 
b/integration-tests/kafka-oauth/src/test/java/org/apache/camel/quarkus/kafka/oauth/it/container/KafkaContainer.java
index 31885bf..e95e062 100644
--- 
a/integration-tests/kafka-oauth/src/test/java/org/apache/camel/quarkus/kafka/oauth/it/container/KafkaContainer.java
+++ 
b/integration-tests/kafka-oauth/src/test/java/org/apache/camel/quarkus/kafka/oauth/it/container/KafkaContainer.java
@@ -16,17 +16,20 @@
  */
 package org.apache.camel.quarkus.kafka.oauth.it.container;
 
-import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
-import java.io.InputStreamReader;
+import java.io.StringReader;
 import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
+import javax.json.Json;
+import javax.json.stream.JsonParser;
+
 import com.github.dockerjava.api.command.InspectContainerResponse;
-import io.strimzi.StrimziKafkaContainer;
+import io.strimzi.test.container.StrimziKafkaContainer;
+import org.apache.commons.io.IOUtils;
 import org.jboss.logging.Logger;
 import org.testcontainers.containers.FixedHostPortGenericContainer;
 import org.testcontainers.containers.Network;
@@ -45,19 +48,22 @@ public class KafkaContainer extends 
FixedHostPortGenericContainer supportedKafkaVersions = new 
ArrayList<>(3);
+private static final List supportedKafkaVersions = new 
ArrayList<>();
 
 static {
-InputStream inputStream = 
StrimziKafkaContainer.class.getResourceAsStream("/kafka-versions.txt");
-InputStreamReader streamReader = new InputStreamReader(inputStream, 
StandardCharsets.UTF_8);
-
-try (BufferedReader bufferedReader = new BufferedReader(streamReader)) 
{
-String kafkaVersion;
-while ((kafkaVersion = bufferedReader.readLine()) != null) {
-supportedKafkaVersions.add(kafkaVersion);
+InputStream inputStream = 
StrimziKafkaContainer.class.getResourceAsStream("/kafka_versions.json");
+try {
+String json = IOUtils.toString(inputStream, 
StandardCharsets.UTF_8);
+try (JsonParser parser = Json.createParser(new 
StringReader(json))) {
+parser.next();
+parser.getObject()
+.get("kafkaVersions")
+.asJsonObject()
+.keySet()
+.forEach(supportedKafkaVersions::add);
 }
 } catch (IOException e) {
-LOGGER.error("Unable to load the supported Kafka versions", e);
+throw new RuntimeException(e);
 }
 
 // sort kafka version from low to high
diff --git a/integration-tests/vertx-websocket/pom.xml 
b/integration-tests/vertx-websocket/pom.xml
index 7947fe5..abd5e2a 100644
--- a/integration-tests/vertx-websocket/pom.xml
+++ b/integration-tests/vertx-websocket/pom.xml
@@ -39,7 +39,7 @@

 
 io.quarkus
-

[camel-quarkus] 02/02: Temporary hacks to handle version misalignment of smallrye-health and smallrye-faulttolerance in Quarkus & Camel

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git

commit 09b597026002f388784ae839388a90d97f5f1927
Author: James Netherton 
AuthorDate: Fri Feb 11 10:08:03 2022 +

Temporary hacks to handle version misalignment of smallrye-health and 
smallrye-faulttolerance in Quarkus & Camel
---
 .../MicroprofileFaultToleranceProcessor.java   |   9 +
 .../microprofile-fault-tolerance/runtime/pom.xml   |   2 +
 .../FaultToleranceConfiguration.java   | 120 +
 .../faulttolerance/FaultToleranceConstants.java}   |  21 +-
 .../faulttolerance/FaultToleranceProcessor.java| 536 +
 .../FaultToleranceProcessorFactory.java}   |  28 +-
 .../faulttolerance/FaultToleranceReifier.java  | 193 
 .../apache/camel/model/CircuitBreakerDefinition|  18 +
 .../deployment/MicroProfileHealthEnabledTest.java  |   4 +-
 .../runtime/CamelMicroProfileHealthCheck.java  |  67 +++
 .../runtime/CamelMicroProfileHealthHelper.java |  63 +++
 .../runtime/CamelMicroProfileHealthRecorder.java   |   3 +-
 .../CamelMicroProfileRepositoryHealthCheck.java|  72 +++
 ...amelQuarkusMicroProfileHealthCheckRegistry.java | 185 +++
 .../it/CoreFaultToleranceProducers.java|   4 +-
 15 files changed, 1287 insertions(+), 38 deletions(-)

diff --git 
a/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
 
b/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
index 05673b1..6e8c382 100644
--- 
a/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
+++ 
b/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
@@ -16,9 +16,13 @@
  */
 package 
org.apache.camel.quarkus.component.microprofile.fault.tolerance.deployment;
 
+import java.nio.file.Paths;
+
 import io.quarkus.deployment.annotations.BuildStep;
 import io.quarkus.deployment.builditem.FeatureBuildItem;
 import 
io.quarkus.deployment.builditem.nativeimage.NativeImageResourceBuildItem;
+import 
org.apache.camel.component.microprofile.faulttolerance.FaultToleranceProcessorFactory;
+import org.apache.camel.quarkus.core.deployment.spi.CamelServiceBuildItem;
 
 class MicroprofileFaultToleranceProcessor {
 
@@ -35,4 +39,9 @@ class MicroprofileFaultToleranceProcessor {
 
"META-INF/services/org/apache/camel/model/CircuitBreakerDefinition");
 }
 
+@BuildStep
+CamelServiceBuildItem camelCronServicePattern() {
+return new 
CamelServiceBuildItem(Paths.get("META-INF/services/org/apache/camel/model/CircuitBreakerDefinition"),
+FaultToleranceProcessorFactory.class.getName());
+}
 }
diff --git a/extensions/microprofile-fault-tolerance/runtime/pom.xml 
b/extensions/microprofile-fault-tolerance/runtime/pom.xml
index 3401f07..22e3962 100644
--- a/extensions/microprofile-fault-tolerance/runtime/pom.xml
+++ b/extensions/microprofile-fault-tolerance/runtime/pom.xml
@@ -56,10 +56,12 @@
 org.apache.camel.quarkus
 camel-quarkus-core
 
+
 
 
 
diff --git 
a/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
 
b/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
new file mode 100644
index 000..7cb3d4d
--- /dev/null
+++ 
b/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
@@ -0,0 +1,120 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package

[camel-quarkus] branch quarkus-main updated (4bb8c7c -> 09b5970)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git.


 discard 4bb8c7c  Temporary hacks to handle version misalignment of 
smallrye-health and smallrye-faulttolerance in Quarkus & Camel
 discard 69eef21  Upgrade to Quarkus 2.8.0.CR1
 add 0b7c1fd  Updated CHANGELOG.md
 add a7b0531  Fix #3548 to exclude geronimo-jta_1.1_spec in camel-activemq 
(#3550)
 add fcdff37  Updated CHANGELOG.md
 add 39de94a  Improve available port discovery in tests
 add aec0df1  gson: complete doc and test coverage #3556
 new 84a0c1d  Upgrade to Quarkus 2.8.0.CR1
 new 09b5970  Temporary hacks to handle version misalignment of 
smallrye-health and smallrye-faulttolerance in Quarkus & Camel

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (4bb8c7c)
\
 N -- N -- N   refs/heads/quarkus-main (09b5970)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG.md   |  5 +++
 .../ROOT/pages/reference/extensions/gson.adoc  |  9 +
 .../gson/runtime/src/main/doc/configuration.adoc   |  5 +++
 .../camel/quarkus/test/AvailablePortFinder.java| 40 --
 .../quarkus/component/as2/it/As2TestResource.java  |  1 +
 .../component/avro/rpc/it/AvroRpcTestResource.java |  1 +
 .../dataformats/json/JsonDataformatsResource.java  |  2 +-
 .../dataformats/json/JsonDataformatsRoute.java |  7 +++-
 .../dataformats/json/model/DummyObject.java| 12 +++
 .../dataformats/json/JsonComponentsTest.java   |  7 ++--
 .../quarkus/component/ftp/it/FtpTestResource.java  |  2 ++
 .../component/sftp/it/SftpTestResource.java|  2 ++
 .../component/grpc/it/GrpcServerTestResource.java  |  1 +
 .../quarkus/component/hl7/it/Hl7TestResource.java  |  2 +-
 .../component/http/it/HttpTestResource.java|  1 +
 .../component/hystrix/it/HystrixTestResource.java  |  1 +
 .../quarkus/component/lra/it/LraTestResource.java  |  1 +
 .../lumberjack/it/LumberjackTestResource.java  |  2 +-
 .../quarkus/component/mllp/it/MllpResource.java|  4 ++-
 .../quarkus/component/mllp/it/MllpRoutes.java  | 11 +++---
 .../camel/quarkus/component/mllp/it/MllpTest.java  |  2 ++
 .../component/mllp/it/MllpTestResource.java}   | 11 +++---
 .../component/nagios/it/NagiosTestResource.java|  1 +
 .../component/netty/tcp/NettyTcpTestResource.java  |  1 +
 .../component/oaipmh/it/MockOaipmhServer.java  |  1 +
 .../sap/netweaver/it/SapNetweaverTestResource.java |  1 +
 .../component/syslog/it/SyslogTestResource.java|  2 +-
 poms/bom/pom.xml   |  4 +++
 28 files changed, 118 insertions(+), 21 deletions(-)
 create mode 100644 extensions/gson/runtime/src/main/doc/configuration.adoc
 copy 
integration-tests/{syslog/src/test/java/org/apache/camel/quarkus/component/syslog/it/SyslogTestResource.java
 => 
mllp/src/test/java/org/apache/camel/quarkus/component/mllp/it/MllpTestResource.java}
 (76%)

[GitHub] [camel-quarkus] github-actions[bot] closed issue #2926: [CI] - Quarkus Main Branch Build Failure



github-actions[bot] closed issue #2926:
URL: https://github.com/apache/camel-quarkus/issues/2926


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

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



github-actions[bot] commented on issue #2926:
URL: https://github.com/apache/camel-quarkus/issues/2926#issuecomment-1042620721


   Build fixed with:
   
   * Camel Quarkus Commit: 09b597026002f388784ae839388a90d97f5f1927
   
   * Quarkus Main Commit: aec0df132e1bd3fdb0ff8287ff1a4725936324a6
   * Link to build: 
https://github.com/apache/camel-quarkus/actions/runs/1856339091


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: Regen

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new a8f30d5  Regen
a8f30d5 is described below

commit a8f30d5798bf04127ac3ac634fca34cb28f1f380
Author: Claus Ibsen 
AuthorDate: Thu Feb 17 07:22:04 2022 +0100

Regen
---
 .../org/apache/camel/support/RouteWatcherReloadStrategyTest.java| 6 +++---
 .../java/org/apache/camel/maven/EipDocumentationEnricherMojo.java   | 1 -
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
index 90a60f0..899a93d 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
@@ -41,8 +41,8 @@ public class RouteWatcherReloadStrategyTest extends 
ContextTestSupport {
 
 File[] fs = folder.listFiles(strategy.getFileFilter());
 assertNotNull(fs);
-assertEquals(1,fs.length);
-assertEquals("log4j2.properties",fs[0].getName());
+assertEquals(1, fs.length);
+assertEquals("log4j2.properties", fs[0].getName());
 }
 
 @Test
@@ -92,7 +92,7 @@ public class RouteWatcherReloadStrategyTest extends 
ContextTestSupport {
 
 File[] fs = folder.listFiles(strategy.getFileFilter());
 assertNotNull(fs);
-assertTrue(fs.length >= 40,String.valueOf(fs.length));
+assertTrue(fs.length >= 40, String.valueOf(fs.length));
 // null goes back to default
 assertEquals("*.yaml,*.xml", strategy.getPattern());
 }
diff --git 
a/tooling/maven/camel-eip-documentation-enricher-maven-plugin/src/main/java/org/apache/camel/maven/EipDocumentationEnricherMojo.java
 
b/tooling/maven/camel-eip-documentation-enricher-maven-plugin/src/main/java/org/apache/camel/maven/EipDocumentationEnricherMojo.java
index 2467dff..b48691f 100644
--- 
a/tooling/maven/camel-eip-documentation-enricher-maven-plugin/src/main/java/org/apache/camel/maven/EipDocumentationEnricherMojo.java
+++ 
b/tooling/maven/camel-eip-documentation-enricher-maven-plugin/src/main/java/org/apache/camel/maven/EipDocumentationEnricherMojo.java
@@ -42,7 +42,6 @@ import org.apache.camel.util.FileUtil;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
-import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.plugins.annotations.LifecyclePhase;
 import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;

[GitHub] [camel-karavan] davsclaus commented on issue #194: Error on deploy in serverless mode



davsclaus commented on issue #194:
URL: https://github.com/apache/camel-karavan/issues/194#issuecomment-1042613685


   No there will not be


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-k] oscerd commented on issue #2983: Release 1.8.1



oscerd commented on issue #2983:
URL: https://github.com/apache/camel-k/issues/2983#issuecomment-1042612845


   Is there something missing? I can start the release on Monday eventually.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: enhance test (#6963)

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new 5e59d49  enhance test (#6963)
5e59d49 is described below

commit 5e59d49484e40f2be3e89df770bf8167a5c7a55a
Author: JP Moresmau 
AuthorDate: Thu Feb 17 07:02:16 2022 +0100

enhance test (#6963)
---
 .../support/RouteWatcherReloadStrategyTest.java| 42 +++---
 1 file changed, 38 insertions(+), 4 deletions(-)

diff --git 
a/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
index f0d24ce..90a60f0 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/support/RouteWatcherReloadStrategyTest.java
@@ -29,7 +29,41 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 public class RouteWatcherReloadStrategyTest extends ContextTestSupport {
 
 @Test
-public void testBasePath() throws Exception {
+public void testBasePathExact() throws Exception {
+RouteWatcherReloadStrategy strategy = new 
RouteWatcherReloadStrategy("./src/test/resources");
+strategy.setPattern("log4j2.properties");
+strategy.setCamelContext(context);
+strategy.doStart();
+
+assertNotNull(strategy.getFileFilter());
+File folder = new File("./src/test/resources");
+assertTrue(folder.isDirectory());
+
+File[] fs = folder.listFiles(strategy.getFileFilter());
+assertNotNull(fs);
+assertEquals(1,fs.length);
+assertEquals("log4j2.properties",fs[0].getName());
+}
+
+@Test
+public void testBasePathWildcardExtension() throws Exception {
+RouteWatcherReloadStrategy strategy = new 
RouteWatcherReloadStrategy("./src/test/resources");
+strategy.setPattern("*.properties");
+strategy.setCamelContext(context);
+strategy.doStart();
+
+assertNotNull(strategy.getFileFilter());
+File folder = new File("./src/test/resources");
+assertTrue(folder.isDirectory());
+
+File[] fs = folder.listFiles(strategy.getFileFilter());
+assertNotNull(fs);
+assertTrue(fs.length >= 5);
+assertTrue(Arrays.stream(fs).anyMatch(f -> 
f.getName().equals("log4j2.properties")));
+}
+
+@Test
+public void testBasePathFullWildcard() throws Exception {
 RouteWatcherReloadStrategy strategy = new 
RouteWatcherReloadStrategy("./src/test/resources");
 strategy.setPattern("*");
 strategy.setCamelContext(context);
@@ -47,18 +81,18 @@ public class RouteWatcherReloadStrategyTest extends 
ContextTestSupport {
 
 @Test
 public void testNullPattern() throws Exception {
-RouteWatcherReloadStrategy strategy = new 
RouteWatcherReloadStrategy("./src/test/resources");
+RouteWatcherReloadStrategy strategy = new 
RouteWatcherReloadStrategy("./src/test/resources/org/apache/camel/model");
 strategy.setPattern(null);
 strategy.setCamelContext(context);
 strategy.doStart();
 
 assertNotNull(strategy.getFileFilter());
-File folder = new File("./src/test/resources");
+File folder = new File("./src/test/resources/org/apache/camel/model");
 assertTrue(folder.isDirectory());
 
 File[] fs = folder.listFiles(strategy.getFileFilter());
 assertNotNull(fs);
-assertEquals(0, fs.length);
+assertTrue(fs.length >= 40,String.valueOf(fs.length));
 // null goes back to default
 assertEquals("*.yaml,*.xml", strategy.getPattern());
 }

[GitHub] [camel] davsclaus merged pull request #6963: [CAMEL-17651] enhance test



davsclaus merged pull request #6963:
URL: https://github.com/apache/camel/pull/6963


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel-quarkus] branch camel-main updated (21b9941 -> c3401c6)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch camel-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git.


 discard 21b9941  Temporary hacks to handle version misalignment of 
smallrye-health and smallrye-faulttolerance in Quarkus & Camel
 discard df29585  Upgrade Camel to 3.16.0
 add a7b0531  Fix #3548 to exclude geronimo-jta_1.1_spec in camel-activemq 
(#3550)
 add fcdff37  Updated CHANGELOG.md
 add 39de94a  Improve available port discovery in tests
 add aec0df1  gson: complete doc and test coverage #3556
 new 5eca097  Upgrade Camel to 3.16.0
 new c3401c6  Temporary hacks to handle version misalignment of 
smallrye-health and smallrye-faulttolerance in Quarkus & Camel

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (21b9941)
\
 N -- N -- N   refs/heads/camel-main (c3401c6)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG.md   |  2 ++
 .../ROOT/pages/reference/extensions/gson.adoc  |  9 +
 .../gson/runtime/src/main/doc/configuration.adoc   |  5 +++
 .../camel/quarkus/test/AvailablePortFinder.java| 40 --
 .../quarkus/component/as2/it/As2TestResource.java  |  1 +
 .../component/avro/rpc/it/AvroRpcTestResource.java |  1 +
 .../dataformats/json/JsonDataformatsResource.java  |  2 +-
 .../dataformats/json/JsonDataformatsRoute.java |  7 +++-
 .../dataformats/json/model/DummyObject.java| 12 +++
 .../dataformats/json/JsonComponentsTest.java   |  7 ++--
 .../quarkus/component/ftp/it/FtpTestResource.java  |  2 ++
 .../component/sftp/it/SftpTestResource.java|  2 ++
 .../component/grpc/it/GrpcServerTestResource.java  |  1 +
 .../quarkus/component/hl7/it/Hl7TestResource.java  |  2 +-
 .../component/http/it/HttpTestResource.java|  1 +
 .../component/hystrix/it/HystrixTestResource.java  |  1 +
 .../quarkus/component/lra/it/LraTestResource.java  |  1 +
 .../lumberjack/it/LumberjackTestResource.java  |  2 +-
 .../quarkus/component/mllp/it/MllpResource.java|  4 ++-
 .../quarkus/component/mllp/it/MllpRoutes.java  | 11 +++---
 .../camel/quarkus/component/mllp/it/MllpTest.java  |  2 ++
 .../component/mllp/it/MllpTestResource.java}   | 11 +++---
 .../component/nagios/it/NagiosTestResource.java|  1 +
 .../component/netty/tcp/NettyTcpTestResource.java  |  1 +
 .../component/oaipmh/it/MockOaipmhServer.java  |  1 +
 .../sap/netweaver/it/SapNetweaverTestResource.java |  1 +
 .../component/syslog/it/SyslogTestResource.java|  2 +-
 poms/bom/pom.xml   |  4 +++
 28 files changed, 115 insertions(+), 21 deletions(-)
 create mode 100644 extensions/gson/runtime/src/main/doc/configuration.adoc
 copy 
integration-tests/{syslog/src/test/java/org/apache/camel/quarkus/component/syslog/it/SyslogTestResource.java
 => 
mllp/src/test/java/org/apache/camel/quarkus/component/mllp/it/MllpTestResource.java}
 (76%)

[camel-quarkus] 01/02: Upgrade Camel to 3.16.0

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch camel-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git

commit 5eca0979f1ad0809f9cd1d6d3b86c072d69c6573
Author: James Netherton 
AuthorDate: Wed Feb 9 11:13:37 2022 +

Upgrade Camel to 3.16.0
---
 docs/antora.yml|  6 +-
 .../ROOT/pages/reference/extensions/yaml-dsl.adoc  |  6 --
 .../it/health/CustomHealthCheckRepository.java | 17 --
 .../microprofile/it/health/FailingHealthCheck.java |  2 +-
 .../it/health/FailureThresholdHealthCheck.java | 51 
 .../it/health/MicroProfileHealthResource.java  | 17 +-
 .../src/main/resources/application.properties  |  7 +--
 .../it/health/MicroProfileHealthTest.java  | 67 --
 pom.xml|  6 +-
 9 files changed, 9 insertions(+), 170 deletions(-)

diff --git a/docs/antora.yml b/docs/antora.yml
index 1413755..0d85853 100644
--- a/docs/antora.yml
+++ b/docs/antora.yml
@@ -29,11 +29,11 @@ asciidoc:
 
 min-maven-version: 3.8.1 # replace ${min-maven-version}
 target-maven-version: 3.8.4 # replace ${target-maven-version}
-camel-version: 3.15.0 # replace ${camel.version}
-camel-docs-version: 3.15.x # replace ${camel.docs.components.version}
+camel-version: 3.16.0-SNAPSHOT # replace ${camel.version}
+camel-docs-version: 3.16.x # replace ${camel.docs.components.version}
 quarkus-version: 2.7.1.Final # replace ${quarkus.version}
 graalvm-version: 21.3.1 # replace ${graalvm.version}
 graalvm-docs-version: 21.3
 # attributes used in xrefs to other Antora components
-cq-camel-components: 3.15.x@components # replace 
${camel.docs.components.xref}
+cq-camel-components: 3.16.x@components # replace 
${camel.docs.components.xref}
 quarkus-examples-version: latest
diff --git a/docs/modules/ROOT/pages/reference/extensions/yaml-dsl.adoc 
b/docs/modules/ROOT/pages/reference/extensions/yaml-dsl.adoc
index bdde5d3..26484a9 100644
--- a/docs/modules/ROOT/pages/reference/extensions/yaml-dsl.adoc
+++ b/docs/modules/ROOT/pages/reference/extensions/yaml-dsl.adoc
@@ -16,12 +16,6 @@
 
 An YAML stack for parsing YAML route definitions
 
-== What's inside
-
-* xref:{cq-camel-components}:others:yaml-dsl.adoc[YAML DSL]
-
-Please refer to the above link for usage and configuration details.
-
 == Maven coordinates
 
 https://code.quarkus.io/?extension-search=camel-quarkus-yaml-dsl[Create a new 
project with this extension on code.quarkus.io, window="_blank"]
diff --git 
a/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/CustomHealthCheckRepository.java
 
b/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/CustomHealthCheckRepository.java
index c54665d..2d31330 100644
--- 
a/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/CustomHealthCheckRepository.java
+++ 
b/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/CustomHealthCheckRepository.java
@@ -16,12 +16,10 @@
  */
 package org.apache.camel.quarkus.component.microprofile.it.health;
 
-import java.util.Collections;
 import java.util.Map;
 import java.util.stream.Stream;
 
 import org.apache.camel.health.HealthCheck;
-import org.apache.camel.health.HealthCheckConfiguration;
 import org.apache.camel.health.HealthCheckRepository;
 import org.apache.camel.health.HealthCheckResultBuilder;
 import org.apache.camel.impl.health.AbstractHealthCheck;
@@ -44,21 +42,6 @@ public class CustomHealthCheckRepository implements 
HealthCheckRepository {
 }
 
 @Override
-public void setConfigurations(Map 
configurations) {
-// Noop
-}
-
-@Override
-public Map getConfigurations() {
-return Collections.emptyMap();
-}
-
-@Override
-public void addConfiguration(String id, HealthCheckConfiguration 
configuration) {
-// Noop
-}
-
-@Override
 public Stream stream() {
 return Stream.of(check);
 }
diff --git 
a/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/FailingHealthCheck.java
 
b/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/FailingHealthCheck.java
index 2d56d54..b278052 100644
--- 
a/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/FailingHealthCheck.java
+++ 
b/integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/FailingHealthCheck.java
@@ -25,7 +25,7 @@ public class FailingHealthCheck extends AbstractHealthCheck {
 
 public FailingHealthCheck() {
 super("failing-check");
-getConfiguration().setEnabled(false);
+

[camel-quarkus] 02/02: Temporary hacks to handle version misalignment of smallrye-health and smallrye-faulttolerance in Quarkus & Camel

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch camel-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git

commit c3401c6a0b4c186ec095132e48a945bdc33b029a
Author: James Netherton 
AuthorDate: Fri Feb 11 10:08:03 2022 +

Temporary hacks to handle version misalignment of smallrye-health and 
smallrye-faulttolerance in Quarkus & Camel
---
 .../MicroprofileFaultToleranceProcessor.java   |   9 +
 .../microprofile-fault-tolerance/runtime/pom.xml   |   2 +
 .../FaultToleranceConfiguration.java   | 120 +
 .../faulttolerance/FaultToleranceConstants.java}   |  21 +-
 .../faulttolerance/FaultToleranceProcessor.java| 536 +
 .../FaultToleranceProcessorFactory.java}   |  28 +-
 .../faulttolerance/FaultToleranceReifier.java  | 193 
 .../apache/camel/model/CircuitBreakerDefinition|  18 +
 .../runtime/CamelMicroProfileHealthRecorder.java   |   3 +-
 ...amelQuarkusMicroProfileHealthCheckRegistry.java |  46 ++
 10 files changed, 942 insertions(+), 34 deletions(-)

diff --git 
a/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
 
b/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
index 05673b1..6e8c382 100644
--- 
a/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
+++ 
b/extensions/microprofile-fault-tolerance/deployment/src/main/java/org/apache/camel/quarkus/component/microprofile/fault/tolerance/deployment/MicroprofileFaultToleranceProcessor.java
@@ -16,9 +16,13 @@
  */
 package 
org.apache.camel.quarkus.component.microprofile.fault.tolerance.deployment;
 
+import java.nio.file.Paths;
+
 import io.quarkus.deployment.annotations.BuildStep;
 import io.quarkus.deployment.builditem.FeatureBuildItem;
 import 
io.quarkus.deployment.builditem.nativeimage.NativeImageResourceBuildItem;
+import 
org.apache.camel.component.microprofile.faulttolerance.FaultToleranceProcessorFactory;
+import org.apache.camel.quarkus.core.deployment.spi.CamelServiceBuildItem;
 
 class MicroprofileFaultToleranceProcessor {
 
@@ -35,4 +39,9 @@ class MicroprofileFaultToleranceProcessor {
 
"META-INF/services/org/apache/camel/model/CircuitBreakerDefinition");
 }
 
+@BuildStep
+CamelServiceBuildItem camelCronServicePattern() {
+return new 
CamelServiceBuildItem(Paths.get("META-INF/services/org/apache/camel/model/CircuitBreakerDefinition"),
+FaultToleranceProcessorFactory.class.getName());
+}
 }
diff --git a/extensions/microprofile-fault-tolerance/runtime/pom.xml 
b/extensions/microprofile-fault-tolerance/runtime/pom.xml
index 3401f07..22e3962 100644
--- a/extensions/microprofile-fault-tolerance/runtime/pom.xml
+++ b/extensions/microprofile-fault-tolerance/runtime/pom.xml
@@ -56,10 +56,12 @@
 org.apache.camel.quarkus
 camel-quarkus-core
 
+
 
 
 
diff --git 
a/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
 
b/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
new file mode 100644
index 000..7cb3d4d
--- /dev/null
+++ 
b/extensions/microprofile-fault-tolerance/runtime/src/main/java/org/apache/camel/component/microprofile/faulttolerance/FaultToleranceConfiguration.java
@@ -0,0 +1,120 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.microprofile.faulttolerance;
+
+public class FaultToleranceConfiguration {
+
+private long delay;
+private int successThreshold;
+private int requestVolumeThreshold;
+private float failureRatio;
+private boolean timeoutEnabled;
+private long timeoutDuration;
+private

[camel-quarkus] branch main updated: Updated CHANGELOG.md

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git


The following commit(s) were added to refs/heads/main by this push:
 new bf18e48  Updated CHANGELOG.md
bf18e48 is described below

commit bf18e48a4d16d560569f796f45a456ddfb6c4390
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Feb 17 03:12:10 2022 +

Updated CHANGELOG.md
---
 CHANGELOG.md | 5 +
 1 file changed, 5 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 94c757a..7501cc8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,6 +10,7 @@
 
 **Closed issues:**
 
+- How do you configure an InterceptStrategy in Quarkus? 
[\#3552](https://github.com/apache/camel-quarkus/issues/3552)
 - LinkageError when using camel-quarkus-activemq together with quarkus-agroal 
[\#3548](https://github.com/apache/camel-quarkus/issues/3548)
 - Incorrect version of `quarkus-maven-plugin` may be resolved 
[\#3520](https://github.com/apache/camel-quarkus/issues/3520)
 - Prevent various plugins from resolving commons-logging:commons-logging 
[\#3517](https://github.com/apache/camel-quarkus/issues/3517)
@@ -19,8 +20,10 @@
 - deadLetterChannel does not work in BaseRouteBuilder configure method 
[\#3436](https://github.com/apache/camel-quarkus/issues/3436)
 - \[Camel 3.15\] Fhir tests are failing 
[\#3416](https://github.com/apache/camel-quarkus/issues/3416)
 - Better integration with Quarkus OpenAPI & OpenAPI Swagger UI 
[\#3346](https://github.com/apache/camel-quarkus/issues/3346)
+- Test examples on Windows 
[\#3343](https://github.com/apache/camel-quarkus/issues/3343)
 - Run integration tests on Windows 
[\#3342](https://github.com/apache/camel-quarkus/issues/3342)
 - `DebeziumMongodbTest` skipped tests 
[\#3213](https://github.com/apache/camel-quarkus/issues/3213)
+- Improve available port discovery in tests 
[\#3199](https://github.com/apache/camel-quarkus/issues/3199)
 - spark:hive is does not work 
[\#1956](https://github.com/apache/camel-quarkus/issues/1956)
 - spark:dataframe does not work on Java 11 
[\#1955](https://github.com/apache/camel-quarkus/issues/1955)
 - \[Dependency hell\] Spark native support 
[\#1701](https://github.com/apache/camel-quarkus/issues/1701)
@@ -30,6 +33,8 @@
 
 **Merged pull requests:**
 
+- gson: complete doc and test coverage \#3556 
[\#3557](https://github.com/apache/camel-quarkus/pull/3557) 
([aldettinger](https://github.com/aldettinger))
+- Improve available port discovery in tests 
[\#3555](https://github.com/apache/camel-quarkus/pull/3555) 
([jamesnetherton](https://github.com/jamesnetherton))
 - Fix \#3548 to exclude geronimo-jta\_1.1\_spec in camel-activemq 
[\#3550](https://github.com/apache/camel-quarkus/pull/3550) 
([zhfeng](https://github.com/zhfeng))
 - Fix link to Quarkus Freemarker docs 
[\#3547](https://github.com/apache/camel-quarkus/pull/3547) 
([jamesnetherton](https://github.com/jamesnetherton))
 - Fixup e72113e send optaplanner test messages asynchronously 
[\#3546](https://github.com/apache/camel-quarkus/pull/3546) 
([jamesnetherton](https://github.com/jamesnetherton))

[camel-k] branch main updated: Updated CHANGELOG.md

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k.git


The following commit(s) were added to refs/heads/main by this push:
 new 4b3cafb  Updated CHANGELOG.md
4b3cafb is described below

commit 4b3cafb5f6adf6cc67698b432cf1136c024e9c0c
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Feb 17 03:09:49 2022 +

Updated CHANGELOG.md
---
 CHANGELOG.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7ddb8e1..75550e5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -22,7 +22,6 @@
 - Kamelet with MQTT Broker Mosquitto 
[\#2690](https://github.com/apache/camel-k/issues/2690)
 - Unable to access MongoDB 
[\#2684](https://github.com/apache/camel-k/issues/2684)
 - KAMEL\_INSTALL\_DEFAULT\_KAMELETS behavior 
[\#2560](https://github.com/apache/camel-k/issues/2560)
-- Documentation for `kamel local run`? 
[\#2490](https://github.com/apache/camel-k/issues/2490)
 - Log warnings with kamelets 
[\#2395](https://github.com/apache/camel-k/issues/2395)
 - Json marshalling is needed for Rest route and uncommented code seems to be 
executed [\#2341](https://github.com/apache/camel-k/issues/2341)
 - build: use golangci/golangci-lint-action 
[\#2334](https://github.com/apache/camel-k/issues/2334)

[GitHub] [camel-k] github-actions[bot] commented on issue #2752: Add official documentation on Quarkus Native introduced in 1.7



github-actions[bot] commented on issue #2752:
URL: https://github.com/apache/camel-k/issues/2752#issuecomment-1042430977


   This issue has been automatically marked as stale due to 90 days of 
inactivity. 
   It will be closed if no further activity occurs within 15 days.
   If you think that’s incorrect or the issue should never stale, please simply 
write any comment.
   Thanks for your contributions!


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-k] github-actions[bot] commented on issue #2758: Add official documentation on Kustomize introduced in 1.7



github-actions[bot] commented on issue #2758:
URL: https://github.com/apache/camel-k/issues/2758#issuecomment-1042430965


   This issue has been automatically marked as stale due to 90 days of 
inactivity. 
   It will be closed if no further activity occurs within 15 days.
   If you think that’s incorrect or the issue should never stale, please simply 
write any comment.
   Thanks for your contributions!


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] ppalaga merged pull request #3557: gson: complete doc and test coverage #3556



ppalaga merged pull request #3557:
URL: https://github.com/apache/camel-quarkus/pull/3557


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel-quarkus] branch main updated: gson: complete doc and test coverage #3556

2022-02-16 Thread ppalaga

This is an automated email from the ASF dual-hosted git repository.

ppalaga pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git


The following commit(s) were added to refs/heads/main by this push:
 new aec0df1  gson: complete doc and test coverage #3556
aec0df1 is described below

commit aec0df132e1bd3fdb0ff8287ff1a4725936324a6
Author: aldettinger 
AuthorDate: Wed Feb 16 16:00:36 2022 +0100

gson: complete doc and test coverage #3556
---
 docs/modules/ROOT/pages/reference/extensions/gson.adoc   |  9 +
 extensions/gson/runtime/src/main/doc/configuration.adoc  |  5 +
 .../component/dataformats/json/JsonDataformatsResource.java  |  2 +-
 .../component/dataformats/json/JsonDataformatsRoute.java |  7 ++-
 .../component/dataformats/json/model/DummyObject.java| 12 
 .../component/dataformats/json/JsonComponentsTest.java   |  7 +--
 6 files changed, 38 insertions(+), 4 deletions(-)

diff --git a/docs/modules/ROOT/pages/reference/extensions/gson.adoc 
b/docs/modules/ROOT/pages/reference/extensions/gson.adoc
index cd7f437..bee02a5 100644
--- a/docs/modules/ROOT/pages/reference/extensions/gson.adoc
+++ b/docs/modules/ROOT/pages/reference/extensions/gson.adoc
@@ -38,3 +38,12 @@ Or add the coordinates to your existing project:
 
 
 Check the xref:user-guide/index.adoc[User guide] for more information about 
writing Camel Quarkus applications.
+
+== Additional Camel Quarkus configuration
+
+=== Marshaling/Unmarshaling objects in native mode
+
+When marshaling/unmarshaling objects in native mode, all the serialized 
classes need to be 
link:https://quarkus.io/guides/writing-native-applications-tips#registering-for-reflection[registered
 for reflection].
+As such, when using `GsonDataFormat.setUnmarshalType(...)`, 
`GsonDataFormat.setUnmarshalTypeName(...)` and even 
`GsonDataFormat.setUnmarshalGenericType(...)`, the unmarshal type as well as 
sub field types should be registered for reflection.
+See a working example in this 
link:https://github.com/apache/camel-quarkus/blob/main/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/model/PojoA.java[integration
 test].
+
diff --git a/extensions/gson/runtime/src/main/doc/configuration.adoc 
b/extensions/gson/runtime/src/main/doc/configuration.adoc
new file mode 100644
index 000..dcb5888
--- /dev/null
+++ b/extensions/gson/runtime/src/main/doc/configuration.adoc
@@ -0,0 +1,5 @@
+=== Marshaling/Unmarshaling objects in native mode
+
+When marshaling/unmarshaling objects in native mode, all the serialized 
classes need to be 
link:https://quarkus.io/guides/writing-native-applications-tips#registering-for-reflection[registered
 for reflection].
+As such, when using `GsonDataFormat.setUnmarshalType(...)`, 
`GsonDataFormat.setUnmarshalTypeName(...)` and even 
`GsonDataFormat.setUnmarshalGenericType(...)`, the unmarshal type as well as 
sub field types should be registered for reflection.
+See a working example in this 
link:https://github.com/apache/camel-quarkus/blob/main/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/model/PojoA.java[integration
 test].
\ No newline at end of file
diff --git 
a/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsResource.java
 
b/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsResource.java
index 6f51cd8..b2c6bb1 100644
--- 
a/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsResource.java
+++ 
b/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsResource.java
@@ -56,7 +56,7 @@ public class JsonDataformatsResource {
 
 @Path("/out")
 @POST
-@Produces(MediaType.TEXT_PLAIN)
+@Produces(MediaType.APPLICATION_JSON)
 public String testOrder(@QueryParam("json-component") String 
jsonComponent) {
 LOG.infof("Invoking testOrder(%s)", jsonComponent);
 return consumerTemplate.receive("vm:" + jsonComponent + 
"-out").getMessage().getBody().toString();
diff --git 
a/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsRoute.java
 
b/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsRoute.java
index b5bc061..13a7831 100644
--- 
a/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsRoute.java
+++ 
b/integration-tests/dataformats-json/src/main/java/org/apache/camel/quarkus/component/dataformats/json/JsonDataformatsRoute.java
@@ -65,6 +65,7 @@ public class JsonDataformatsRoute extends RouteBuilder {
 Type genericType = new TypeToken>() {

[camel] branch main updated (a4a5377 -> 7a95786)

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git.


from a4a5377  CAMEL-17662: camel-jbang - Should setup compoenents to use 
for rest-dsl
 add e00b0ab  (chores) camel-tooling-model: remove commented code
 add dfedd1a  (chores) camel-tooling-model: replace duplicated check for 
file existence
 add d060d85  (chores) camel-tooling-model: use final when possible
 add 58a6fac  (chores) camel-csimple-maven-plugin: code cleanups
 add 49b460b  (chores) camel-api-component-maven-plugin: code cleanups
 add 7a95786  (chores) camel-eip-documentation-enricher-maven-plugin: 
removed unused exceptions

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/camel/maven/GenerateMojo.java  |   3 +-
 .../org/apache/camel/tooling/model/Strings.java|  34 ---
 .../org/apache/camel/tooling/util/FileUtil.java|  13 ++-
 .../camel/maven/bom/generator/DependencySet.java   |   3 -
 .../bom/generator/ExternalBomConflictCheck.java|   3 -
 .../bom/generator/ExternalBomConflictCheckSet.java |   3 -
 .../maven/AbstractApiMethodGeneratorMojo.java  | 110 +
 .../camel/maven/ApiComponentGeneratorMojo.java |  20 
 .../org/apache/camel/maven/JavaSourceParser.java   |   4 +-
 .../camel/maven/EipDocumentationEnricherMojo.java  |   2 +-
 10 files changed, 125 insertions(+), 70 deletions(-)

[GitHub] [camel] orpiske merged pull request #6961: Maven plugin cleanups



orpiske merged pull request #6961:
URL: https://github.com/apache/camel/pull/6961


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel-quarkus] branch main updated (fcdff37 -> 39de94a)

2022-02-16 Thread jamesnetherton

This is an automated email from the ASF dual-hosted git repository.

jamesnetherton pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git.


from fcdff37  Updated CHANGELOG.md
 add 39de94a  Improve available port discovery in tests

No new revisions were added by this update.

Summary of changes:
 .../camel/quarkus/test/AvailablePortFinder.java| 40 --
 .../quarkus/component/as2/it/As2TestResource.java  |  1 +
 .../component/avro/rpc/it/AvroRpcTestResource.java |  1 +
 .../quarkus/component/ftp/it/FtpTestResource.java  |  2 ++
 .../component/sftp/it/SftpTestResource.java|  2 ++
 .../component/grpc/it/GrpcServerTestResource.java  |  1 +
 .../quarkus/component/hl7/it/Hl7TestResource.java  |  2 +-
 .../component/http/it/HttpTestResource.java|  1 +
 .../component/hystrix/it/HystrixTestResource.java  |  1 +
 .../quarkus/component/lra/it/LraTestResource.java  |  1 +
 .../lumberjack/it/LumberjackTestResource.java  |  2 +-
 .../quarkus/component/mllp/it/MllpResource.java|  4 ++-
 .../quarkus/component/mllp/it/MllpRoutes.java  | 11 +++---
 .../camel/quarkus/component/mllp/it/MllpTest.java  |  2 ++
 .../component/mllp/it/MllpTestResource.java}   | 11 +++---
 .../component/nagios/it/NagiosTestResource.java|  1 +
 .../component/netty/tcp/NettyTcpTestResource.java  |  1 +
 .../component/oaipmh/it/MockOaipmhServer.java  |  1 +
 .../sap/netweaver/it/SapNetweaverTestResource.java |  1 +
 .../component/syslog/it/SyslogTestResource.java|  2 +-
 20 files changed, 71 insertions(+), 17 deletions(-)
 copy 
integration-tests/{syslog/src/test/java/org/apache/camel/quarkus/component/syslog/it/SyslogTestResource.java
 => 
mllp/src/test/java/org/apache/camel/quarkus/component/mllp/it/MllpTestResource.java}
 (76%)

[GitHub] [camel-quarkus] jamesnetherton closed issue #3199: Improve available port discovery in tests



jamesnetherton closed issue #3199:
URL: https://github.com/apache/camel-quarkus/issues/3199


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] jamesnetherton merged pull request #3555: Improve available port discovery in tests



jamesnetherton merged pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (c4bf1b9 -> a4a5377)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from c4bf1b9  CAMEL-17637: camel-jbang - Run goal added --dep to specify 
dependency JARs to download and add to classpath from CLI
 add b1f8259  CAMEL-17662: camel-rest should lookup rest-dsl components in 
a better order
 add a4a5377  CAMEL-17662: camel-jbang - Should setup compoenents to use 
for rest-dsl

No new revisions were added by this update.

Summary of changes:
 .../src/main/java/org/apache/camel/component/rest/RestEndpoint.java   | 4 ++--
 .../src/main/java/org/apache/camel/main/KameletMain.java  | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

[GitHub] [camel-karavan] mikeintoch edited a comment on issue #194: Error on deploy in serverless mode



mikeintoch edited a comment on issue #194:
URL: https://github.com/apache/camel-karavan/issues/194#issuecomment-1042265404


   Will be there  backward compatibility with the old integration CRD?


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated (c4bf1b9 -> a4a5377)

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git.


from c4bf1b9  CAMEL-17637: camel-jbang - Run goal added --dep to specify 
dependency JARs to download and add to classpath from CLI
 add b1f8259  CAMEL-17662: camel-rest should lookup rest-dsl components in 
a better order
 add a4a5377  CAMEL-17662: camel-jbang - Should setup compoenents to use 
for rest-dsl

No new revisions were added by this update.

Summary of changes:
 .../src/main/java/org/apache/camel/component/rest/RestEndpoint.java   | 4 ++--
 .../src/main/java/org/apache/camel/main/KameletMain.java  | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

[GitHub] [camel-karavan] mikeintoch commented on issue #194: Error on deploy in serverless mode



mikeintoch commented on issue #194:
URL: https://github.com/apache/camel-karavan/issues/194#issuecomment-1042265404


   Will be there retrocompatibility with the old integration CRD?


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-karavan] mikeintoch commented on issue #197: route element on CRD cause issues when running a integration



mikeintoch commented on issue #197:
URL: https://github.com/apache/camel-karavan/issues/197#issuecomment-1042249845


   Thanks Claus


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-karavan] mikeintoch closed issue #197: route element on CRD cause issues when running a integration



mikeintoch closed issue #197:
URL: https://github.com/apache/camel-karavan/issues/197


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-karavan] davsclaus commented on issue #197: route element on CRD cause issues when running a integration



davsclaus commented on issue #197:
URL: https://github.com/apache/camel-karavan/issues/197#issuecomment-1042109144


   See this ticket: https://github.com/apache/camel-karavan/issues/194


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (ab9f5c8 -> c4bf1b9)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from ab9f5c8  Document Aggregator sync usage.
 add c4bf1b9  CAMEL-17637: camel-jbang - Run goal added --dep to specify 
dependency JARs to download and add to classpath from CLI

No new revisions were added by this update.

Summary of changes:
 .../apache/camel/dsl/jbang/core/commands/Run.java  |  8 ++-
 .../main/CommandLineDependencyDownloader.java  | 78 ++
 .../java/org/apache/camel/main/KameletMain.java|  6 ++
 .../org/apache/camel/main/KameletYamlRoutes.java   |  1 -
 4 files changed, 91 insertions(+), 2 deletions(-)
 create mode 100644 
dsl/camel-kamelet-main/src/main/java/org/apache/camel/main/CommandLineDependencyDownloader.java

[GitHub] [camel] JPMoresmau opened a new pull request #6963: [CAMEL-17651] enhance test



JPMoresmau opened a new pull request #6963:
URL: https://github.com/apache/camel/pull/6963


   Just a bit more testing to be sure... Thanks @davsclaus for moving and 
simplifying the test!


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-karavan] mikeintoch opened a new issue #197: route element on CRD cause issues when running a integration



mikeintoch opened a new issue #197:
URL: https://github.com/apache/camel-karavan/issues/197


   The new version of karavan create a CRD using route element like this:
   
   ```
   apiVersion: camel.apache.org/v1
   kind: Integration
   metadata:
 name: hello-world.yaml
   spec:
 flows:
   - route:
   from:
 uri: timer:foo
 steps:
   - setBody:
   expression:
 constant:
   expression: Hello World
   - log:
   message: ${body}
   logName: demo
 parameters:
   delay: '3'
   
   ```
   And when create the Integration using 'kubectl create -f integration.yaml' 
shows the next message
   **java.lang.IllegalArgumentException: Unsupported field: steps**
   
   Nevertheless when  use a CRD without route element definition everything 
runs well
   `
   apiVersion: camel.apache.org/v1
   kind: Integration
   metadata:
 name: hello-world.yaml
   spec:
 flows:
   - from:
   uri: timer:foo
   steps:
 - set-body:
 expression:
   constant: Hello World
 - log:
 log-name: demo
 message: '${body}'
   parameters:
 delay: '2000'
 dependencies:
   - 'camel:timer'
   `
   
   I'm using Camel K 1.8 and k3d
   
   Regards
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: CAMEL-17637: camel-jbang - Run goal added --dep to specify dependency JARs to download and add to classpath from CLI

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new c4bf1b9  CAMEL-17637: camel-jbang - Run goal added --dep to specify 
dependency JARs to download and add to classpath from CLI
c4bf1b9 is described below

commit c4bf1b9ef521285dd3bc34cabf08022dc9600c5a
Author: Claus Ibsen 
AuthorDate: Wed Feb 16 19:58:59 2022 +0100

CAMEL-17637: camel-jbang - Run goal added --dep to specify dependency JARs 
to download and add to classpath from CLI
---
 .../apache/camel/dsl/jbang/core/commands/Run.java  |  8 ++-
 .../main/CommandLineDependencyDownloader.java  | 78 ++
 .../java/org/apache/camel/main/KameletMain.java|  6 ++
 .../org/apache/camel/main/KameletYamlRoutes.java   |  1 -
 4 files changed, 91 insertions(+), 2 deletions(-)

diff --git 
a/dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/Run.java
 
b/dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/Run.java
index 6bb4837..778b90d 100644
--- 
a/dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/Run.java
+++ 
b/dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/Run.java
@@ -52,6 +52,9 @@ class Run implements Callable {
 @Parameters(description = "The Camel file(s) to run", arity = "1")
 private String[] files;
 
+@Option(names = { "--dep", "--dependency" }, description = "Additional 
dependencies to add to the classpath", arity = "0")
+private String[] dependencies;
+
 //CHECKSTYLE:OFF
 @Option(names = {"-h", "--help"}, usageHelp = true, description = "Display 
the help and sub-commands")
 private boolean helpRequested;
@@ -99,7 +102,7 @@ class Run implements Callable {
 private String jfrProfile;
 
 @Option(names = { "--local-kamelet-dir" },
-description = "Local directory to load Kamelets from (take 
precedence))")
+description = "Local directory for loading Kamelets (takes 
precedence)")
 private String localKameletDir;
 
 @Option(names = { "--port" }, description = "Embeds a local HTTP server on 
this port")
@@ -188,6 +191,9 @@ class Run implements Callable {
 main.addInitialProperty("camel.jbang.jfr", "jfr");
 main.addInitialProperty("camel.jbang.jfr-profile", jfrProfile);
 }
+if (dependencies != null) {
+main.addInitialProperty("camel.jbang.dependencies", 
String.join(",", dependencies));
+}
 
 if (fileLock) {
 lockFile = createLockFile();
diff --git 
a/dsl/camel-kamelet-main/src/main/java/org/apache/camel/main/CommandLineDependencyDownloader.java
 
b/dsl/camel-kamelet-main/src/main/java/org/apache/camel/main/CommandLineDependencyDownloader.java
new file mode 100644
index 000..4146d19
--- /dev/null
+++ 
b/dsl/camel-kamelet-main/src/main/java/org/apache/camel/main/CommandLineDependencyDownloader.java
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.main;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.apache.camel.CamelContext;
+import org.apache.camel.CamelContextAware;
+import org.apache.camel.support.service.ServiceSupport;
+
+public class CommandLineDependencyDownloader extends ServiceSupport implements 
CamelContextAware {
+
+private CamelContext camelContext;
+private final String dependencies;
+
+public CommandLineDependencyDownloader(String dependencies) {
+this.dependencies = dependencies;
+}
+
+@Override
+public CamelContext getCamelContext() {
+return camelContext;
+}
+
+@Override
+public void setCamelContext(CamelContext camelContext) {
+this.camelContext = camelContext;
+}
+
+@Override
+protected void doInit() throws Exception {
+downloadDependencies();
+}
+
+private void downloadDependencies() {
+final List gavs = new ArrayList<>();
+for (String dep : dependencies.split(",")) {
+String gav =

[GitHub] [camel-karavan] mgubaidullin merged pull request #196: Parameters



mgubaidullin merged pull request #196:
URL: https://github.com/apache/camel-karavan/pull/196


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (eeb3145 -> ab9f5c8)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from eeb3145  CAMEL-17651: fix bug in reload strategy fileFilter introduced 
by PR #6956. The filter needs the filename not the path.
 add ab9f5c8  Document Aggregator sync usage.

No new revisions were added by this update.

Summary of changes:
 .../src/main/docs/modules/eips/pages/aggregate-eip.adoc | 2 ++
 1 file changed, 2 insertions(+)

[GitHub] [camel-kafka-connector] hakidehari commented on issue #1339: Camel Kafka Salesforce Source connector keeps disconnecting/resubscribing and eventually fails



hakidehari commented on issue #1339:
URL: 
https://github.com/apache/camel-kafka-connector/issues/1339#issuecomment-1041969186


   Thanks for the response @zregvart.  I also had this hunch that using the 
same connected app for each connector could be causing this issue.  I am glad 
you brought this up as well.  I will go ahead and try using one connected app 
for one connector and see what happens.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6962: CAMEL-17308: Remove "verb" as an allowed CAMEL REST YAML element.



davsclaus commented on pull request #6962:
URL: https://github.com/apache/camel/pull/6962#issuecomment-1041961715


   Can you add a note in the 3.16 upgrade doc, about the yaml-dsl change (see 
root/docs folder).
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-kafka-connector] zregvart edited a comment on issue #1339: Camel Kafka Salesforce Source connector keeps disconnecting/resubscribing and eventually fails

zregvart edited a comment on issue #1339:
URL:
https://github.com/apache/camel-kafka-connector/issues/1339#issuecomment-1041952020

@hakidehari we had reports of similar issues in the past and as far as we're
aware they have been fixed up to Camel 3.5, that is the folk that reported them
haven't raised any new concerns.
[This](https://issues.apache.org/jira/browse/CAMEL-15233?jql=project%20%3D%20CAMEL%20AND%20component%20%3D%20camel-salesforce%20ORDER%20BY%20affectedVersion%20DESC%2C%20priority%20DESC%2C%20updated%20DESC)
is the list of issues in the Salesforce component, look for CometD and
platform event ones.
These issues are really difficult to troubleshoot and the best we can offer
here is this anecdotal advice. It seems that connecting via the same Connected
App could cause authentication issues, we have never confirmed this to be the
case.
So, if you're using the same Connected App configuration for all clients I
would suggest using different one for each one, also make sure that no one else
is using the same Connected Apps as the ones you have configured.
Beyond this, to fix this we would need a very detailed and clear way of
reproducing this issue.

--
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-kafka-connector] zregvart commented on issue #1339: Camel Kafka Salesforce Source connector keeps disconnecting/resubscribing and eventually fails



zregvart commented on issue #1339:
URL: 
https://github.com/apache/camel-kafka-connector/issues/1339#issuecomment-1041952020


   @hakidehari we had reports of similar issues in the past and as far as we're 
aware they have been fixed up to Camel 3.5, that is the folk that reported them 
haven't raised any new concerns. 
[This](https://issues.apache.org/jira/browse/CAMEL?jql=project %3D CAMEL AND 
component %3D camel-salesforce ORDER BY affectedVersion DESC%2C priority 
DESC%2C updated DESC) is the list of issues in the Salesforce component, look 
for CometD and platform event ones.
   These issues are really difficult to troubleshoot and the best we can offer 
here is this anecdotal advice. It seems that connecting via the same Connected 
App could cause authentication issues, we have never confirmed this to be the 
case.
   So, if you're using the same Connected App configuration for all clients I 
would suggest using different one for each one, also make sure that no one else 
is using the same Connected Apps as the ones you have configured.
   Beyond this, to fix this we would need a very detailed and clear way of 
reproducing this issue.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #3555: Improve available port discovery in tests



ppalaga commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808289948



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -54,15 +57,23 @@ private AvailablePortFinder() {
  * @return   the available port
  */
 public static int getNextAvailable() {
+// Using AvailablePortFinder in native applications can be problematic
+// E.g The reserved port may be allocated at build time and preserved 
indefinitely at runtime. I.e it never changes on each execution of the native 
application
+logWarningIfNativeApplication();
+
 while (true) {
 try (ServerSocket ss = new ServerSocket()) {
 ss.setReuseAddress(true);
 ss.bind(new InetSocketAddress((InetAddress) null, 0), 1);
 
 int port = ss.getLocalPort();
 if (!isQuarkusReservedPort(port)) {
-LOGGER.info("getNextAvailable() -> {}", port);
-return port;
+String callerClassName = getCallerClassName();
+String value = RESERVED_PORTS.putIfAbsent(port, 
callerClassName);

Review comment:
   Thanks for explaining. If it is just for logging, indeed, multiple 
RESERVED_PORTS instances in concurrent per-module JVMs do not matter.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] klease commented on pull request #6962: CAMEL-17308: Remove "verb" as an allowed CAMEL REST YAML element.



klease commented on pull request #6962:
URL: https://github.com/apache/camel/pull/6962#issuecomment-1041920666


   The generated YAML model now contains a "verbs" definition (as an array of 
VerbDefinition which is no longer defined in the Yaml). This doesn't seem 
consistent but doesn't prevent the test cases using the method names from 
running.


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] klease opened a new pull request #6962: CAMEL-17308: Remove "verb" as an allowed CAMEL REST YAML element.



klease opened a new pull request #6962:
URL: https://github.com/apache/camel/pull/6962


   This also affects the generated XML model which also will only accept
   the XML elements for different methods such as get, put and post.
   Remove test cases in RestTest.groovy using verb.
   
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] orpiske opened a new pull request #6961: Maven plugin cleanups



orpiske opened a new pull request #6961:
URL: https://github.com/apache/camel/pull/6961


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-k] phantomjinx opened a new pull request #3020: Adds OLM channels to bundle metadata



phantomjinx opened a new pull request #3020:
URL: https://github.com/apache/camel-k/pull/3020


   * Adds the additional stable, candidate and latest
 channels for OLM provisioning
   
   
   Camel-k has published its releases through the OLM using versioned channel 
names, eg. stable-1.9.0. This starts to migrate away from this paradigm to a 
more conventional design where 3 channels are provided and may contain releases.
   - stable: will contain those releases that have been proven stable and 
tested (not the very latest versions)
   - candidate: will contain the very latest releases, that may still be 
snapshots, allowing users early access to new features
   - latest: will contain releases as soon as they are released
   
   
   
   
   
   **Release Note**
   ```release-note
   NONE
   ```
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] davsclaus commented on pull request #6960: CAMEL-17658 camel-debezium: additionalProperties are trimmed



davsclaus commented on pull request #6960:
URL: https://github.com/apache/camel/pull/6960#issuecomment-1041897553


   Ah so are you saying the key in the map is a string with a dot?
   
   As if you did this in regular Java
   
   Map map = ...
   map.put("A.B", "myValue")
   
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-kafka-connector] oscerd commented on issue #1339: Camel Kafka Salesforce Source connector keeps disconnecting/resubscribing and eventually fails



oscerd commented on issue #1339:
URL: 
https://github.com/apache/camel-kafka-connector/issues/1339#issuecomment-1041881372


   I don't think so, maybe @zregvart could help


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-kafka-connector] hakidehari commented on issue #1339: Camel Kafka Salesforce Source connector keeps disconnecting/resubscribing and eventually fails



hakidehari commented on issue #1339:
URL: 
https://github.com/apache/camel-kafka-connector/issues/1339#issuecomment-1041862782


   We upgraded to the latest version 0.11.0 and are still getting the same 
Unknown Client error after some elapsed time.  Is there any way to forcefully 
restart the connector automatically through some sort of config? @oscerd 


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus-examples] jamesnetherton commented on a change in pull request #76: Dynamically generate native test matrix



jamesnetherton commented on a change in pull request #76:
URL: 
https://github.com/apache/camel-quarkus-examples/pull/76#discussion_r808185980



##
File path: .github/workflows/ci-build.yaml
##
@@ -139,17 +127,15 @@ jobs:
 run: |
   tar -xzf ../maven-repo.tgz -C ~
   - name: Integration Tests
+env:
+  TEST_MODULES: ${{matrix.tests}}
+shell: bash
 run: |
-  EXAMPLE_PROJECTS=""
-  for EXAMPLE in $(yq -M -N e ".${{ matrix.category }}" 
.github/test-categories.yaml | cut -f2 -d' '); do
-if [ "${EXAMPLE}" == "null" ]; then
-  continue
-fi
-
+  for MODULE in ${TEST_MODULES//,/ }; do
 ./mvnw ${MAVEN_ARGS} clean verify \
   -Dformatter.skip -Dimpsort.skip \
   -Pnative,docker \
-  -f "${EXAMPLE}/pom.xml"
+  -f "${MODULE}/pom.xml"
   done

Review comment:
   Latest commit produces a simple summary of any build failures.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] aldettinger opened a new pull request #3557: gson: complete doc and test coverage #3556



aldettinger opened a new pull request #3557:
URL: https://github.com/apache/camel-quarkus/pull/3557


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: CAMEL-17651: fix bug in reload strategy fileFilter introduced by PR #6956. The filter needs the filename not the path.

2022-02-16 Thread klease

This is an automated email from the ASF dual-hosted git repository.

klease pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new eeb3145  CAMEL-17651: fix bug in reload strategy fileFilter introduced 
by PR #6956. The filter needs the filename not the path.
eeb3145 is described below

commit eeb3145ff54ced45697f580b72f1ba5346c56979
Author: klease 
AuthorDate: Wed Feb 16 17:06:58 2022 +0100

CAMEL-17651: fix bug in reload strategy fileFilter introduced by PR #6956.
The filter needs the filename not the path.
---
 .../main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
 
b/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
index 53d1920..08400fb 100644
--- 
a/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
+++ 
b/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
@@ -114,9 +114,8 @@ public class RouteWatcherReloadStrategy extends 
FileWatcherResourceReloadStrateg
 // strip starting directory, so we have a relative name to 
the starting folder
 String path = f.getAbsolutePath();
 if (path.startsWith(base)) {
-path = FileUtil.onlyPath(path);
+path = FileUtil.stripPath(path);
 }
-path = FileUtil.stripLeadingSeparator(path);
 
 boolean result = matcher.match(part, path, false);
 LOG.trace("Accepting file pattern:{} path:{} -> {}", part, 
path, result);

[camel] branch regen_bot updated (4cec24e -> 1b2a27f)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 4cec24e  Nicer logging when using MP config
 add 1b2a27f  Source code generate the META-INF services files

No new revisions were added by this update.

Summary of changes:
 .../org/apache/camel/properties-source-factory |  2 ++
 .../config/CamelMicroProfilePropertiesSource.java  |  2 ++
 .../org/apache/camel/properties-source-factory | 18 --
 3 files changed, 4 insertions(+), 18 deletions(-)
 create mode 100644 
components/camel-microprofile/camel-microprofile-config/src/generated/resources/META-INF/services/org/apache/camel/properties-source-factory
 delete mode 100644 
components/camel-microprofile/camel-microprofile-config/src/main/resources/META-INF/services/org/apache/camel/properties-source-factory

[camel] branch regen_bot updated (2a65583 -> 4cec24e)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 2a65583  CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor 
should be able to control to stop routing
 add e36741e  (chores) camel-util-json: use try-with-resources when possible
 add 9018cbf  (chores) camel-util-json: remove commented code
 add 1b409d1  (chores) camel-package-maven-plugin: do close stream after 
opening it
 add 9469366  (chores) camel-package-maven-plugin: replace read logic
 add 4cec24e  Nicer logging when using MP config

No new revisions were added by this update.

Summary of changes:
 .../config/CamelMicroProfilePropertiesSource.java  |  5 +++
 .../apache/camel/tooling/util/PackageHelper.java   | 24 +--
 .../java/org/apache/camel/util/json/Jsoner.java| 48 +-
 .../camel/maven/packaging/ComponentDslMojo.java|  3 +-
 .../camel/maven/packaging/EndpointDslMojo.java |  4 +-
 .../camel/maven/packaging/PrepareExampleMojo.java  |  6 +--
 .../packaging/generics/PackagePluginUtils.java |  8 
 7 files changed, 32 insertions(+), 66 deletions(-)

[camel] branch regen_bot updated (9469366 -> 2a65583)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


omit 9469366  (chores) camel-package-maven-plugin: replace read logic
omit 1b409d1  (chores) camel-package-maven-plugin: do close stream after 
opening it
omit 9018cbf  (chores) camel-util-json: remove commented code
omit e36741e  (chores) camel-util-json: use try-with-resources when possible

This update removed existing revisions from the reference, leaving the
reference pointing at a previous point in the repository history.

 * -- * -- N   refs/heads/regen_bot (2a65583)
\
 O -- O -- O   (9469366)

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .../apache/camel/tooling/util/PackageHelper.java   | 24 ++-
 .../java/org/apache/camel/util/json/Jsoner.java| 48 +-
 .../camel/maven/packaging/ComponentDslMojo.java|  3 +-
 .../camel/maven/packaging/EndpointDslMojo.java |  4 +-
 .../camel/maven/packaging/PrepareExampleMojo.java  |  6 +--
 .../packaging/generics/PackagePluginUtils.java |  8 
 6 files changed, 66 insertions(+), 27 deletions(-)

[camel] branch regen_bot updated (cf6c948 -> 9469366)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from cf6c948  CAMEL-17655: Fix NPE in camel-opentracing
 add 2a65583  CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor 
should be able to control to stop routing
 add e36741e  (chores) camel-util-json: use try-with-resources when possible
 add 9018cbf  (chores) camel-util-json: remove commented code
 add 1b409d1  (chores) camel-package-maven-plugin: do close stream after 
opening it
 add 9469366  (chores) camel-package-maven-plugin: replace read logic

No new revisions were added by this update.

Summary of changes:
 .../errorhandler/RedeliveryErrorHandler.java   |  9 +++
 .../ErrorHandlerOnRedeliveryStopTest.java} | 64 +++---
 .../apache/camel/tooling/util/PackageHelper.java   | 24 +---
 .../java/org/apache/camel/util/json/Jsoner.java| 48 
 .../camel/maven/packaging/ComponentDslMojo.java|  3 +-
 .../camel/maven/packaging/EndpointDslMojo.java |  4 +-
 .../camel/maven/packaging/PrepareExampleMojo.java  |  6 +-
 .../packaging/generics/PackagePluginUtils.java |  8 +++
 8 files changed, 67 insertions(+), 99 deletions(-)
 copy 
core/camel-core/src/test/java/org/apache/camel/{component/bean/BeanThrowAssertionErrorTest.java
 => issues/ErrorHandlerOnRedeliveryStopTest.java} (51%)

[GitHub] [camel-quarkus] jamesnetherton commented on a change in pull request #3555: Improve available port discovery in tests



jamesnetherton commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808107427



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -54,15 +57,23 @@ private AvailablePortFinder() {
  * @return   the available port
  */
 public static int getNextAvailable() {
+// Using AvailablePortFinder in native applications can be problematic
+// E.g The reserved port may be allocated at build time and preserved 
indefinitely at runtime. I.e it never changes on each execution of the native 
application
+logWarningIfNativeApplication();
+
 while (true) {
 try (ServerSocket ss = new ServerSocket()) {
 ss.setReuseAddress(true);
 ss.bind(new InetSocketAddress((InetAddress) null, 0), 1);
 
 int port = ss.getLocalPort();
 if (!isQuarkusReservedPort(port)) {
-LOGGER.info("getNextAvailable() -> {}", port);
-return port;
+String callerClassName = getCallerClassName();
+String value = RESERVED_PORTS.putIfAbsent(port, 
callerClassName);

Review comment:
   Ok - after re-reading. It's about separate `RESERVED_PORTS` instances.
   
   In which case, yes, I suppose the map contents would be different between 
concurrent JVMs or class loaders.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated (4cec24e -> 1b2a27f)

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 4cec24e  Nicer logging when using MP config
 add 1b2a27f  Source code generate the META-INF services files

No new revisions were added by this update.

Summary of changes:
 .../org/apache/camel/properties-source-factory |  2 ++
 .../config/CamelMicroProfilePropertiesSource.java  |  2 ++
 .../org/apache/camel/properties-source-factory | 18 --
 3 files changed, 4 insertions(+), 18 deletions(-)
 create mode 100644 
components/camel-microprofile/camel-microprofile-config/src/generated/resources/META-INF/services/org/apache/camel/properties-source-factory
 delete mode 100644 
components/camel-microprofile/camel-microprofile-config/src/main/resources/META-INF/services/org/apache/camel/properties-source-factory

[camel] branch main updated (9469366 -> 4cec24e)

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 9469366  (chores) camel-package-maven-plugin: replace read logic
 add 4cec24e  Nicer logging when using MP config

No new revisions were added by this update.

Summary of changes:
 .../microprofile/config/CamelMicroProfilePropertiesSource.java   | 5 +
 1 file changed, 5 insertions(+)

[GitHub] [camel-quarkus] JiriOndrusek commented on issue #3488: Debezium mysql - passing additionalProperties results in wrong properties beeing passed



JiriOndrusek commented on issue #3488:
URL: https://github.com/apache/camel-quarkus/issues/3488#issuecomment-1041586933


   I've found that problem is caused by the camel. I reported it as 
https://issues.apache.org/jira/browse/CAMEL-17658


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] JiriOndrusek opened a new pull request #6960: CAMEL-17658 camel-debezium: additionalProperties are trimmed



JiriOndrusek opened a new pull request #6960:
URL: https://github.com/apache/camel/pull/6960


   Issue: https://issues.apache.org/jira/browse/CAMEL-17658
   
   Problem is caused not entirely by debezium component. Automatic properties 
configuration can "trim" property in specific cases.
   
   - Let say, that we have a property **map.A.B=value**, where  **map** is an 
URI property defined as ` @UriParam(label = "common", prefix = "map.", 
multiValue = true,`. Automatic configuration initialization will detect a 
HashMap object as a **map** property, but then it executes "set property 
A=value into HashMap (and property B is ignored)"
   - I'm trying to change mechanism to not ignore the remaining part of the 
property name and execute "set property A.B=value into HashMap"
   
   I'm not sure whether this change could be approved, Therefore this is only 
draft and I'd like to see whether there will be some failing tests.
   
   In case that this approach is not desired or potentially dangerous, I will 
fix it directly on debezium components - which is safe.
   
   
   
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] 01/04: (chores) camel-util-json: use try-with-resources when possible

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit e36741ec617ea07af76de08ba278343766b1d032
Author: Otavio Rodolfo Piske 
AuthorDate: Wed Feb 16 10:31:05 2022 +0100

(chores) camel-util-json: use try-with-resources when possible
---
 .../java/org/apache/camel/util/json/Jsoner.java| 43 +-
 1 file changed, 10 insertions(+), 33 deletions(-)

diff --git 
a/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java 
b/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
index e1d089a..9d0119c 100644
--- 
a/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
+++ 
b/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
@@ -355,24 +355,17 @@ public final class Jsoner {
  * @see StringReader
  */
 public static Object deserialize(final String deserializable) throws 
DeserializationException {
-Object returnable;
-StringReader readableDeserializable = null;
-try {
-readableDeserializable = new StringReader(deserializable);
-returnable = Jsoner.deserialize(readableDeserializable);
+try (StringReader readableDeserializable = new 
StringReader(deserializable)) {
+
+return Jsoner.deserialize(readableDeserializable);
 } catch (IOException | NullPointerException caught) {
 /*
  * They both have the same recovery scenario. See StringReader. If
  * deserializable is null, it should be reasonable to expect null
  * back.
  */
-returnable = null;
-} finally {
-if (readableDeserializable != null) {
-readableDeserializable.close();
-}
+return null;
 }
-return returnable;
 }
 
 /**
@@ -386,21 +379,13 @@ public final class Jsoner {
  * @see   Jsoner#deserialize(Reader)
  */
 public static JsonArray deserialize(final String deserializable, final 
JsonArray defaultValue) {
-StringReader readable = null;
-JsonArray returnable;
-try {
-readable = new StringReader(deserializable);
-returnable = Jsoner.deserialize(readable, 
EnumSet.of(DeserializationOptions.ALLOW_JSON_ARRAYS)).<
+try (StringReader readable = new StringReader(deserializable)) {
+return Jsoner.deserialize(readable, 
EnumSet.of(DeserializationOptions.ALLOW_JSON_ARRAYS)).<
 JsonArray> getCollection(0);
 } catch (NullPointerException | IOException | DeserializationException 
caught) {
 /* Don't care, just return the default value. */
-returnable = defaultValue;
-} finally {
-if (readable != null) {
-readable.close();
-}
+return defaultValue;
 }
-return returnable;
 }
 
 /**
@@ -415,21 +400,13 @@ public final class Jsoner {
  * @see   Jsoner#deserialize(Reader)
  */
 public static JsonObject deserialize(final String deserializable, final 
JsonObject defaultValue) {
-StringReader readable = null;
-JsonObject returnable;
-try {
-readable = new StringReader(deserializable);
-returnable = Jsoner.deserialize(readable, 
EnumSet.of(DeserializationOptions.ALLOW_JSON_OBJECTS)).<
+try (StringReader readable = new StringReader(deserializable)) {
+return Jsoner.deserialize(readable, 
EnumSet.of(DeserializationOptions.ALLOW_JSON_OBJECTS)).<
 JsonObject> getMap(0);
 } catch (NullPointerException | IOException | DeserializationException 
caught) {
 /* Don't care, just return the default value. */
-returnable = defaultValue;
-} finally {
-if (readable != null) {
-readable.close();
-}
+return defaultValue;
 }
-return returnable;
 }
 
 /**

[camel] 04/04: (chores) camel-package-maven-plugin: replace read logic

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 946936626bc37bfff5f3f948b019d99e4ee31934
Author: Otavio Rodolfo Piske 
AuthorDate: Wed Feb 16 14:05:28 2022 +0100

(chores) camel-package-maven-plugin: replace read logic

 Replace builtin read stream logic with builtin one from newer Java versions
---
 .../apache/camel/tooling/util/PackageHelper.java   | 24 +-
 .../camel/maven/packaging/ComponentDslMojo.java|  3 ++-
 .../camel/maven/packaging/EndpointDslMojo.java |  4 +++-
 .../packaging/generics/PackagePluginUtils.java |  8 
 4 files changed, 14 insertions(+), 25 deletions(-)

diff --git 
a/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
 
b/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
index 1c7cf6f..2f7fc16 100644
--- 
a/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
+++ 
b/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
@@ -16,12 +16,9 @@
  */
 package org.apache.camel.tooling.util;
 
-import java.io.BufferedReader;
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
-import java.io.InputStreamReader;
-import java.io.LineNumberReader;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.HashMap;
@@ -42,29 +39,10 @@ public final class PackageHelper {
 /**
  * Loads the entire stream into memory as a String and returns it.
  * 
- * Notice: This implementation appends a \n as line 
terminator at the of the text.
- * 
  * Warning, don't use for crazy big streams :)
  */
 public static String loadText(InputStream in) throws IOException {
-StringBuilder builder = new StringBuilder();
-InputStreamReader isr = new InputStreamReader(in);
-try {
-BufferedReader reader = new LineNumberReader(isr);
-while (true) {
-String line = reader.readLine();
-if (line != null) {
-builder.append(line);
-builder.append("\n");
-} else {
-break;
-}
-}
-return builder.toString();
-} finally {
-isr.close();
-in.close();
-}
+return new String(in.readAllBytes());
 }
 
 public static String loadText(File file) throws IOException {
diff --git 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/ComponentDslMojo.java
 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/ComponentDslMojo.java
index 0d56017..25bcdc2e 100644
--- 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/ComponentDslMojo.java
+++ 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/ComponentDslMojo.java
@@ -41,6 +41,7 @@ import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.MavenProjectHelper;
 import org.sonatype.plexus.build.incremental.BuildContext;
 
+import static 
org.apache.camel.maven.packaging.generics.PackagePluginUtils.joinHeaderAndSource;
 import static org.apache.camel.tooling.util.PackageHelper.findCamelDirectory;
 import static org.apache.camel.tooling.util.PackageHelper.loadText;
 
@@ -224,7 +225,7 @@ public class ComponentDslMojo extends AbstractGeneratorMojo 
{
 Path target = outputDir.toPath().resolve(filePath).resolve(fileName);
 
 try {
-String code = licenseHeader + source;
+final String code = joinHeaderAndSource(licenseHeader, source);
 
 if (getLog().isDebugEnabled()) {
 getLog().debug("Source code generated:\n" + code);
diff --git 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
index 822570a..8bd0943 100644
--- 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
+++ 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/EndpointDslMojo.java
@@ -58,6 +58,7 @@ import org.jboss.forge.roaster.model.source.MethodSource;
 import org.jboss.forge.roaster.model.source.ParameterSource;
 import org.sonatype.plexus.build.incremental.BuildContext;
 
+import static 
org.apache.camel.maven.packaging.generics.PackagePluginUtils.joinHeaderAndSource;
 import static org.apache.camel.tooling.util.PackageHelper.findCamelDirectory;
 import static org.apache.camel.tooling.util.PackageHelper.loadText;
 
@@ -1006,7 +1007,8 @@ public class EndpointDslMojo extends

[camel] 02/04: (chores) camel-util-json: remove commented code

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 9018cbf778a450ee179598e5a9fa0953617df632
Author: Otavio Rodolfo Piske 
AuthorDate: Wed Feb 16 10:31:38 2022 +0100

(chores) camel-util-json: remove commented code
---
 .../src/main/java/org/apache/camel/util/json/Jsoner.java | 5 -
 1 file changed, 5 deletions(-)

diff --git 
a/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java 
b/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
index 9d0119c..193cd8c 100644
--- 
a/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
+++ 
b/tooling/camel-util-json/src/main/java/org/apache/camel/util/json/Jsoner.java
@@ -650,7 +650,6 @@ public final class Jsoner {
 }
 break;
 }
-// System.out.println(lexed);
 } while (!lexed.getType().equals(Yytoken.Types.END));
 } catch (final DeserializationException caught) {
 /* This is according to the method's contract. */
@@ -659,9 +658,6 @@ public final class Jsoner {
 /* See StringReader. */
 return null;
 }
-// System.out.println(printable);
-// System.out.println(returnable);
-// System.out.println(Jsoner.escape(returnable.toString()));
 return returnable.toString();
 }
 
@@ -957,7 +953,6 @@ public final class Jsoner {
 + "4) If you feel it should have serialized you 
could use a more tolerant serialization for debugging purposes.");
 }
 }
-// System.out.println(writableDestination.toString());
 }
 // CHECKSTYLE:ON

[camel] 03/04: (chores) camel-package-maven-plugin: do close stream after opening it

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 1b409d15e25a7cca5dcaf574ff4555496e5bd51b
Author: Otavio Rodolfo Piske 
AuthorDate: Wed Feb 16 14:03:38 2022 +0100

(chores) camel-package-maven-plugin: do close stream after opening it
---
 .../java/org/apache/camel/maven/packaging/PrepareExampleMojo.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java
 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java
index efc42f1..2f15b64 100644
--- 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java
+++ 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PrepareExampleMojo.java
@@ -18,6 +18,7 @@ package org.apache.camel.maven.packaging;
 
 import java.io.File;
 import java.io.IOException;
+import java.io.InputStream;
 import java.nio.charset.Charset;
 import java.nio.file.Paths;
 import java.util.ArrayList;
@@ -190,9 +191,8 @@ public class PrepareExampleMojo extends AbstractMojo {
 }
 
 private String templateExamples(List models, long 
deprecated) throws MojoExecutionException {
-try {
-String template = PackageHelper
-
.loadText(UpdateReadmeMojo.class.getClassLoader().getResourceAsStream("readme-examples.mvel"));
+try (InputStream templateStream = 
UpdateReadmeMojo.class.getClassLoader().getResourceAsStream("readme-examples.mvel"))
 {
+String template = PackageHelper.loadText(templateStream);
 Map map = new HashMap<>();
 map.put("examples", models);
 map.put("numberOfDeprecated", deprecated);

[camel] branch main updated (2a65583 -> 9469366)

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 2a65583  CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor 
should be able to control to stop routing
 new e36741e  (chores) camel-util-json: use try-with-resources when possible
 new 9018cbf  (chores) camel-util-json: remove commented code
 new 1b409d1  (chores) camel-package-maven-plugin: do close stream after 
opening it
 new 9469366  (chores) camel-package-maven-plugin: replace read logic

The 4 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../apache/camel/tooling/util/PackageHelper.java   | 24 +--
 .../java/org/apache/camel/util/json/Jsoner.java| 48 +-
 .../camel/maven/packaging/ComponentDslMojo.java|  3 +-
 .../camel/maven/packaging/EndpointDslMojo.java |  4 +-
 .../camel/maven/packaging/PrepareExampleMojo.java  |  6 +--
 .../packaging/generics/PackagePluginUtils.java |  8 
 6 files changed, 27 insertions(+), 66 deletions(-)

[GitHub] [camel] orpiske merged pull request #6959: More camel-package-maven-plugin cleanups



orpiske merged pull request #6959:
URL: https://github.com/apache/camel/pull/6959


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor should be able to control to stop routing

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new 2a65583  CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor 
should be able to control to stop routing
2a65583 is described below

commit 2a655836767de6f7d8e9db10ccf6dffda2c3bee1
Author: Claus Ibsen 
AuthorDate: Wed Feb 16 15:41:49 2022 +0100

CAMEL-17660: amel-core - Error handler OnRedeliveryProcessor should be able 
to control to stop routing
---
 .../errorhandler/RedeliveryErrorHandler.java   |  9 +++
 .../issues/ErrorHandlerOnRedeliveryStopTest.java   | 84 ++
 2 files changed, 93 insertions(+)

diff --git 
a/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryErrorHandler.java
 
b/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryErrorHandler.java
index 218556b..2bdff91 100644
--- 
a/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryErrorHandler.java
+++ 
b/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryErrorHandler.java
@@ -864,6 +864,15 @@ public abstract class RedeliveryErrorHandler extends 
ErrorHandlerSupport
 // letting onRedeliver be executed at first
 deliverToOnRedeliveryProcessor();
 
+if (exchange.isRouteStop()) {
+// the on redelivery can mark that the exchange should stop 
and therefore not perform a redelivery
+// and if so then we are done so continue callback
+AsyncCallback cb = callback;
+taskFactory.release(this);
+reactiveExecutor.schedule(cb);
+return;
+}
+
 if (LOG.isTraceEnabled()) {
 LOG.trace("Redelivering exchangeId: {} -> {} for Exchange: 
{}", exchange.getExchangeId(), outputAsync,
 exchange);
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/issues/ErrorHandlerOnRedeliveryStopTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/issues/ErrorHandlerOnRedeliveryStopTest.java
new file mode 100644
index 000..a3b1ee9
--- /dev/null
+++ 
b/core/camel-core/src/test/java/org/apache/camel/issues/ErrorHandlerOnRedeliveryStopTest.java
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.issues;
+
+import java.util.concurrent.RejectedExecutionException;
+import java.util.concurrent.atomic.AtomicInteger;
+
+import org.apache.camel.ContextTestSupport;
+import org.apache.camel.Exchange;
+import org.apache.camel.Processor;
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.support.ExpressionAdapter;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.fail;
+
+public class ErrorHandlerOnRedeliveryStopTest extends ContextTestSupport {
+
+private final AtomicInteger counter = new AtomicInteger(5);
+
+@Test
+public void testRetryWhile() throws Exception {
+getMockEndpoint("mock:result").expectedMessageCount(0);
+
+try {
+template.sendBody("direct:start", "Hello World");
+fail("Should throw an exception");
+} catch (Exception e) {
+RejectedExecutionException ree = 
assertIsInstanceOf(RejectedExecutionException.class, e.getCause());
+Assertions.assertEquals("I do not want to do this anymore", 
ree.getMessage());
+}
+
+assertMockEndpointsSatisfied();
+}
+
+@Override
+protected RouteBuilder createRouteBuilder() throws Exception {
+return new RouteBuilder() {
+@Override
+public void configure() throws Exception {
+
+errorHandler(defaultErrorHandler()
+.retryWhile(new ExpressionAdapter() {
+@Override
+public Object evaluate(Exchange exchange) {
+return counter.getAndDecrement() > 0;
+

[GitHub] [camel-quarkus] jamesnetherton commented on a change in pull request #3555: Improve available port discovery in tests



jamesnetherton commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808045495



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -54,15 +57,23 @@ private AvailablePortFinder() {
  * @return   the available port
  */
 public static int getNextAvailable() {
+// Using AvailablePortFinder in native applications can be problematic
+// E.g The reserved port may be allocated at build time and preserved 
indefinitely at runtime. I.e it never changes on each execution of the native 
application
+logWarningIfNativeApplication();
+
 while (true) {
 try (ServerSocket ss = new ServerSocket()) {
 ss.setReuseAddress(true);
 ss.bind(new InetSocketAddress((InetAddress) null, 0), 1);
 
 int port = ss.getLocalPort();
 if (!isQuarkusReservedPort(port)) {
-LOGGER.info("getNextAvailable() -> {}", port);
-return port;
+String callerClassName = getCallerClassName();
+String value = RESERVED_PORTS.putIfAbsent(port, 
callerClassName);

Review comment:
   Can you elaborate a bit more? It's not entirely clear to me why it 
matters in this context.
   
   All it's doing is trying to track which class reserved a specific port. 
Arguably we don't need to do that. It just provides some nice logging output 
and a way to clear out stale entries from the `RESERVED_PORTS` map.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3555: Improve available port discovery in tests



aldettinger commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808031267



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))
+.findFirst()

Review comment:
   Ok, actually it's Map so the value would appear 2 times in 
the map => OK




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #3555: Improve available port discovery in tests



ppalaga commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808019076



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -54,15 +57,23 @@ private AvailablePortFinder() {
  * @return   the available port
  */
 public static int getNextAvailable() {
+// Using AvailablePortFinder in native applications can be problematic
+// E.g The reserved port may be allocated at build time and preserved 
indefinitely at runtime. I.e it never changes on each execution of the native 
application
+logWarningIfNativeApplication();
+
 while (true) {
 try (ServerSocket ss = new ServerSocket()) {
 ss.setReuseAddress(true);
 ss.bind(new InetSocketAddress((InetAddress) null, 0), 1);
 
 int port = ss.getLocalPort();
 if (!isQuarkusReservedPort(port)) {
-LOGGER.info("getNextAvailable() -> {}", port);
-return port;
+String callerClassName = getCallerClassName();
+String value = RESERVED_PORTS.putIfAbsent(port, 
callerClassName);

Review comment:
   Is this making some assumptions about whether this class can 
concurrently exist in multiple JVMs or multiple class loaders within a single 
maven invocation? IIRC surefire is forking a new JVM for each Maven module. So 
if building with mvnd, the AvailablePortFinder class (and thus also separate 
RESERVED_PORTS instances) may exist in several concurrent JVMs. 




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot_3.14.x updated (3bee2c3 -> b734369)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot_3.14.x
in repository https://gitbox.apache.org/repos/asf/camel.git.


from 3bee2c3  CAMEL-17650: camel-vertx-websocket: sendToAll operation 
should consider the path WebSocket clients are connected to
 add b734369  CAMEL-17655: Fix NPE in camel-opentracing

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/camel/opentracing/TagProcessor.java| 14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #3555: Improve available port discovery in tests



ppalaga commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808021037



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))

Review comment:
   > The class declaration is:
   > 
   > ```java
   > public final class AvailablePortFinder {
   > }
   > ```
   
   I am blind. Sorry for the noise.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #3555: Improve available port discovery in tests



ppalaga commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808019076



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -54,15 +57,23 @@ private AvailablePortFinder() {
  * @return   the available port
  */
 public static int getNextAvailable() {
+// Using AvailablePortFinder in native applications can be problematic
+// E.g The reserved port may be allocated at build time and preserved 
indefinitely at runtime. I.e it never changes on each execution of the native 
application
+logWarningIfNativeApplication();
+
 while (true) {
 try (ServerSocket ss = new ServerSocket()) {
 ss.setReuseAddress(true);
 ss.bind(new InetSocketAddress((InetAddress) null, 0), 1);
 
 int port = ss.getLocalPort();
 if (!isQuarkusReservedPort(port)) {
-LOGGER.info("getNextAvailable() -> {}", port);
-return port;
+String callerClassName = getCallerClassName();
+String value = RESERVED_PORTS.putIfAbsent(port, 
callerClassName);

Review comment:
   Is this making some assumptions about whether this class can 
concurrently exist in multiple JVMs or multiple class loaders within a single 
maven invocation? IIRC surefire is forking a new JVM for each Maven module. So 
if building with mvnd, the AvailablePortFinder class may exist in several 
concurrent JVMs. 




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3555: Improve available port discovery in tests



aldettinger commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808015374



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))
+.findFirst()

Review comment:
   So `findFirst()` returns the first class directly using 
`AvailablePortFinder` ? Would it work when we use `AvailablePortFinder` 2 times 
from the same class to reserve 2 distinct ports (if make sense) ?




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (aeb2f99 -> cf6c948)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


from aeb2f99  Regen for commit 5209fc65284c1faa91b8373ee8feeae2bd6f04f8
 add cf6c948  CAMEL-17655: Fix NPE in camel-opentracing

No new revisions were added by this update.

Summary of changes:
 .../java/org/apache/camel/opentracing/TagProcessor.java| 14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3555: Improve available port discovery in tests



aldettinger commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808015374



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))
+.findFirst()

Review comment:
   So `findFirst()` returns the first class directly using 
AvailablePortFinder ? Would it work when we use AvailablePortFinder 2 times 
from the same class to reserve 2 distinct ports (if make sense) ?




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] aldettinger commented on a change in pull request #3555: Improve available port discovery in tests



aldettinger commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r808010570



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))

Review comment:
   Filtering on className should be enough then.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] jamesnetherton commented on a change in pull request #3555: Improve available port discovery in tests



jamesnetherton commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r807999526



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))

Review comment:
   > Shouldn't we make AvailablePortFinder class final
   
   The class declaration is:
   
   ```java
   public final class AvailablePortFinder {
   }
   ```




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel] orpiske opened a new pull request #6959: More camel-package-maven-plugin cleanups



orpiske opened a new pull request #6959:
URL: https://github.com/apache/camel/pull/6959


   Also replace one slow method (sampled below as `testReadCustom`) with a 
faster one (`testJavaNativeWithAppend`). 
   
   Doesn't really help in the grand scheme of things, but the new code is 
cleaner. 
   
   ```
   MyBenchmark.testReadCustom   thrpt   15   466.568 ± 1.880  
ops/ms
   MyBenchmark.testJavaNativeWithAppend thrpt   15  1306.446 ± 9.873  
ops/ms
   ```
   
   
   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus] ppalaga commented on a change in pull request #3555: Improve available port discovery in tests



ppalaga commented on a change in pull request #3555:
URL: https://github.com/apache/camel-quarkus/pull/3555#discussion_r807992801



##
File path: 
integration-tests-support/test-support/src/main/java/org/apache/camel/quarkus/test/AvailablePortFinder.java
##
@@ -103,4 +124,19 @@ private static boolean isQuarkusReservedPort(int port) {
 }
 return false;
 }
+
+private static String getCallerClassName() {
+return 
StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE)
+.walk(s -> s.map(StackWalker.StackFrame::getClassName)
+.filter(className -> 
!className.equals(AvailablePortFinder.class.getName()))

Review comment:
   Shouldn't we make AvailablePortFinder class final, to better match the 
assumption made on the above line?




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus-examples] ppalaga commented on a change in pull request #76: Dynamically generate native test matrix



ppalaga commented on a change in pull request #76:
URL: 
https://github.com/apache/camel-quarkus-examples/pull/76#discussion_r807965901



##
File path: .github/workflows/ci-build.yaml
##
@@ -139,17 +127,15 @@ jobs:
 run: |
   tar -xzf ../maven-repo.tgz -C ~
   - name: Integration Tests
+env:
+  TEST_MODULES: ${{matrix.tests}}
+shell: bash
 run: |
-  EXAMPLE_PROJECTS=""
-  for EXAMPLE in $(yq -M -N e ".${{ matrix.category }}" 
.github/test-categories.yaml | cut -f2 -d' '); do
-if [ "${EXAMPLE}" == "null" ]; then
-  continue
-fi
-
+  for MODULE in ${TEST_MODULES//,/ }; do
 ./mvnw ${MAVEN_ARGS} clean verify \
   -Dformatter.skip -Dimpsort.skip \
   -Pnative,docker \
-  -f "${EXAMPLE}/pom.xml"
+  -f "${MODULE}/pom.xml"
   done

Review comment:
   Indeed, that would be nice to have. 




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch main updated: CAMEL-17655: Fix NPE in camel-opentracing

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
 new cf6c948  CAMEL-17655: Fix NPE in camel-opentracing
cf6c948 is described below

commit cf6c9488fc0f834ceec99e1c9229ffc814116a27
Author: Claus Ibsen 
AuthorDate: Wed Feb 16 14:19:36 2022 +0100

CAMEL-17655: Fix NPE in camel-opentracing
---
 .../java/org/apache/camel/opentracing/TagProcessor.java| 14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git 
a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
 
b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
index bd36b4d..0d980c6 100644
--- 
a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
+++ 
b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
@@ -53,12 +53,14 @@ public class TagProcessor extends AsyncProcessorSupport 
implements Traceable, Id
 public boolean process(Exchange exchange, AsyncCallback callback) {
 try {
 OpenTracingSpanAdapter camelSpan = (OpenTracingSpanAdapter) 
ActiveSpanManager.getSpan(exchange);
-Span span = camelSpan.getOpenTracingSpan();
-if (span != null) {
-String tag = expression.evaluate(exchange, String.class);
-span.setTag(tagName, tag);
-} else {
-LOG.warn("OpenTracing: could not find managed span for 
exchange={}", exchange);
+if (camelSpan != null) {
+Span span = camelSpan.getOpenTracingSpan();
+if (span != null) {
+String tag = expression.evaluate(exchange, String.class);
+span.setTag(tagName, tag);
+} else {
+LOG.warn("OpenTracing: cannot find managed span for 
exchange={}", exchange);
+}
 }
 } catch (Exception e) {
 exchange.setException(e);

[camel] branch camel-3.14.x updated: CAMEL-17655: Fix NPE in camel-opentracing

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch camel-3.14.x
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/camel-3.14.x by this push:
 new b734369  CAMEL-17655: Fix NPE in camel-opentracing
b734369 is described below

commit b734369f720480e19abe599136a67e4533c6408e
Author: Claus Ibsen 
AuthorDate: Wed Feb 16 14:19:36 2022 +0100

CAMEL-17655: Fix NPE in camel-opentracing
---
 .../java/org/apache/camel/opentracing/TagProcessor.java| 14 --
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git 
a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
 
b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
index b72bde7..99cc7d3 100644
--- 
a/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
+++ 
b/components/camel-opentracing/src/main/java/org/apache/camel/opentracing/TagProcessor.java
@@ -52,12 +52,14 @@ public class TagProcessor extends AsyncProcessorSupport 
implements Traceable, Id
 public boolean process(Exchange exchange, AsyncCallback callback) {
 try {
 OpenTracingSpanAdapter camelSpan = (OpenTracingSpanAdapter) 
ActiveSpanManager.getSpan(exchange);
-Span span = camelSpan.getOpenTracingSpan();
-if (span != null) {
-String tag = expression.evaluate(exchange, String.class);
-span.setTag(tagName, tag);
-} else {
-LOG.warn("OpenTracing: could not find managed span for 
exchange={}", exchange);
+if (camelSpan != null) {
+Span span = camelSpan.getOpenTracingSpan();
+if (span != null) {
+String tag = expression.evaluate(exchange, String.class);
+span.setTag(tagName, tag);
+} else {
+LOG.warn("OpenTracing: cannot find managed span for 
exchange={}", exchange);
+}
 }
 } catch (Exception e) {
 exchange.setException(e);

[GitHub] [camel-k] crosscode-nl opened a new issue #3019: cron.concurrency-policy does not work correctly



crosscode-nl opened a new issue #3019:
URL: https://github.com/apache/camel-k/issues/3019


   When using cron.concurrency-policy with a long running route it does 
functionally not matter which value is chosen: 
   
   For example, when running:
   
   ```console
   kamel run timer-schedule.xml -n camel-k --trait cron.concurrency-policy=Allow
   ```
   
   with route: 
   
   xml
   http://www.w3.org/2001/XMLSchema-instance;
   xmlns="http://camel.apache.org/schema/spring;
   xsi:schemaLocation="http://camel.apache.org/schema/spring 
http://camel.apache.org/schema/spring/camel-spring.xsd;>
   
   
   
   
   
   120
   
   
   6
   
   
   
   
   
   ```
   
   It will create the cron job with concurrency policy: Allow , but the route 
will be restarted every minute anyway. Same goes for Forbid. CronJob contains 
Forbid as concurrency policy, so that works but the route still stops every 
minute instead of going on. 
   
   What is the purpose of this trait if for any value this route gets stopped? 
Do I understand this feature correctly if I expect when Replace is set that the 
route is stopped and replaced with the new route, but when forbid is chosen 
nothing should happen to the route and no new route is started, and when allow 
is chosen that each time 1 minute is reached?
   



-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus-examples] jamesnetherton commented on a change in pull request #76: Dynamically generate native test matrix



jamesnetherton commented on a change in pull request #76:
URL: 
https://github.com/apache/camel-quarkus-examples/pull/76#discussion_r807903848



##
File path: .github/workflows/ci-build.yaml
##
@@ -139,17 +127,15 @@ jobs:
 run: |
   tar -xzf ../maven-repo.tgz -C ~
   - name: Integration Tests
+env:
+  TEST_MODULES: ${{matrix.tests}}
+shell: bash
 run: |
-  EXAMPLE_PROJECTS=""
-  for EXAMPLE in $(yq -M -N e ".${{ matrix.category }}" 
.github/test-categories.yaml | cut -f2 -d' '); do
-if [ "${EXAMPLE}" == "null" ]; then
-  continue
-fi
-
+  for MODULE in ${TEST_MODULES//,/ }; do
 ./mvnw ${MAVEN_ARGS} clean verify \
   -Dformatter.skip -Dimpsort.skip \
   -Pnative,docker \
-  -f "${EXAMPLE}/pom.xml"
+  -f "${MODULE}/pom.xml"
   done

Review comment:
   > Will this run all tests of the given group even if some non-last test 
fails? And will the GH Actions job fail if any of the mvnw invocations fails?
   
   I believe it will 'fail fast'. I.e the job will terminate on the first 
`mvnw` failure. To my knowledge, that's how its always worked in this project.
   
   Maybe we could try to be smart and report any failures at the end.




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (c6eeece -> aeb2f99)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


 discard c6eeece  Regen for commit cdbb31b75f7209618014eafb0bcfc25100279528
 add aeb2f99  Regen for commit 5209fc65284c1faa91b8373ee8feeae2bd6f04f8

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (c6eeece)
\
 N -- N -- N   refs/heads/regen_bot (aeb2f99)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:

[GitHub] [camel] github-actions[bot] closed pull request #6958: Generated sources regen



github-actions[bot] closed pull request #6958:
URL: https://github.com/apache/camel/pull/6958


   


-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [camel-quarkus-examples] ppalaga commented on a change in pull request #76: Dynamically generate native test matrix



ppalaga commented on a change in pull request #76:
URL: 
https://github.com/apache/camel-quarkus-examples/pull/76#discussion_r807869802



##
File path: .github/workflows/ci-build.yaml
##
@@ -139,17 +127,15 @@ jobs:
 run: |
   tar -xzf ../maven-repo.tgz -C ~
   - name: Integration Tests
+env:
+  TEST_MODULES: ${{matrix.tests}}
+shell: bash
 run: |
-  EXAMPLE_PROJECTS=""
-  for EXAMPLE in $(yq -M -N e ".${{ matrix.category }}" 
.github/test-categories.yaml | cut -f2 -d' '); do
-if [ "${EXAMPLE}" == "null" ]; then
-  continue
-fi
-
+  for MODULE in ${TEST_MODULES//,/ }; do
 ./mvnw ${MAVEN_ARGS} clean verify \
   -Dformatter.skip -Dimpsort.skip \
   -Pnative,docker \
-  -f "${EXAMPLE}/pom.xml"
+  -f "${MODULE}/pom.xml"
   done

Review comment:
   Will this run all tests of the given group even if some non-last test 
fails? And will the GH Actions job fail if any of the mvnw invocations fails?




-- 
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 specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[camel] branch regen_bot updated (fa16d70 -> c6eeece)

This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a change to branch regen_bot
in repository https://gitbox.apache.org/repos/asf/camel.git.


 discard fa16d70  Regen for commit 4a858f86ea636fce024334da53aff487628d6e19
 add cdbb31b  CAMEL-17656: camel-main - Add start and stop logging with app 
name and version
 add c6eeece  Regen for commit cdbb31b75f7209618014eafb0bcfc25100279528

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (fa16d70)
\
 N -- N -- N   refs/heads/regen_bot (c6eeece)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .../camel/impl/engine/AbstractCamelContext.java|  5 +-
 .../org/apache/camel/main/BaseMainSupport.java |  2 +-
 .../java/org/apache/camel/main/MainHelper.java | 58 ++
 .../java/org/apache/camel/main/MainSupport.java| 20 
 .../apache/camel/dsl/jbang/core/commands/Run.java  |  2 +-
 5 files changed, 82 insertions(+), 5 deletions(-)

[GitHub] [camel-quarkus] jamesnetherton opened a new issue #3554: AWS Lambda example tests should use mock event server