Re: [PR] Adding Jenkins file for PPC64LE JDK11 [camel-karaf]
oscerd commented on PR #251: URL: https://github.com/apache/camel-karaf/pull/251#issuecomment-1996841390 https://ci-builds.apache.org/job/Camel/job/Camel%20Karaf%20Build%203.21.x%20(ppc64le)/job/camel-3.21.x/1/console -- 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
Re: [PR] CAMEL-20095 : add langchain component [camel]
zbendhiba commented on PR #13341: URL: https://github.com/apache/camel/pull/13341#issuecomment-1996842171 yeah thanks everyone. -- 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
Re: [PR] CAMEL-20095 : add langchain component [camel]
zbendhiba merged PR #13341: URL: https://github.com/apache/camel/pull/13341 -- 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
Re: [PR] Bump com.sun.xml.bind:jaxb-core from 4.0.4 to 4.0.5 [camel]
davsclaus merged PR #13468: URL: https://github.com/apache/camel/pull/13468 -- 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
Re: [PR] Bump com.sun.xml.bind:jaxb-jxc from 4.0.4 to 4.0.5 [camel]
davsclaus merged PR #13477: URL: https://github.com/apache/camel/pull/13477 -- 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
Re: [PR] Bump org.jolokia:jolokia-agent-jvm from 2.0.1 to 2.0.2 [camel]
davsclaus merged PR #13470: URL: https://github.com/apache/camel/pull/13470 -- 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
Re: [PR] Bump aws-java-sdk2-version from 2.25.8 to 2.25.9 [camel]
davsclaus merged PR #13473: URL: https://github.com/apache/camel/pull/13473 -- 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
Re: [PR] Bump micrometer-tracing-version from 1.2.3 to 1.2.4 [camel]
davsclaus merged PR #13474: URL: https://github.com/apache/camel/pull/13474 -- 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
Re: [PR] Bump org.openapitools:openapi-generator from 7.3.0 to 7.4.0 [camel]
davsclaus merged PR #13469: URL: https://github.com/apache/camel/pull/13469 -- 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
Re: [PR] Adding Jenkins file for PPC64LE JDK11 [camel-karaf]
sanskruti1098 commented on PR #251: URL: https://github.com/apache/camel-karaf/pull/251#issuecomment-1996852862 Thank you! -- 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
[PR] (chores) camel-qdrant: disable tests on PPC64LE and s390x [camel]
orpiske opened a new pull request, #13479: URL: https://github.com/apache/camel/pull/13479 (no comment) -- 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
Re: [PR] (chores) camel-qdrant: disable tests on PPC64LE and s390x [camel]
github-actions[bot] commented on PR #13479: URL: https://github.com/apache/camel/pull/13479#issuecomment-1996860986 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
Re: [PR] Bump com.cloudbees.thirdparty:zendesk-java-client from 0.24.2 to 0.25.0 [camel]
davsclaus merged PR #13475: URL: https://github.com/apache/camel/pull/13475 -- 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
Re: [I] Integration builds fail with arm64 base image [camel-k]
tdiesler commented on issue #5238: URL: https://github.com/apache/camel-k/issues/5238#issuecomment-1996879169 During build time, we extract the base image digest from the manifest list, like this ... ``` ARM64_DIGEST := $(shell docker manifest inspect eclipse-temurin:17 | jq -r '.manifests[] | select(.platform.architecture == "arm64") | .digest') ``` We use that base image, if no other was explicitly defined ``` $ kamel install --olm=false --operator-image apache/camel-k:2.3.0-SNAPSHOT Using linux/arm64 base image: eclipse-temurin:17@sha256:321ac6af97a11d5aa5a580d1635ab6f945582b00eea1fe672f0cbb437b4293c8 Camel K installed in namespace default ``` -- 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
Re: [PR] Bump com.google.api-client:google-api-client from 2.3.0 to 2.4.0 [camel]
davsclaus merged PR #13472: URL: https://github.com/apache/camel/pull/13472 -- 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
Re: [PR] Bump mongo-java-driver-version from 4.11.1 to 5.0.0 [camel]
davsclaus merged PR #13476: URL: https://github.com/apache/camel/pull/13476 -- 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
Re: [I] Integration builds fail with arm64 base image [camel-k]
tdiesler commented on issue #5238: URL: https://github.com/apache/camel-k/issues/5238#issuecomment-1996900355 Slack: https://redhat-internal.slack.com/archives/C06DH6BJA80/p1710406352615669 -- 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
[PR] Generated sources regen [camel]
github-actions[bot] opened a new pull request, #13480: URL: https://github.com/apache/camel/pull/13480 Regen bot :robot: found some uncommitted changes after running build on :camel: `main` branch. Please do not delete `regen_bot` branch after merge/rebase. -- 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
Re: [PR] Generated sources regen [camel]
oscerd merged PR #13480: URL: https://github.com/apache/camel/pull/13480 -- 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
[PR] CAMEL-20548: include a capability section to advertise which artifact provides a specific feature [camel]
lburgazzoli opened a new pull request, #13481: URL: https://github.com/apache/camel/pull/13481 # Description # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [ ] I checked that each commit in the pull request has a meaningful subject line and body. - [ ] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
Re: [PR] CAMEL-20548: include a capability section to advertise which artifact provides a specific feature [camel]
lburgazzoli commented on code in PR #13481:
URL: https://github.com/apache/camel/pull/13481#discussion_r1524542644
##
catalog/camel-catalog/src/main/resources/org/apache/camel/catalog/capabilities.properties:
##
@@ -0,0 +1,17 @@
+## ---
+## 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.
+## ---
+platform-http=other/platform-http-main
Review Comment:
capabilities are defined as simple properties with the following syntax:
```
${capability-name} = ${kind}/${name}
```
It is enough for the runtimes to create such file and make the location
discoverable by overriding the `RuntimeProvider::getCapabilitiesCatalog`
--
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
Re: [PR] CAMEL-20548: include a capability section to advertise which artifact provides a specific feature [camel]
lburgazzoli commented on code in PR #13481: URL: https://github.com/apache/camel/pull/13481#discussion_r1524544103 ## tooling/camel-tooling-model/src/main/java/org/apache/camel/tooling/model/EntityRef.java: ## @@ -0,0 +1,24 @@ +/* + * 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.tooling.model; + +/** + * Reference a Camel entity. + */ +// TODO: the org.apache.camel.catalog.Kind should probably be moved in this package/module Review Comment: I don't know if it would break anything but it would be useful if the `org.apache.camel.catalog.Kind` enum would be moved in the `camel-tooling-model` module -- 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
[PR] feat(trait): use quarkus master feature [camel-k]
squakez opened a new pull request, #5248: URL: https://github.com/apache/camel-k/pull/5248 Enabling the feature via Camel Quarkus runtime directly. Closes #4048 **Release Note** ```release-note feat(trait): use quarkus master feature ``` -- 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
Re: [PR] feat(trait): use quarkus master feature [camel-k]
squakez closed pull request #5248: feat(trait): use quarkus master feature URL: https://github.com/apache/camel-k/pull/5248 -- 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
Re: [PR] feat(trait): use quarkus master feature [camel-k]
squakez commented on PR #5248: URL: https://github.com/apache/camel-k/pull/5248#issuecomment-1997039684 Created by mistake, sorry. -- 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
Re: [PR] feat(trait): use quarkus master feature [camel-k]
github-actions[bot] commented on PR #5248: URL: https://github.com/apache/camel-k/pull/5248#issuecomment-1997053198 :heavy_check_mark: Unit test coverage report - coverage increased from 37.2% to 37.4% (**+0.2%**) -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
lburgazzoli commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997061272 @christophd following the discussion in https://github.com/apache/camel-k/issues/5242, do we still need the property ? -- 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
[PR] fix: replace kebab-case with camelCase [camel-kamelets-examples]
tplevko opened a new pull request, #77: URL: https://github.com/apache/camel-kamelets-examples/pull/77 replace `set-header` and `set-body` with their camelCased counterparts -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
christophd commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997079362 @lburgazzoli I am currently trying to avoid the property right now and set the error handler ref directly on the route as Camel JBang would do. Unfortunately we would end up having both global error handler and route based error handler both referencing the same error handler. This is because we would like to support both Camel < 4.4.0 and 4.4.0+ -- 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
Re: [PR] fix: replace kebab-case with camelCase [camel-kamelets-examples]
oscerd merged PR #77: URL: https://github.com/apache/camel-kamelets-examples/pull/77 -- 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
Re: [PR] Integration builds fail with arm64 base image [camel-k]
lburgazzoli commented on code in PR #5246:
URL: https://github.com/apache/camel-k/pull/5246#discussion_r1524578978
##
pkg/util/defaults/defaults.go:
##
@@ -44,5 +44,10 @@ const (
installDefaultKamelets = true
)
+// Maps base image variants to their respective digests
+var baseImageVariantMapping = map[string]string{
Review Comment:
This may hijack a little bit the issue but this comment about runtime lookup
of the right base image make me thinking if we should do it for every
architecture and having the `IntegrationPlatform` reflecting the images in its
status.
The reason for that is something we never have discussed but it should be
possible to have a mixed kubernetes set-up where some nodes are amd64 and some
arm64 so you may have some kits working for some nodes and not for some others.
--
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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
lburgazzoli commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997087228 > @lburgazzoli I am currently trying to avoid the property right now and set the error handler ref directly on the route as Camel JBang would do. Unfortunately we would end up having both global error handler and route based error handler both referencing the same error handler. This is because we would like to support both Camel < 4.4.0 and 4.4.0+ is there anything we can do in the camel-k runtime maybe ? so in fact the operator won't have to change. -- 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
Re: [PR] Camel-AWS-Bedrock: Add documentation for models [camel]
github-actions[bot] commented on PR #13482: URL: https://github.com/apache/camel/pull/13482#issuecomment-1997095496 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
[PR] Camel-AWS-Bedrock: Add documentation for models [camel]
oscerd opened a new pull request, #13482: URL: https://github.com/apache/camel/pull/13482 # Description # Target - [x] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [x] I checked that each commit in the pull request has a meaningful subject line and body. - [x] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
[PR] Regen [camel]
lburgazzoli opened a new pull request, #13483: URL: https://github.com/apache/camel/pull/13483 # Description # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [ ] I checked that each commit in the pull request has a meaningful subject line and body. - [ ] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
[PR] Update quarkus platform to 3.2.10 [camel-k-runtime]
claudio4j opened a new pull request, #1184: URL: https://github.com/apache/camel-k-runtime/pull/1184 **Release Note** ```release-note Update quarkus platform to 3.2.10 ``` -- 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
Re: [PR] CAMEL-20522: camel-test - CamelTestSupport does not call tearDown for… [camel]
davsclaus closed pull request #13392: CAMEL-20522: camel-test - CamelTestSupport does not call tearDown for… URL: https://github.com/apache/camel/pull/13392 -- 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
Re: [PR] Regen [camel]
oscerd merged PR #13483: URL: https://github.com/apache/camel/pull/13483 -- 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
Re: [PR] Camel-AWS-Bedrock: Add documentation for models [camel]
oscerd merged PR #13482: URL: https://github.com/apache/camel/pull/13482 -- 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
Re: [PR] CAMEL-20521: camel-amqp - AMQP publisher application is losing messages with local JMS transaction enabled [camel]
luigidemasi commented on PR #13403: URL: https://github.com/apache/camel/pull/13403#issuecomment-1997168742 > Given the scope and criticality of this (as losing messages is truly a bad thing), I think it would be important to consider adding actual integration tests, instead of relying on mocks. This is particularly important if this can happen in other circumstances / other brokers, IMHO @orpiske The issue is not limited to the amqp component, I can reproduce it with plain camel-jms too, I added a test that reproduces the issue using a real Artemis broker running in a docker container. -- 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
Re: [PR] (chores) camel-qdrant: disable tests on PPC64LE and s390x [camel]
orpiske merged PR #13479: URL: https://github.com/apache/camel/pull/13479 -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
christophd commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997209935 > is there anything we can do in the camel-k runtime maybe ? so in fact the operator won't have to change. I try to explain my findings: So the proper fix in my eyes would be to set the error handler on the route (just like Camel JBang does) instead of adding it as a global error handler (this is what the Camel K operator does). This is because since Camel 4.4 the raised error is handled by the NoErrorHandler on the route by default. So we need to set the error handler on the route when running with Camel 4.4 onwards. But unfortunately the Camel YAML DSL won't let me do this as it only supports to add the error handler as a global one on the Camel context. So we would need to change this on the Camel YAML DSL first and then let the operator set the error handler on the route. An alternative to all that would be to set this `camel.component.kamelet.noErrorHandler=false` property on the `application.properties` which makes Camel 4.4 behave like it was before and the global error handler will handle the errors. Unfortunately Camel K operator is keen on supporting both runtimes Camel < 4.4.0 and 4.4.0+. So setting the property or adding the error handler to the route instead of global is dependent on the runtime version used to run the integration. Now you may give advice what and where to handle this. In the operator or in camel-k runtime layer. -- 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
Re: [I] Jasypt password sysenv: prefix leads to an NPE at build time if the environment variable does not exist [camel-quarkus]
jamesnetherton closed issue #5874: Jasypt password sysenv: prefix leads to an NPE at build time if the environment variable does not exist URL: https://github.com/apache/camel-quarkus/issues/5874 -- 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
Re: [PR] Avoid potential NPE when handling Jasypt password prefixes [camel-quarkus]
jamesnetherton merged PR #5875: URL: https://github.com/apache/camel-quarkus/pull/5875 -- 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
Re: [PR] Update quarkus platform to 3.2.10 [camel-k-runtime]
claudio4j merged PR #1184: URL: https://github.com/apache/camel-k-runtime/pull/1184 -- 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
Re: [PR] Revert camel-quarkus-version remove from PR apache#1157 [camel-k-runtime]
claudio4j commented on PR #1170: URL: https://github.com/apache/camel-k-runtime/pull/1170#issuecomment-1997264055 The nighly builds are fixed and can we close this PR ? -- 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
Re: [PR] Backport CAMEL-20518 [camel]
github-actions[bot] commented on PR #13484: URL: https://github.com/apache/camel/pull/13484#issuecomment-1997279519 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
[PR] Backport CAMEL-20518 [camel]
orpiske opened a new pull request, #13484: URL: https://github.com/apache/camel/pull/13484 (no comment) -- 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
[PR] [3.8.x] Upgrade Quarkus CXF to 3.8.1 [camel-quarkus]
zhfeng opened a new pull request, #5876: URL: https://github.com/apache/camel-quarkus/pull/5876 -- 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
Re: [PR] [3.8.x] Upgrade Quarkus CXF to 3.8.1 [camel-quarkus]
ppalaga commented on PR #5876: URL: https://github.com/apache/camel-quarkus/pull/5876#issuecomment-1997357119 Thanks @zhfeng I was just about to do this :) -- 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
Re: [PR] [3.8.x] Upgrade Quarkus CXF to 3.8.1 [camel-quarkus]
ppalaga commented on PR #5876: URL: https://github.com/apache/camel-quarkus/pull/5876#issuecomment-1997360241 @zhfeng are you going to send the same PR to main? -- 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
[PR] Upgrade to Quarkus CXF 3.8.1 [camel-quarkus]
zhfeng opened a new pull request, #5877: URL: https://github.com/apache/camel-quarkus/pull/5877 -- 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
Re: [PR] [3.8.x] Upgrade Quarkus CXF to 3.8.1 [camel-quarkus]
zhfeng commented on PR #5876: URL: https://github.com/apache/camel-quarkus/pull/5876#issuecomment-1997365211 Yeah, I open the same one to main #5877 -- 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
[PR] Upgrade Camel to 4.4.1 [camel-quarkus]
jamesnetherton opened a new pull request, #5878: URL: https://github.com/apache/camel-quarkus/pull/5878 (no comment) -- 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
[PR] CAMEL-20545: Using replaceFromWith with camel-test and having route t… [camel]
davsclaus opened a new pull request, #13485: URL: https://github.com/apache/camel/pull/13485 …emplates can lead to duplicate consumer on starutp error. # Description # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [ ] I checked that each commit in the pull request has a meaningful subject line and body. - [ ] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
Re: [PR] CAMEL-20545: Using replaceFromWith with camel-test and having route t… [camel]
github-actions[bot] commented on PR #13485: URL: https://github.com/apache/camel/pull/13485#issuecomment-1997456976 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
[PR] Jaxb properties should be present in parent/pom.xml CAMEL-20566 [camel]
JiriOndrusek opened a new pull request, #13486: URL: https://github.com/apache/camel/pull/13486 fixes https://issues.apache.org/jira/browse/CAMEL-20566 # Description # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [ ] I checked that each commit in the pull request has a meaningful subject line and body. - [ ] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
Re: [PR] Jaxb properties should be present in parent/pom.xml CAMEL-20566 [camel]
github-actions[bot] commented on PR #13486: URL: https://github.com/apache/camel/pull/13486#issuecomment-1997467256 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
Re: [PR] Jaxb properties should be present in parent/pom.xml CAMEL-20566 [camel]
JiriOndrusek commented on PR #13486: URL: https://github.com/apache/camel/pull/13486#issuecomment-1997468057 @davsclaus I moved groovy version as well, because I haven't found a reason why it should stay, WDYT? -- 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
Re: [PR] CAMEL-20545: Using replaceFromWith with camel-test and having route t… [camel]
davsclaus merged PR #13485: URL: https://github.com/apache/camel/pull/13485 -- 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
Re: [PR] Migration blog for camel-karaf users [camel-website]
davsclaus commented on PR #1083: URL: https://github.com/apache/camel-website/pull/1083#issuecomment-1997487209 I need to find some time post 4.5 release -- 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
Re: [PR] Updated rules.js [camel-website]
davsclaus commented on PR #995: URL: https://github.com/apache/camel-website/pull/995#issuecomment-1997486605 closing old PR that was not active -- 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
Re: [PR] Updated rules.js [camel-website]
davsclaus closed pull request #995: Updated rules.js URL: https://github.com/apache/camel-website/pull/995 -- 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
Re: [I] Basic example fails on Docker Desktop MacOS [camel-k]
tdiesler commented on issue #5095: URL: https://github.com/apache/camel-k/issues/5095#issuecomment-1997531129 Out-of-date -- 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
[PR] Upgrade GraalJS to 23.1.2 [camel]
jamesnetherton opened a new pull request, #13487: URL: https://github.com/apache/camel/pull/13487 (no comment) -- 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
Re: [PR] Upgrade GraalJS to 23.1.2 [camel]
github-actions[bot] commented on PR #13487: URL: https://github.com/apache/camel/pull/13487#issuecomment-1997535281 :robot: The Apache Camel test robot will run the tests for you :+1: -- 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
Re: [I] Basic example fails on Docker Desktop MacOS [camel-k]
tdiesler closed issue #5095: Basic example fails on Docker Desktop MacOS URL: https://github.com/apache/camel-k/issues/5095 -- 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
Re: [PR] Upgrade GraalJS to 23.1.2 [camel]
github-actions[bot] commented on PR #13487: URL: https://github.com/apache/camel/pull/13487#issuecomment-1997534017 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
Re: [PR] Upgrade GraalJS to 23.1.2 [camel]
jamesnetherton commented on PR #13487: URL: https://github.com/apache/camel/pull/13487#issuecomment-1997534170 /component-test javascript -- 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
Re: [I] Integration builds fail with arm64 base image [camel-k]
tdiesler commented on issue #5238: URL: https://github.com/apache/camel-k/issues/5238#issuecomment-1997537212 In this second attempt, we simply tell Jib to consider multiple platforms for the FROM image ``` -Djib.from.platforms=linux/amd64,linux/arm64 -Djib.from.image=eclipse-temurin:17 ``` I also upgraded to jib-3.4.1 -- 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
Re: [PR] Integration builds fail with arm64 base image [camel-k]
tdiesler commented on PR #5246: URL: https://github.com/apache/camel-k/pull/5246#issuecomment-1997538456 In this second attempt, we simply tell Jib to consider multiple platforms for the FROM image ``` -Djib.from.platforms=linux/amd64,linux/arm64 -Djib.from.image=eclipse-temurin:17 ``` I also upgraded to jib-3.4.1 -- 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
Re: [I] Integration builds fail with arm64 base image [camel-k]
gansheer commented on issue #5238: URL: https://github.com/apache/camel-k/issues/5238#issuecomment-1997553059 > In this second attempt, we simply tell Jib to consider multiple platforms for the FROM image > > ``` > -Djib.from.platforms=linux/amd64,linux/arm64 -Djib.from.image=eclipse-temurin:17 > ``` > > I also upgraded to jib-3.4.1 That's nice ! I guess that means by default we will create images compatible with both platforms. -- 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
Re: [I] Docker compose fails [camel-karavan]
mgubaidullin commented on issue #1173: URL: https://github.com/apache/camel-karavan/issues/1173#issuecomment-1997579354 It would be very helpful if you share docker-compose.yaml that you are using -- 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
Re: [I] Docker install fails [camel-karavan]
mgubaidullin commented on issue #1164: URL: https://github.com/apache/camel-karavan/issues/1164#issuecomment-1997581164 It would be very helpful if you share docker-compose.yaml that you are using -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
lburgazzoli commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997587158 my workaround-ish feeling would lead to register a Kamelet component in the `camel-k-runtime` with the `noErrorHandler` set to false so it would apply only to specific versions. This could probably buy some time to fix the Camel YAML DSL and reason about how this should be better solved, tough, I do not hold any strong opinion, what do you think ? -- 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
Re: [PR] Upgrade GraalJS to 23.1.2 [camel]
jamesnetherton merged PR #13487: URL: https://github.com/apache/camel/pull/13487 -- 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
Re: [PR] Upgrade to Quarkus CXF 3.8.1 [camel-quarkus]
zhfeng merged PR #5877: URL: https://github.com/apache/camel-quarkus/pull/5877 -- 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
Re: [PR] [3.8.x] Upgrade Quarkus CXF to 3.8.1 [camel-quarkus]
zhfeng merged PR #5876: URL: https://github.com/apache/camel-quarkus/pull/5876 -- 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
Re: [I] Integration builds fail with arm64 base image [camel-k]
tdiesler commented on issue #5238: URL: https://github.com/apache/camel-k/issues/5238#issuecomment-1997598404 @christophd could you perhaps verify whether this branch works for you? -- 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
[PR] Upgrade quarkus-jgit to 3.1.0 [camel-quarkus]
jamesnetherton opened a new pull request, #5879: URL: https://github.com/apache/camel-quarkus/pull/5879 (no comment) -- 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
[PR] Camel 20567 make it possible to define restconfiguration in xml-io-dsl and xml-jaxb-dsl [camel]
johnpoth opened a new pull request, #13488: URL: https://github.com/apache/camel/pull/13488 # Description Done in a similar way as the yaml-dsl, hope this is ok! Not sure why it's possible to define [multiple](https://github.com/apache/camel/blob/e6cede73c02c88d83ffd5876090e0072f9ea8e7b/core/camel-core-model/src/main/java/org/apache/camel/model/app/BeansDefinition.java#L92) restConfigurations in xml. It also seems that the xml-io-dsl can handle a root \ tag while the xml-jaxb-dsl cannot. Thanks ! # Target - [ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [ ] I checked that each commit in the pull request has a meaningful subject line and body. - [ ] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
[PR] chore(ci): Upgrade kind action to 2.0.4 [camel-k]
gansheer opened a new pull request, #5249: URL: https://github.com/apache/camel-k/pull/5249 **Release Note** ```release-note chore(ci): Upgrade kind action to 2.0.4 ``` -- 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
Re: [PR] Camel 20567 make it possible to define restconfiguration in xml-io-dsl and xml-jaxb-dsl [camel]
github-actions[bot] commented on PR #13488: URL: https://github.com/apache/camel/pull/13488#issuecomment-1997670763 :star2: Thank you for your contribution to the Apache Camel project! :star2: :robot: CI automation will test this PR automatically. :camel: Apache Camel Committers, please review the following items: * First-time contributors **require MANUAL approval** for the GitHub Actions to run * You can use the command `/component-test (camel-)component-name1 (camel-)component-name2..` to request a test from the test bot. * You can label PRs using `build-all`, `build-dependents`, `skip-tests` and `test-dependents` to fine-tune the checks executed by this PR. * Build and test logs are available in the Summary page. **Only** [Apache Camel committers](https://camel.apache.org/community/team/#committers) have access to the summary. * :warning: Be careful when sharing logs. Review their contents before sharing them publicly. -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
christophd commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997674194 @lburgazzoli if this is better received than the changes in this PR I am fine with it. It still is a Camel K specific workaround though -- 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
Re: [PR] Upgrade Camel to 4.4.1 [camel-quarkus]
jamesnetherton merged PR #5878: URL: https://github.com/apache/camel-quarkus/pull/5878 -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
lburgazzoli commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997700464 I think at this stage it would be better but it is gut feeling -- 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
[PR] [3.8.x] Backports [camel-quarkus]
jamesnetherton opened a new pull request, #5880: URL: https://github.com/apache/camel-quarkus/pull/5880 (no comment) -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
christophd closed pull request #5245: fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 URL: https://github.com/apache/camel-k/pull/5245 -- 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
Re: [PR] fix(#5242): Disable noErrorHandler setting for Camel 4.4.0 [camel-k]
christophd commented on PR #5245: URL: https://github.com/apache/camel-k/pull/5245#issuecomment-1997728926 @lburgazzoli no worries I am fine. let's close this PR 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
Re: [PR] Install operator image according to the target architecture [camel-k]
squakez merged PR #5200: URL: https://github.com/apache/camel-k/pull/5200 -- 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
Re: [PR] Install operator image according to the target architecture [camel-k]
squakez commented on PR #5200: URL: https://github.com/apache/camel-k/pull/5200#issuecomment-1997758990 Thanks, let's keep tuned to see the manifest during nightly build. -- 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
[PR] CAMEL-20568: Set error handler on route level in YAML DSL [camel]
christophd opened a new pull request, #13489: URL: https://github.com/apache/camel/pull/13489 # Description Allow to set error handler on route level in Camel YAML DSL # Target - [x] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [x] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). See CAMEL-20568 # Apache Camel coding standards and style - [x] I checked that each commit in the pull request has a meaningful subject line and body. - [x] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
Re: [I] Pipe error handler not working with Camel 4.4.0 [camel-k]
christophd commented on issue #5242: URL: https://github.com/apache/camel-k/issues/5242#issuecomment-1997771799 Relates to https://github.com/apache/camel/pull/13489 -- 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
Re: [PR] CAMEL-20568: Set error handler on route level in YAML DSL [camel]
christophd commented on PR #13489: URL: https://github.com/apache/camel/pull/13489#issuecomment-1997781029 @lburgazzoli 👀 🙏 -- 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
Re: [PR] Jaxb properties should be present in parent/pom.xml CAMEL-20566 [camel]
davsclaus merged PR #13486: URL: https://github.com/apache/camel/pull/13486 -- 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
[PR] Update year in NOTICE.txt [camel]
esivakumar26 opened a new pull request, #13490: URL: https://github.com/apache/camel/pull/13490 # Description Update year in NOTICE.txt # Target - [X] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch) # Tracking - [ ] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it). # Apache Camel coding standards and style - [X] I checked that each commit in the pull request has a meaningful subject line and body. - [X] I have run `mvn clean install -DskipTests` locally and I have committed all auto-generated changes -- 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
Re: [PR] Update year in NOTICE.txt [camel]
davsclaus merged PR #13490: URL: https://github.com/apache/camel/pull/13490 -- 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
Re: [PR] Camel 20567 make it possible to define restconfiguration in xml-io-dsl and xml-jaxb-dsl [camel]
davsclaus commented on PR #13488: URL: https://github.com/apache/camel/pull/13488#issuecomment-1997830840 Can you update this PR as there are modified files modified: dsl/camel-xml-io-dsl/src/test/java/org/apache/camel/dsl/xml/io/XmlLoadAppTest.java modified: dsl/camel-xml-jaxb-dsl-test/definition/src/test/java/org/apache/camel/dsl/xml/jaxb/definition/LoadRestConfigurationFromXmlTest.java -- 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
Re: [PR] Backport CAMEL-20518 [camel]
davsclaus merged PR #13484: URL: https://github.com/apache/camel/pull/13484 -- 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
Re: [PR] Upgrade quarkus-jgit to 3.1.0 [camel-quarkus]
jamesnetherton merged PR #5879: URL: https://github.com/apache/camel-quarkus/pull/5879 -- 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
