Re: [PR] Bump io.grpc:grpc-bom from 1.61.1 to 1.62.2 [camel]
github-actions[bot] commented on PR #13415: URL: https://github.com/apache/camel/pull/13415#issuecomment-1985215762 :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
(camel) branch dependabot/maven/io.grpc-grpc-bom-1.62.2 created (now 5ea83dc995c)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/io.grpc-grpc-bom-1.62.2 in repository https://gitbox.apache.org/repos/asf/camel.git at 5ea83dc995c Bump io.grpc:grpc-bom from 1.61.1 to 1.62.2 No new revisions were added by this update.
[PR] Bump io.grpc:grpc-bom from 1.61.1 to 1.62.2 [camel]
dependabot[bot] opened a new pull request, #13415: URL: https://github.com/apache/camel/pull/13415 Bumps [io.grpc:grpc-bom](https://github.com/grpc/grpc-java) from 1.61.1 to 1.62.2. Release notes Sourced from https://github.com/grpc/grpc-java/releases;>io.grpc:grpc-bom's releases. v1.62.2 gRPC Java 1.62.2 Release Notes Note that this is the initial 1.62.x release API Changes services: Remove io.grpc.services.BinaryLogs, which was deprecated since 2021. io.grpc.protobuf.services.BinaryLogs should be used instead (https://redirect.github.com/grpc/grpc-java/issues/10832;>#10832). Allow users outside of io.grpc.xds package to create custom xDS resources (https://redirect.github.com/grpc/grpc-java/issues/10834;>#10834) (6d96e6588) New Features api:Add ClientTransportFilter. Similarly to ServerTransportFilter, this will provide an observability hook and it allows direct modification of the transport's attributes. (https://redirect.github.com/grpc/grpc-java/issues/10646;>#10646) Improvements java_grpc_library.bzl: Add support for Auto Exec Groups (cb03bd234). This is mostly a behind-the-scenes change to adjust to the newer way Bazel operates java_grpc_library.bzl: Support runfiles for protoc and the plugin (65a6b3bc2). Neither binary uses runfiles, but the task will be ready if they need to in the future xds: Add EC key support for XdsChannelCredentials/XdsServerCredentials (100d5a55f) binder:Change log level from WARNING to FINER for expected exception during close with error, to reduce log spamming (https://redirect.github.com/grpc/grpc-java/issues/10899;>#10899) (7ba0718bb) Bug Fixes xds: Fix a bug in WeightedRoundRobinLoadBalancer policy that could raise NullPointerException and further cause channel panic when picking a subchannel. This bug can only be triggered when connection can not be established and the channel reports TRANSIENT_FAILURE state. (https://redirect.github.com/grpc/grpc-java/issues/10868;>#10868) Dependencies The protoc plugin no longer supports macOS Big Sur (macOS 11). Binaries are now built using Monterey (macOS 12) Acknowledgements https://github.com/joybestourous;>@joybestourous Commits https://github.com/grpc/grpc-java/commit/3e993a9f44ff52bd3d5ac59dfa978d8e7d30e28b;>3e993a9 Bump version to 1.62.1 https://github.com/grpc/grpc-java/commit/1da945ba543b187a9dab49587b2eb8ce4288580f;>1da945b Update README etc to reference 1.62.1 https://github.com/grpc/grpc-java/commit/7089f04718fad3912e9960f744a6f0008122bccc;>7089f04 Change GAE interop tests to use java11 runtime (https://redirect.github.com/grpc/grpc-java/issues/10933;>#10933) https://github.com/grpc/grpc-java/commit/597f26e8349845d8122a0f6f63e288996e828090;>597f26e Bump version to 1.62.1-SNAPSHOT https://github.com/grpc/grpc-java/commit/10eb91f1073b770a7f9a9c8ec96bf9917fd2eba4;>10eb91f Bump version to 1.62.0 https://github.com/grpc/grpc-java/commit/28dffe500ad970c09b24f75031854ecb0cefcf64;>28dffe5 Update README etc to reference 1.62.0 https://github.com/grpc/grpc-java/commit/5ba8b71137c456f163289a72cf5f56bd1b851bbf;>5ba8b71 util: MultiChildLoadBalance.shutdown() log to FINE (https://redirect.github.com/grpc/grpc-java/issues/10935;>#10935) https://github.com/grpc/grpc-java/commit/1795348b24e2c8457fdc030ce885aa1210afd451;>1795348 Remove semi-circular dependency between core and util https://github.com/grpc/grpc-java/commit/95b847e7995d8c9fed96fcad2cb39e882da917fc;>95b847e interop-testing: Use separate event loops in RetryTest https://github.com/grpc/grpc-java/commit/7ba0718bb9a61f27bc21ff26eb73bb8d7cb91f27;>7ba0718 Change log level from WARNING to FINER for expected exception (https://redirect.github.com/grpc/grpc-java/issues/10899;>#10899) Additional commits viewable in https://github.com/grpc/grpc-java/compare/v1.61.1...v1.62.2;>compare view [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it -
[I] jms-amqp-10-sink opens a new connection for each exchange [camel-kamelets]
kevinboone opened a new issue, #1936:
URL: https://github.com/apache/camel-kamelets/issues/1936
In a simple route like the one below, I observe that the `jms-amqp-10-sink`
is opening, using, and closing a new connection for each exchange that is
generated.
```
public class Hello extends RouteBuilder
{
@Override
public void configure() throws Exception {
from("timer:java?period=1000")
.setBody()
.simple ("Hello, World")
.log ("${body}")
.to
("kamelet:jms-amqp-10-sink?remoteURI=amqp://localhost:61616=foo");
}
}
```
It would be nice if connection could be pooled in some way. It would be even
nicer if the pooling parameters could be made configurable.
Tested on version 4.4.0 of Camel-JBang.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
(camel) branch dependabot/maven/jaxb-xjc-version-4.0.5 created (now 376abbc04c6)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/jaxb-xjc-version-4.0.5 in repository https://gitbox.apache.org/repos/asf/camel.git at 376abbc04c6 Bump jaxb-xjc-version from 4.0.4 to 4.0.5 No new revisions were added by this update.
[PR] Bump jaxb-xjc-version from 4.0.4 to 4.0.5 [camel]
dependabot[bot] opened a new pull request, #13414: URL: https://github.com/apache/camel/pull/13414 Bumps `jaxb-xjc-version` from 4.0.4 to 4.0.5. Updates `com.sun.xml.bind:jaxb-xjc` from 4.0.4 to 4.0.5 Updates `com.sun.xml.bind:jaxb-jxc` from 4.0.4 to 4.0.5 Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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 io.qdrant:client from 1.7.1 to 1.8.0 [camel]
github-actions[bot] commented on PR #13413: URL: https://github.com/apache/camel/pull/13413#issuecomment-1985210436 :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
(camel) branch dependabot/maven/io.qdrant-client-1.8.0 created (now bb9df672d25)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/io.qdrant-client-1.8.0 in repository https://gitbox.apache.org/repos/asf/camel.git at bb9df672d25 Bump io.qdrant:client from 1.7.1 to 1.8.0 No new revisions were added by this update.
[PR] Bump io.qdrant:client from 1.7.1 to 1.8.0 [camel]
dependabot[bot] opened a new pull request, #13413: URL: https://github.com/apache/camel/pull/13413 Bumps [io.qdrant:client](https://github.com/qdrant/java-client) from 1.7.1 to 1.8.0. Release notes Sourced from https://github.com/qdrant/java-client/releases;>io.qdrant:client's releases. v1.8.0 What's Changed Use Testcontainers Qdrant Module by https://github.com/eddumelendez;>@eddumelendez in https://redirect.github.com/qdrant/java-client/pull/24;>qdrant/java-client#24 chore: sort redundant static imports by https://github.com/Anush008;>@Anush008 in https://redirect.github.com/qdrant/java-client/pull/25;>qdrant/java-client#25 v1.8.0 by https://github.com/Anush008;>@Anush008 in https://redirect.github.com/qdrant/java-client/pull/26;>qdrant/java-client#26 Full Changelog: https://github.com/qdrant/java-client/compare/v1.7.2...v1.8.0;>https://github.com/qdrant/java-client/compare/v1.7.2...v1.8.0 v1.7.2 What's Changed refactor: Added method overloads with request objects by https://github.com/Anush008;>@Anush008 in https://redirect.github.com/qdrant/java-client/pull/22;>qdrant/java-client#22 Full Changelog: https://github.com/qdrant/java-client/compare/v1.7.1...v1.7.2;>https://github.com/qdrant/java-client/compare/v1.7.1...v1.7.2 Commits https://github.com/qdrant/java-client/commit/23cfac0e25d2f319e843c9b7f9c7178641a137e0;>23cfac0 v1.8.0 (https://redirect.github.com/qdrant/java-client/issues/26;>#26) https://github.com/qdrant/java-client/commit/1678c56bc5eaefdae9786358d76d77e2b8ed9b2b;>1678c56 chore: sort static imports (https://redirect.github.com/qdrant/java-client/issues/25;>#25) https://github.com/qdrant/java-client/commit/d1625cec782792f4551f490a5aa0653b5271a875;>d1625ce Use Testcontainers Qdrant Module (https://redirect.github.com/qdrant/java-client/issues/24;>#24) https://github.com/qdrant/java-client/commit/d18696581add8995a069c2f3c01d2840c5550971;>d186965 chore: Bump version 1.7.2 https://github.com/qdrant/java-client/commit/06b6c8da0503a423ee4dd4061f0bf913c4a7b5f5;>06b6c8d refactor: Add request object overloads (https://redirect.github.com/qdrant/java-client/issues/22;>#22) See full diff in https://github.com/qdrant/java-client/compare/v1.7.1...v1.8.0;>compare view [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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(traits): use Comparable matches [camel-k]
lburgazzoli commented on code in PR #5230:
URL: https://github.com/apache/camel-k/pull/5230#discussion_r1517349545
##
pkg/trait/builder.go:
##
@@ -56,9 +57,18 @@ func (t *builderTrait) InfluencesKit() bool {
return true
}
-// InfluencesBuild overrides base class method.
-func (t *builderTrait) InfluencesBuild(this, prev map[string]interface{}) bool
{
- return true
+func (t *builderTrait) Matches(trait Trait) bool {
+ otherTrait, ok := trait.(*builderTrait)
+ if !ok {
+ return false
+ }
+ if t.BaseImage != otherTrait.BaseImage || len(t.Properties) !=
len(otherTrait.Properties) {
+ return false
+ }
+ // More sofisticated check if len is the same. Sort and compare via
slices equal func.
+ slices.Sort(t.Properties)
+ slices.Sort(otherTrait.Properties)
Review Comment:
That would be a sensible option.
Another one, but would require some more work so I don't think it is worth
at this stage, could be to do something similar what it is done in the golang
standard library where there is often the possibility to pass a function for
equality, sorting, etc. like https://pkg.go.dev/slices#EqualFunc
--
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) 02/02: Disable Infinispan tests due to #5844
This is an automated email from the ASF dual-hosted git repository.
jamesnetherton pushed a commit to branch quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git
commit 81223288a196d54ae97476e959ce7ee1b514e72b
Author: James Netherton
AuthorDate: Wed Mar 6 16:06:31 2024 +
Disable Infinispan tests due to #5844
---
integration-tests/infinispan-quarkus-client/pom.xml | 2 ++
integration-tests/infinispan/pom.xml | 2 ++
.../apache/camel/quarkus/component/infinispan/InfinispanIT.java | 2 ++
.../camel/quarkus/component/infinispan/InfinispanTest.java | 1 +
poms/bom/pom.xml | 9 +
poms/bom/src/main/generated/flattened-full-pom.xml | 8
poms/bom/src/main/generated/flattened-reduced-pom.xml| 8
poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml| 8
8 files changed, 40 insertions(+)
diff --git a/integration-tests/infinispan-quarkus-client/pom.xml
b/integration-tests/infinispan-quarkus-client/pom.xml
index a87b90409d..d77f674203 100644
--- a/integration-tests/infinispan-quarkus-client/pom.xml
+++ b/integration-tests/infinispan-quarkus-client/pom.xml
@@ -70,6 +70,7 @@
+
virtualDependencies
diff --git a/integration-tests/infinispan/pom.xml
b/integration-tests/infinispan/pom.xml
index 90f2f30fc9..e5514af2dd 100644
--- a/integration-tests/infinispan/pom.xml
+++ b/integration-tests/infinispan/pom.xml
@@ -82,6 +82,7 @@
+
virtualDependencies
diff --git
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
index 25a4208dc0..f63f2c078d 100644
---
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
+++
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
@@ -17,7 +17,9 @@
package org.apache.camel.quarkus.component.infinispan;
import io.quarkus.test.junit.QuarkusIntegrationTest;
+import org.junit.jupiter.api.Disabled;
+@Disabled("https://github.com/apache/camel-quarkus/issues/5844;)
@QuarkusIntegrationTest
public class InfinispanIT extends InfinispanTest {
}
diff --git
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
index 637fa714a2..b469e343c3 100644
---
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
+++
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
@@ -26,6 +26,7 @@ import org.junit.jupiter.api.Test;
import static org.hamcrest.Matchers.is;
+@Disabled("https://github.com/apache/camel-quarkus/issues/5844;)
@QuarkusTest
@QuarkusTestResource(InfinispanServerTestResource.class)
public class InfinispanTest extends InfinispanCommonTest {
diff --git a/poms/bom/pom.xml b/poms/bom/pom.xml
index 2b6dd5ca23..30a1f81f01 100644
--- a/poms/bom/pom.xml
+++ b/poms/bom/pom.xml
@@ -1426,6 +1426,15 @@
camel-infinispan
${camel.version}
+
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+org.infinispan
+
infinispan-client-hotrod-jakarta
+
org.slf4j
jcl-over-slf4j
diff --git a/poms/bom/src/main/generated/flattened-full-pom.xml
b/poms/bom/src/main/generated/flattened-full-pom.xml
index 9aab1bc7d7..ef1499de74 100644
--- a/poms/bom/src/main/generated/flattened-full-pom.xml
+++ b/poms/bom/src/main/generated/flattened-full-pom.xml
@@ -1364,6 +1364,14 @@
camel-infinispan
4.4.0
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+org.infinispan
+infinispan-client-hotrod-jakarta
+
org.slf4j
jcl-over-slf4j
diff --git a/poms/bom/src/main/generated/flattened-reduced-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-pom.xml
index 0fa7ef209c..b0b7abfca9 100644
--- a/poms/bom/src/main/generated/flattened-reduced-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-pom.xml
@@ -1364,6 +1364,14 @@
camel-infinispan
4.4.0
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+
(camel-quarkus) branch quarkus-main updated (3d5d391076 -> 81223288a1)
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a change to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git discard 3d5d391076 Disable Infinispan tests due to #5844 discard 4fbc6774db Upgrade Quarkus to 3.9.0.CR1 new 7dbb9d484f Upgrade Quarkus to 3.9.0.CR1 new 81223288a1 Disable Infinispan tests due to #5844 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 (3d5d391076) \ N -- N -- N refs/heads/quarkus-main (81223288a1) 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: extensions/opentelemetry/deployment/pom.xml | 5 + 1 file changed, 5 insertions(+)
(camel-quarkus) 01/02: Upgrade Quarkus to 3.9.0.CR1
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a commit to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git commit 7dbb9d484f00e399ed060dcefbe2c630b08dc7c3 Author: James Netherton AuthorDate: Wed Feb 28 08:13:46 2024 + Upgrade Quarkus to 3.9.0.CR1 --- extensions/google-bigquery/deployment/pom.xml | 4 extensions/google-bigquery/runtime/pom.xml| 4 extensions/google-pubsub/deployment/pom.xml | 4 extensions/google-pubsub/runtime/pom.xml | 4 extensions/grpc/deployment/pom.xml| 4 extensions/grpc/runtime/pom.xml | 8 extensions/opentelemetry/deployment/pom.xml | 5 + extensions/salesforce/deployment/pom.xml | 4 extensions/salesforce/runtime/pom.xml | 4 .../jdbc/db2/src/test/resources/container-license-acceptance.txt | 2 +- pom.xml | 8 poms/bom/src/main/generated/flattened-full-pom.xml| 2 +- poms/bom/src/main/generated/flattened-reduced-pom.xml | 2 +- poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml | 2 +- .../java/org/apache/camel/quarkus/test/AfterEachCallback.java | 1 + .../java/org/apache/camel/quarkus/test/BeforeEachCallback.java| 1 + .../test/userTestCases/AdviceInDoBeforeEachMethodsTest.java | 6 +++--- 17 files changed, 54 insertions(+), 11 deletions(-) diff --git a/extensions/google-bigquery/deployment/pom.xml b/extensions/google-bigquery/deployment/pom.xml index e4cb3b702f..193d2935dc 100644 --- a/extensions/google-bigquery/deployment/pom.xml +++ b/extensions/google-bigquery/deployment/pom.xml @@ -54,6 +54,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/google-bigquery/runtime/pom.xml b/extensions/google-bigquery/runtime/pom.xml index 15a4b77477..9dfe0da15d 100644 --- a/extensions/google-bigquery/runtime/pom.xml +++ b/extensions/google-bigquery/runtime/pom.xml @@ -90,6 +90,10 @@ io.grpc grpc-netty + +com.google.code.findbugs +jsr305 + com.google.android annotations diff --git a/extensions/google-pubsub/deployment/pom.xml b/extensions/google-pubsub/deployment/pom.xml index 31b8b31739..3820cbf30b 100644 --- a/extensions/google-pubsub/deployment/pom.xml +++ b/extensions/google-pubsub/deployment/pom.xml @@ -58,6 +58,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/google-pubsub/runtime/pom.xml b/extensions/google-pubsub/runtime/pom.xml index 9161883ace..437a7ebb91 100644 --- a/extensions/google-pubsub/runtime/pom.xml +++ b/extensions/google-pubsub/runtime/pom.xml @@ -113,6 +113,10 @@ com.google.android annotations + +com.google.code.findbugs +jsr305 + commons-logging commons-logging diff --git a/extensions/grpc/deployment/pom.xml b/extensions/grpc/deployment/pom.xml index db1b712e67..e606eeb03e 100644 --- a/extensions/grpc/deployment/pom.xml +++ b/extensions/grpc/deployment/pom.xml @@ -50,6 +50,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/grpc/runtime/pom.xml b/extensions/grpc/runtime/pom.xml index bc4e5e3974..b58d4f005d 100644 --- a/extensions/grpc/runtime/pom.xml +++ b/extensions/grpc/runtime/pom.xml @@ -39,6 +39,10 @@ io.quarkus quarkus-grpc-common + +com.google.code.findbugs +jsr305 + io.quarkus quarkus-vertx @@ -55,6 +59,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/opentelemetry/deployment/pom.xml
(camel-quarkus) branch 3.8.x updated: Upgrade Quarkus to 3.8.2
This is an automated email from the ASF dual-hosted git repository.
jamesnetherton pushed a commit to branch 3.8.x
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git
The following commit(s) were added to refs/heads/3.8.x by this push:
new 44b7e83b82 Upgrade Quarkus to 3.8.2
44b7e83b82 is described below
commit 44b7e83b827f5ca1ce99aed5a0eb09c9341d060a
Author: James Netherton
AuthorDate: Thu Mar 7 13:23:26 2024 +
Upgrade Quarkus to 3.8.2
---
docs/antora.yml | 2 +-
pom.xml | 4 ++--
poms/bom/src/main/generated/flattened-full-pom.xml| 2 +-
poms/bom/src/main/generated/flattened-reduced-pom.xml | 2 +-
poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml | 2 +-
5 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/docs/antora.yml b/docs/antora.yml
index 0db87b5b6c..4fb65df67d 100644
--- a/docs/antora.yml
+++ b/docs/antora.yml
@@ -28,7 +28,7 @@ asciidoc:
camel-version: 4.4.0 # replace ${camel.version}
camel-docs-version: 4.4.x
camel-quarkus-version: 3.8.1 # replace ${camel-quarkus.version}
-quarkus-version: 3.8.1 # replace ${quarkus.version}
+quarkus-version: 3.8.2 # replace ${quarkus.version}
graalvm-version: 23.0.1 # replace ${graalvm.version}
graalvm-docs-version: jdk21 # replace ${graalvm-docs.version}
mapstruct-version: 1.5.5.Final # replace ${mapstruct.version}
diff --git a/pom.xml b/pom.xml
index b8c520f0b1..57259b8b64 100644
--- a/pom.xml
+++ b/pom.xml
@@ -61,7 +61,7 @@
2.2.1
2.3.0
2.0.2
-3.8.1
+3.8.2
4.0.0
2.6.0
@@ -148,7 +148,7 @@
3.4.34
${reactor-netty-version}
2.9.0
-
4.16.2
+
4.18.0
2.2
${snakeyaml-engine-version}
${spring-version}
diff --git a/poms/bom/src/main/generated/flattened-full-pom.xml
b/poms/bom/src/main/generated/flattened-full-pom.xml
index 99c9ac2584..981929a787 100644
--- a/poms/bom/src/main/generated/flattened-full-pom.xml
+++ b/poms/bom/src/main/generated/flattened-full-pom.xml
@@ -6452,7 +6452,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
diff --git a/poms/bom/src/main/generated/flattened-reduced-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-pom.xml
index 948f15cf41..d8190432e2 100644
--- a/poms/bom/src/main/generated/flattened-reduced-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-pom.xml
@@ -6407,7 +6407,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
diff --git a/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
index 830b87bccb..1c8a4baa52 100644
--- a/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
@@ -6407,7 +6407,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
Re: [PR] [3.8.x] Upgrade Quarkus to 3.8.2 [camel-quarkus]
jamesnetherton merged PR #5852: URL: https://github.com/apache/camel-quarkus/pull/5852 -- 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(traits): use Comparable matches [camel-k]
squakez commented on code in PR #5230:
URL: https://github.com/apache/camel-k/pull/5230#discussion_r1517281977
##
pkg/trait/builder.go:
##
@@ -56,9 +57,18 @@ func (t *builderTrait) InfluencesKit() bool {
return true
}
-// InfluencesBuild overrides base class method.
-func (t *builderTrait) InfluencesBuild(this, prev map[string]interface{}) bool
{
- return true
+func (t *builderTrait) Matches(trait Trait) bool {
+ otherTrait, ok := trait.(*builderTrait)
+ if !ok {
+ return false
+ }
+ if t.BaseImage != otherTrait.BaseImage || len(t.Properties) !=
len(otherTrait.Properties) {
+ return false
+ }
+ // More sofisticated check if len is the same. Sort and compare via
slices equal func.
+ slices.Sort(t.Properties)
+ slices.Sort(otherTrait.Properties)
Review Comment:
Yes, I see. For the sake of making the thing in a clean way, I can copy the
array and sorting/comparing them. It should not be too heavy from a computation
pov and it will save us against remote (but still possible) inconsistencies.
--
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 jetty-version from 12.0.6 to 12.0.7 [camel]
github-actions[bot] commented on PR #13412: URL: https://github.com/apache/camel/pull/13412#issuecomment-1985153231 :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
(camel) branch dependabot/maven/jetty-version-12.0.7 created (now 711bad1085a)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/jetty-version-12.0.7 in repository https://gitbox.apache.org/repos/asf/camel.git at 711bad1085a Bump jetty-version from 12.0.6 to 12.0.7 No new revisions were added by this update.
[PR] Bump jetty-version from 12.0.6 to 12.0.7 [camel]
dependabot[bot] opened a new pull request, #13412: URL: https://github.com/apache/camel/pull/13412 Bumps `jetty-version` from 12.0.6 to 12.0.7. Updates `org.eclipse.jetty.ee10:jetty-ee10-webapp` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.ee10:jetty-ee10-servlet` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-server` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.ee10.websocket:jetty-ee10-websocket-jakarta-server` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.websocket:jetty-websocket-jetty-server` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-security` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.ee10:jetty-ee10-servlets` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-client` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-jmx` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-util` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.http2:jetty-http2-client` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.http2:jetty-http2-client-transport` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-util-ajax` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-io` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty:jetty-proxy` from 12.0.6 to 12.0.7 Updates `org.eclipse.jetty.ee9:jetty-ee9-servlet` from 12.0.6 to 12.0.7 Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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(traits): use Comparable matches [camel-k]
lburgazzoli commented on code in PR #5230:
URL: https://github.com/apache/camel-k/pull/5230#discussion_r1517247695
##
pkg/trait/builder.go:
##
@@ -56,9 +57,18 @@ func (t *builderTrait) InfluencesKit() bool {
return true
}
-// InfluencesBuild overrides base class method.
-func (t *builderTrait) InfluencesBuild(this, prev map[string]interface{}) bool
{
- return true
+func (t *builderTrait) Matches(trait Trait) bool {
+ otherTrait, ok := trait.(*builderTrait)
+ if !ok {
+ return false
+ }
+ if t.BaseImage != otherTrait.BaseImage || len(t.Properties) !=
len(otherTrait.Properties) {
+ return false
+ }
+ // More sofisticated check if len is the same. Sort and compare via
slices equal func.
+ slices.Sort(t.Properties)
+ slices.Sort(otherTrait.Properties)
Review Comment:
The structure backing the two object is definitively different, my point was
more on the fact if `Matches` is or will be used concurrently, then it may
result in an undefined behavior.
Given the current PR I don't think it can happens but since the method is a
public method part of some of the traits, then it may be used also outside the
current scope and I did want to raise it to avoid falling in the same trap as
https://github.com/apache/camel-k/pull/5188#discussion_r1512280139
--
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] chore(e2e): Improve E2E tests [camel-k]
christophd commented on PR #5188: URL: https://github.com/apache/camel-k/pull/5188#issuecomment-1985141422 > Minikube with 16 GB or RAM and 4 CPUs should be good. At least, it works for me. Hm, quite heavy setup for me as I am running on MacOS and need to run this in Docker desktop that also requires quite some memory -- 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 (534c0a3c1dc -> fadc74f0c52)
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 534c0a3c1dc CAMEL-20532: fix SJMS tests add fadc74f0c52 Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things (#13411) No new revisions were added by this update. Summary of changes: .../components/aws-bedrock-agent-runtime.json | 27 +- .../agentruntime/aws-bedrock-agent-runtime.json| 27 +- .../component/aws2/bedrock/BedrockModels.java | 5 +- .../BedrockAgentRuntimeConfiguration.java | 15 +++--- .../agentruntime/BedrockAgentRuntimeConstants.java | 12 - .../agentruntime/BedrockAgentRuntimeProducer.java | 16 -- .../integration/BedrockAgentRuntimeProducerIT.java | 4 -- ...BedrockAgentRuntimeComponentBuilderFactory.java | 19 --- .../BedrockAgentRuntimeEndpointBuilderFactory.java | 61 +++--- 9 files changed, 114 insertions(+), 72 deletions(-)
(camel) branch enrich-retrieval deleted (was f009f8fa480)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch enrich-retrieval in repository https://gitbox.apache.org/repos/asf/camel.git was f009f8fa480 Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
(camel) branch main updated (534c0a3c1dc -> fadc74f0c52)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/camel.git from 534c0a3c1dc CAMEL-20532: fix SJMS tests add fadc74f0c52 Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things (#13411) No new revisions were added by this update. Summary of changes: .../components/aws-bedrock-agent-runtime.json | 27 +- .../agentruntime/aws-bedrock-agent-runtime.json| 27 +- .../component/aws2/bedrock/BedrockModels.java | 5 +- .../BedrockAgentRuntimeConfiguration.java | 15 +++--- .../agentruntime/BedrockAgentRuntimeConstants.java | 12 - .../agentruntime/BedrockAgentRuntimeProducer.java | 16 -- .../integration/BedrockAgentRuntimeProducerIT.java | 4 -- ...BedrockAgentRuntimeComponentBuilderFactory.java | 19 --- .../BedrockAgentRuntimeEndpointBuilderFactory.java | 61 +++--- 9 files changed, 114 insertions(+), 72 deletions(-)
Re: [PR] Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate re… [camel]
oscerd merged PR #13411: URL: https://github.com/apache/camel/pull/13411 -- 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 (f0357f63e9b -> 534c0a3c1dc)
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 f0357f63e9b camel-jbang - Upgrade to Q 3.8.2
add 10addb01a3e CAMEL-20532: prevent shutting down static services
add 534c0a3c1dc CAMEL-20532: fix SJMS tests
No new revisions were added by this update.
Summary of changes:
.../component/sjms/ReconnectConsumerTest.java | 20 +++-
.../component/sjms/ReconnectInOutProducerTest.java | 15 ++-
.../component/sjms/ReconnectProducerTest.java | 15 ++-
...sTestSupport.java => JmsCommonTestSupport.java} | 95 ---
.../sjms/support/JmsExclusiveTestSupport.java | 80 +
.../component/sjms/support/JmsTestSupport.java | 133 ++---
.../artemis/services/ArtemisServiceFactory.java| 8 +-
.../infra/common/services/SingletonService.java| 13 +-
.../kafka/services/ContainerLocalKafkaService.java | 1 +
.../infra/kafka/services/KafkaServiceFactory.java | 12 ++
10 files changed, 209 insertions(+), 183 deletions(-)
copy
components/camel-sjms/src/test/java/org/apache/camel/component/sjms/support/{JmsTestSupport.java
=> JmsCommonTestSupport.java} (82%)
create mode 100644
components/camel-sjms/src/test/java/org/apache/camel/component/sjms/support/JmsExclusiveTestSupport.java
Re: [PR] chore(e2e): Improve E2E tests [camel-k]
squakez commented on PR #5188: URL: https://github.com/apache/camel-k/pull/5188#issuecomment-1985108943 Documentation to reflect the changes in https://camel.apache.org/camel-k/2.2.x/contributing/e2e.html would be more than welcome. -- 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] chore(e2e): Improve E2E tests [camel-k]
squakez commented on PR #5188: URL: https://github.com/apache/camel-k/pull/5188#issuecomment-1985106790 > @squakez I tried to run the native tests locally which ends up in timeouts while building the integrations. I tested this also on the main branch with the same result. So I guess my local setup is not ready to run these native E2E tests. Do you have any working setup or information how to properly run the tests locally? Minikube with 16 GB or RAM and 4 CPUs should be good. At least, it works for me. -- 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 (f0357f63e9b -> 534c0a3c1dc)
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 f0357f63e9b camel-jbang - Upgrade to Q 3.8.2
add 10addb01a3e CAMEL-20532: prevent shutting down static services
add 534c0a3c1dc CAMEL-20532: fix SJMS tests
No new revisions were added by this update.
Summary of changes:
.../component/sjms/ReconnectConsumerTest.java | 20 +++-
.../component/sjms/ReconnectInOutProducerTest.java | 15 ++-
.../component/sjms/ReconnectProducerTest.java | 15 ++-
...sTestSupport.java => JmsCommonTestSupport.java} | 95 ---
.../sjms/support/JmsExclusiveTestSupport.java | 80 +
.../component/sjms/support/JmsTestSupport.java | 133 ++---
.../artemis/services/ArtemisServiceFactory.java| 8 +-
.../infra/common/services/SingletonService.java| 13 +-
.../kafka/services/ContainerLocalKafkaService.java | 1 +
.../infra/kafka/services/KafkaServiceFactory.java | 12 ++
10 files changed, 209 insertions(+), 183 deletions(-)
copy
components/camel-sjms/src/test/java/org/apache/camel/component/sjms/support/{JmsTestSupport.java
=> JmsCommonTestSupport.java} (82%)
create mode 100644
components/camel-sjms/src/test/java/org/apache/camel/component/sjms/support/JmsExclusiveTestSupport.java
Re: [PR] CAMEL-20532: prevent shutting down static services [camel]
orpiske merged PR #13409: URL: https://github.com/apache/camel/pull/13409 -- 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(traits): use Comparable matches [camel-k]
squakez commented on code in PR #5230:
URL: https://github.com/apache/camel-k/pull/5230#discussion_r1517210674
##
pkg/trait/builder.go:
##
@@ -56,9 +57,18 @@ func (t *builderTrait) InfluencesKit() bool {
return true
}
-// InfluencesBuild overrides base class method.
-func (t *builderTrait) InfluencesBuild(this, prev map[string]interface{}) bool
{
- return true
+func (t *builderTrait) Matches(trait Trait) bool {
+ otherTrait, ok := trait.(*builderTrait)
+ if !ok {
+ return false
+ }
+ if t.BaseImage != otherTrait.BaseImage || len(t.Properties) !=
len(otherTrait.Properties) {
+ return false
+ }
+ // More sofisticated check if len is the same. Sort and compare via
slices equal func.
+ slices.Sort(t.Properties)
+ slices.Sort(otherTrait.Properties)
Review Comment:
Yes, but in this case I don't think it's a problem. This func is not
directly used by the trait per se but is a supporting function used in the
`matchesComparableTrait` func. Here what we do is to create 2 brand new traits
which are only used for comparison, so, I tend to think they are not really
affecting the rest of the execution as they are 2 new objects:
https://github.com/apache/camel-k/pull/5230/files#diff-80f4821d91dcccf8c11c32962f56b41b7dcfccadeb6366df3cf49baa55188ba1R637-R658
I think the structure backing the two object is different. But I may be
wrong. If you feel we need to prove it I can try to develop a unit test to make
sure this is not happening. Thanks.
--
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 Agent Runtime: Enriching the retrieveAndGenerate re… [camel]
github-actions[bot] commented on PR #13411: URL: https://github.com/apache/camel/pull/13411#issuecomment-1985098842 :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: [PR] Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate re… [camel]
oscerd commented on PR #13411: URL: https://github.com/apache/camel/pull/13411#issuecomment-1985098364 /component-test aws -- 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 Agent Runtime: Enriching the retrieveAndGenerate re… [camel]
oscerd opened a new pull request, #13411: URL: https://github.com/apache/camel/pull/13411 …sponse and some minor things # 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 - [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). # 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] Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate re… [camel]
github-actions[bot] commented on PR #13411: URL: https://github.com/apache/camel/pull/13411#issuecomment-1985098232 :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
(camel) branch enrich-retrieval created (now f009f8fa480)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch enrich-retrieval in repository https://gitbox.apache.org/repos/asf/camel.git at f009f8fa480 Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things This branch includes the following new commits: new f009f8fa480 Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things The 1 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.
(camel) 01/01: Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response and some minor things
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch enrich-retrieval
in repository https://gitbox.apache.org/repos/asf/camel.git
commit f009f8fa4803519a7d43daa0aee9b59b02d72c66
Author: Andrea Cosentino
AuthorDate: Fri Mar 8 07:11:30 2024 +0100
Camel-AWS-Bedrock Agent Runtime: Enriching the retrieveAndGenerate response
and some minor things
Signed-off-by: Andrea Cosentino
---
.../components/aws-bedrock-agent-runtime.json | 27 +-
.../agentruntime/aws-bedrock-agent-runtime.json| 27 +-
.../component/aws2/bedrock/BedrockModels.java | 5 +-
.../BedrockAgentRuntimeConfiguration.java | 15 +++---
.../agentruntime/BedrockAgentRuntimeConstants.java | 12 -
.../agentruntime/BedrockAgentRuntimeProducer.java | 16 --
.../integration/BedrockAgentRuntimeProducerIT.java | 4 --
...BedrockAgentRuntimeComponentBuilderFactory.java | 19 ---
.../BedrockAgentRuntimeEndpointBuilderFactory.java | 61 +++---
9 files changed, 114 insertions(+), 72 deletions(-)
diff --git
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws-bedrock-agent-runtime.json
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws-bedrock-agent-runtime.json
index 94f3e81f90d..a22825c743c 100644
---
a/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws-bedrock-agent-runtime.json
+++
b/catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws-bedrock-agent-runtime.json
@@ -26,22 +26,22 @@
"configuration": { "index": 0, "kind": "property", "displayName":
"Configuration", "group": "producer", "label": "", "required": false, "type":
"object", "javaType":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"deprecated": false, "autowired": false, "secret": false, "description":
"Component configuration" },
"knowledgeBaseId": { "index": 1, "kind": "property", "displayName":
"Knowledge Base Id", "group": "producer", "label": "", "required": true,
"type": "string", "javaType": "java.lang.String", "deprecated": false,
"deprecationNote": "", "autowired": false, "secret": false,
"configurationClass":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"configurationField": "configuration", "description": "Define the Knowledge
Base Id we are going to use" },
"lazyStartProducer": { "index": 2, "kind": "property", "displayName":
"Lazy Start Producer", "group": "producer", "label": "producer", "required":
false, "type": "boolean", "javaType": "boolean", "deprecated": false,
"autowired": false, "secret": false, "defaultValue": false, "description":
"Whether the producer should be started lazy (on the first message). By
starting lazy you can use this to allow CamelContext and routes to startup in
situations where a producer may otherwise fail [...]
-"modelId": { "index": 3, "kind": "property", "displayName": "Model Id",
"group": "producer", "label": "", "required": true, "type": "string",
"javaType": "java.lang.String", "enum": [ "amazon.titan-text-express-v1",
"amazon.titan-text-lite-v1", "amazon.titan-image-generator-v1",
"amazon.titan-embed-text-v1", "ai21.j2-ultra-v1", "ai21.j2-mid-v1" ],
"deprecated": false, "deprecationNote": "", "autowired": false, "secret":
false, "configurationClass": "org.apache.camel.component.aws2.be [...]
+"modelId": { "index": 3, "kind": "property", "displayName": "Model Id",
"group": "producer", "label": "", "required": true, "type": "string",
"javaType": "java.lang.String", "enum": [ "anthropic.claude-instant-v1",
"anthropic.claude-v2", "anthropic.claude-v2:1" ], "deprecated": false,
"deprecationNote": "", "autowired": false, "secret": false,
"configurationClass":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"configurationField": "configur [...]
"operation": { "index": 4, "kind": "property", "displayName": "Operation",
"group": "producer", "label": "", "required": true, "type": "object",
"javaType":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeOperations",
"enum": [ "retrieveAndGenerate" ], "deprecated": false, "deprecationNote": "",
"autowired": false, "secret": false, "configurationClass":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"configurationFiel [...]
"overrideEndpoint": { "index": 5, "kind": "property", "displayName":
"Override Endpoint", "group": "producer", "label": "", "required": false,
"type": "boolean", "javaType": "boolean", "deprecated": false, "autowired":
false, "secret": false, "defaultValue": false, "configurationClass":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"configurationField": "configuration", "description": "Set the need for
Re: [PR] fix(traits): use Comparable matches [camel-k]
squakez commented on PR #5230: URL: https://github.com/apache/camel-k/pull/5230#issuecomment-1985096678 ``` === NAME TestNativeHighMemoryIntegrations native_with_sources_test.go:60: Timed out after 60.000s. Expected : 2024-03-07 17:04:07,602 INFO [org.apa.cam.k.Runtime] (main) Apache Camel K Runtime 3.2.3 2024-03-07 17:04:07,602 INFO [org.apa.cam.qua.cor.CamelBootstrapRecorder] (main) Bootstrap runtime: org.apache.camel.quarkus.main.CamelMainRuntime 2024-03-07 17:04:07,602 INFO [org.apa.cam.mai.MainSupport] (main) Apache Camel (Main) 4.0.2 is starting 2024-03-07 17:04:07,610 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (main) Apache Camel 4.0.2 (camel-1) is starting 2024-03-07 17:04:07,610 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (main) Routes startup (started:0) 2024-03-07 17:04:07,610 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (main) Apache Camel 4.0.2 (camel-1) started in 0ms (build:0ms init:0ms start:0ms) 2024-03-07 17:04:07,612 INFO [io.quarkus] (main) camel-k-integration 2.3.0-SNAPSHOT native (powered by Quarkus 3.2.9.Final) started in 0.052s. Listening on: http://0.0.0.0:8080 2024-03-07 17:04:07,612 INFO [io.quarkus] (main) Profile prod activated. 2024-03-07 17:04:07,612 INFO [io.quarkus] (main) Installed features: [camel-bean, camel-cloudevents, camel-core, camel-java-joor-dsl, camel-k-core, camel-k-runtime, camel-knative, camel-kubernetes, camel-timer, cdi, kubernetes-client, smallrye-context-propagation, vertx] to contain substring : Java Magicstring! ``` Fixing 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] fix(traits): use Comparable matches [camel-k]
squakez commented on PR #5230: URL: https://github.com/apache/camel-k/pull/5230#issuecomment-1985096440 Result of Quarkus native integrations: ``` --- PASS: TestNativeBinding (549.82s) --- PASS: TestNativeBinding/binding_with_native_build (535.03s) --- PASS: TestNativeIntegrations (1114.16s) --- PASS: TestNativeIntegrations/unsupported_integration_source_language (0.90s) --- PASS: TestNativeIntegrations/xml_native_support (632.23s) --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit (472.05s) --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit/yaml_native_should_not_rebuild (2.20s) ``` -- 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 (1c2986acc89 -> f0357f63e9b)
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 1c2986acc89 CAMEL-20538: modeline for jbang DEPS should support style that jbang has, and if not found a better error message. add f0357f63e9b camel-jbang - Upgrade to Q 3.8.2 No new revisions were added by this update. Summary of changes: .../org/apache/camel/dsl/jbang/core/commands/ExportBaseCommand.java | 2 +- .../camel/dsl/jbang/core/commands/catalog/CatalogBaseCommand.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
(camel) branch main updated (1c2986acc89 -> f0357f63e9b)
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 1c2986acc89 CAMEL-20538: modeline for jbang DEPS should support style that jbang has, and if not found a better error message. add f0357f63e9b camel-jbang - Upgrade to Q 3.8.2 No new revisions were added by this update. Summary of changes: .../org/apache/camel/dsl/jbang/core/commands/ExportBaseCommand.java | 2 +- .../camel/dsl/jbang/core/commands/catalog/CatalogBaseCommand.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
[I] Kubernetes Install Failing [camel-karavan]
skumaravelan opened a new issue, #1165: URL: https://github.com/apache/camel-karavan/issues/1165 Hi, When I did apply kubectl apply -k . Getting the below error. error: json: cannot unmarshal object into Go struct field Kustomization.patchesStrategicMerge of type patch.StrategicMerge Syntax looks good. What is the cause of this error, Please suggest a solution. 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(camel-kamelets) 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-kamelets.git The following commit(s) were added to refs/heads/main by this push: new ca7687c2 Updated CHANGELOG.md ca7687c2 is described below commit ca7687c2d1286879d79fe4cdfae81e49051b8da5 Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> AuthorDate: Fri Mar 8 03:06:36 2024 + Updated CHANGELOG.md --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index a0754df3..fd4f4ae7 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,7 @@ **Closed issues:** +- CDC Storage Source Kamelets, rename to -event-based instead of cdc [\#1930](https://github.com/apache/camel-kamelets/issues/1930) - Add Kafka Batch with Apicurio with keycloak support [\#1922](https://github.com/apache/camel-kamelets/issues/1922) - Kafka Apicurio Secured Sink Kamelet [\#1916](https://github.com/apache/camel-kamelets/issues/1916) - Kafka Apicurio Secured Source Kamelet [\#1915](https://github.com/apache/camel-kamelets/issues/1915)
Re: [I] [CI] - Quarkus Main Branch Build Failure [camel-quarkus]
github-actions[bot] commented on issue #2926: URL: https://github.com/apache/camel-quarkus/issues/2926#issuecomment-1984958655 The [quarkus-main](https://github.com/apache/camel-quarkus/tree/quarkus-main) branch build has failed: * Build ID: 8197711795-1415-122b51ed-165c-42d1-8f5e-0b0ee817ded1 * Camel Quarkus Commit: 3d5d3910764986af10373f94a609dca30762f44b * Quarkus Main Commit: ff9d9545e267d6481e5315d6dc9645c8162d65e5 * Link to build: https://github.com/apache/camel-quarkus/actions/runs/8197711795 -- 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(deps): bump github.com/prometheus/common from 0.48.0 to 0.50.0 [camel-k]
dependabot[bot] opened a new pull request, #5232: URL: https://github.com/apache/camel-k/pull/5232 Bumps [github.com/prometheus/common](https://github.com/prometheus/common) from 0.48.0 to 0.50.0. Release notes Sourced from https://github.com/prometheus/common/releases;>github.com/prometheus/common's releases. v0.50.0 What's Changed Synchronize common files from prometheus/prometheus by https://github.com/prombot;>@prombot in https://redirect.github.com/prometheus/common/pull/594;>prometheus/common#594 Bump github.com/stretchr/testify from 1.8.4 to 1.9.0 in /sigv4 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/593;>prometheus/common#593 Bump github.com/aws/aws-sdk-go from 1.50.27 to 1.50.29 in /sigv4 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/592;>prometheus/common#592 Bump github.com/aws/aws-sdk-go from 1.50.29 to 1.50.31 in /sigv4 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/595;>prometheus/common#595 Remove unused 'Host' member from HTTPClientConfig by https://github.com/bboreham;>@bboreham in https://redirect.github.com/prometheus/common/pull/597;>prometheus/common#597 Add OpenMetrics unit support by https://github.com/vesari;>@vesari in https://redirect.github.com/prometheus/common/pull/544;>prometheus/common#544 Remove deprecated version function by https://github.com/SuperQ;>@SuperQ in https://redirect.github.com/prometheus/common/pull/591;>prometheus/common#591 Synchronize common files from prometheus/prometheus by https://github.com/prombot;>@prombot in https://redirect.github.com/prometheus/common/pull/599;>prometheus/common#599 Bump golang.org/x/oauth2 from 0.17.0 to 0.18.0 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/600;>prometheus/common#600 Bump google.golang.org/protobuf from 1.32.0 to 1.33.0 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/601;>prometheus/common#601 Full Changelog: https://github.com/prometheus/common/compare/v0.49.0...v0.50.0;>https://github.com/prometheus/common/compare/v0.49.0...v0.50.0 v0.49.0 What's Changed Synchronize common files from prometheus/prometheus by https://github.com/prombot;>@prombot in https://redirect.github.com/prometheus/common/pull/574;>prometheus/common#574 Bump github.com/aws/aws-sdk-go from 1.49.13 to 1.50.8 in /sigv4 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/571;>prometheus/common#571 Synchronize common files from prometheus/prometheus by https://github.com/prombot;>@prombot in https://redirect.github.com/prometheus/common/pull/581;>prometheus/common#581 Update Go by https://github.com/SuperQ;>@SuperQ in https://redirect.github.com/prometheus/common/pull/588;>prometheus/common#588 Deprecate version.NewCollector by https://github.com/SuperQ;>@SuperQ in https://redirect.github.com/prometheus/common/pull/579;>prometheus/common#579 Bump github.com/aws/aws-sdk-go from 1.50.8 to 1.50.27 in /sigv4 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/587;>prometheus/common#587 Avoid off-spec openmetrics exposition when exemplars have empty labels by https://github.com/orls;>@orls in https://redirect.github.com/prometheus/common/pull/569;>prometheus/common#569 Bump golang.org/x/oauth2 from 0.16.0 to 0.17.0 by https://github.com/dependabot;>@dependabot in https://redirect.github.com/prometheus/common/pull/585;>prometheus/common#585 Write created lines when negotiating OpenMetrics by https://github.com/ArthurSens;>@ArthurSens in https://redirect.github.com/prometheus/common/pull/504;>prometheus/common#504 Upgrade client_model to v.0.6.0 by https://github.com/vesari;>@vesari in https://redirect.github.com/prometheus/common/pull/589;>prometheus/common#589 http_config: Add host by https://github.com/jkroepke;>@jkroepke in https://redirect.github.com/prometheus/common/pull/549;>prometheus/common#549 LabelSet: Fix alphabetical sorting for prometheus LabelSet by https://github.com/wasim-nihal;>@wasim-nihal in https://redirect.github.com/prometheus/common/pull/575;>prometheus/common#575 labelset: optimise String() function by https://github.com/bboreham;>@bboreham in https://redirect.github.com/prometheus/common/pull/590;>prometheus/common#590 New Contributors https://github.com/orls;>@orls made their first contribution in https://redirect.github.com/prometheus/common/pull/569;>prometheus/common#569 https://github.com/vesari;>@vesari made their first contribution in https://redirect.github.com/prometheus/common/pull/589;>prometheus/common#589 Full Changelog:
(camel-k) branch dependabot/go_modules/github.com/prometheus/common-0.50.0 created (now ddda4d36f)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/github.com/prometheus/common-0.50.0 in repository https://gitbox.apache.org/repos/asf/camel-k.git at ddda4d36f chore(deps): bump github.com/prometheus/common from 0.48.0 to 0.50.0 No new revisions were added by this update.
Re: [I] [CI] - Camel Main Branch Build Failure [camel-quarkus]
github-actions[bot] commented on issue #2927: URL: https://github.com/apache/camel-quarkus/issues/2927#issuecomment-1984858284 The [camel-main](https://github.com/apache/camel-quarkus/tree/camel-main) branch build has failed: * Build ID: 8196596437-1311-76d11251-4f94-4131-85fa-027130a72007 * Camel Quarkus Commit: a1b11bdb389a8099fb5b795fa0122c4c0cd2583f * Camel Main Commit: ff9d9545e267d6481e5315d6dc9645c8162d65e5 * Link to build: https://github.com/apache/camel-quarkus/actions/runs/8196596437 -- 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] [Github Actions] Periodic Sync Camel Spring Boot (Camel 4) [camel-spring-boot]
github-actions[bot] closed pull request #1101: [Github Actions] Periodic Sync Camel Spring Boot (Camel 4) URL: https://github.com/apache/camel-spring-boot/pull/1101 -- 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] Provide full description for trait in CRD [camel-k]
github-actions[bot] commented on issue #4727: URL: https://github.com/apache/camel-k/issues/4727#issuecomment-1984825686 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
(camel-k) branch main updated (727e295ab -> 26c400f42)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git from 727e295ab chore(lint): Fix lint errors add 26c400f42 chore: nightly automatic updates No new revisions were added by this update. Summary of changes: CHANGELOG.md | 3 +++ coverage.out | 65 ++-- 2 files changed, 35 insertions(+), 33 deletions(-)
Re: [PR] Install operator/base image according to the target architecture [camel-k]
lburgazzoli commented on code in PR #5200:
URL: https://github.com/apache/camel-k/pull/5200#discussion_r1516913748
##
pkg/util/kubernetes/node.go:
##
@@ -0,0 +1,52 @@
+/*
+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 kubernetes
+
+import (
+ "context"
+ "github.com/apache/camel-k/v2/pkg/client"
+ "github.com/apache/camel-k/v2/pkg/util/sets"
+ "github.com/apache/camel-k/v2/pkg/util/slice"
+ corev1 "k8s.io/api/core/v1"
+ metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+)
+
+// NodeListLookup lookup the list of nodes in the cluster
+func NodeListLookup(ctx context.Context, c client.Client) ([]corev1.Node,
error) {
+ nodes, err := c.CoreV1().Nodes().List(ctx, metav1.ListOptions{})
+ if err != nil {
+ return nil, err
+ }
+ return nodes.Items, err
Review Comment:
nitpick: usually it is better to explicitly return `nil` if there are no
error and the reason is that, [a nil error value not equal to nil](
https://go.dev/doc/faq#nil_error). In this snipped, it won't make a difference
as `err` is never reused, but to avoid subtle bugs, I would recommend
explicitly return `nil`
--
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
[I] Update buil/Dokerfile to use go 1.21 [camel-k]
lburgazzoli opened a new issue, #5231: URL: https://github.com/apache/camel-k/issues/5231 ### What happened? The `buil/Dokerfile` still use go 1.21 https://github.com/apache/camel-k/blob/727e295abab3f20355b8bfac2f4c090326d71587/build/Dockerfile#L64 Ref: - https://github.com/apache/camel-k/issues/4795 - https://github.com/apache/camel-k/pull/5221 ### Steps to reproduce _No response_ ### Relevant log output _No response_ ### Camel K version _No response_ -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Install operator/base image according to the target architecture [camel-k]
lburgazzoli commented on code in PR #5200:
URL: https://github.com/apache/camel-k/pull/5200#discussion_r1516898792
##
pkg/util/kubernetes/node_test.go:
##
@@ -0,0 +1,38 @@
+/*
+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 kubernetes
+
+import (
+ "context"
+ "fmt"
+ "github.com/apache/camel-k/v2/pkg/client"
+ "github.com/stretchr/testify/assert"
+ "testing"
+)
+
+func TestNodeArchLookup(t *testing.T) {
Review Comment:
wonder if this test is better suited to be on an e2e or have a conditional
execution as you may not be connected to a k8s cluster and/or you may not have
enough privileges.
--
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/base image according to the target architecture [camel-k]
lburgazzoli commented on code in PR #5200:
URL: https://github.com/apache/camel-k/pull/5200#discussion_r1516892129
##
pkg/util/slice/slice.go:
##
@@ -0,0 +1,47 @@
+/*
+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 slice
+
+// SliceFilter retain elements for which the given predicate returns true
+func SliceFilter[T any](elements []T, predicate func(T) bool) []T {
Review Comment:
nitpick: usually, it is better to avoid repeating the package name in a
function, i.e. `slice.Filter` instead of `slice.SliceFilter` as the package
already defines its purpose (or at least it should)
--
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(traits): use Comparable matches [camel-k]
lburgazzoli commented on code in PR #5230:
URL: https://github.com/apache/camel-k/pull/5230#discussion_r1516882356
##
pkg/trait/builder.go:
##
@@ -56,9 +57,18 @@ func (t *builderTrait) InfluencesKit() bool {
return true
}
-// InfluencesBuild overrides base class method.
-func (t *builderTrait) InfluencesBuild(this, prev map[string]interface{}) bool
{
- return true
+func (t *builderTrait) Matches(trait Trait) bool {
+ otherTrait, ok := trait.(*builderTrait)
+ if !ok {
+ return false
+ }
+ if t.BaseImage != otherTrait.BaseImage || len(t.Properties) !=
len(otherTrait.Properties) {
+ return false
+ }
+ // More sofisticated check if len is the same. Sort and compare via
slices equal func.
+ slices.Sort(t.Properties)
+ slices.Sort(otherTrait.Properties)
Review Comment:
nitpick: this method has a side effect as it alters the order of both the
current and the other trait properties.
--
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] chore(e2e): Improve E2E tests [camel-k]
christophd commented on PR #5188: URL: https://github.com/apache/camel-k/pull/5188#issuecomment-1984543351 @squakez I tried to run the native tests locally which ends up in timeouts while building the integrations. I tested this also on the main branch with the same result. So I guess my local setup is not ready to run these native E2E tests. Do you have any working setup or information how to properly run the tests locally? -- 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-k-runtime) branch dependabot/maven/org.jolokia-jolokia-agent-jvm-2.0.2 created (now a46c22d1)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.jolokia-jolokia-agent-jvm-2.0.2 in repository https://gitbox.apache.org/repos/asf/camel-k-runtime.git at a46c22d1 build(deps): bump org.jolokia:jolokia-agent-jvm from 2.0.1 to 2.0.2 No new revisions were added by this update.
[PR] build(deps): bump org.jolokia:jolokia-agent-jvm from 2.0.1 to 2.0.2 [camel-k-runtime]
dependabot[bot] opened a new pull request, #1181: URL: https://github.com/apache/camel-k-runtime/pull/1181 Bumps org.jolokia:jolokia-agent-jvm from 2.0.1 to 2.0.2. [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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-k-runtime) branch dependabot/maven/quarkus-platform-version-3.8.2 created (now 525f2f4a)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/quarkus-platform-version-3.8.2 in repository https://gitbox.apache.org/repos/asf/camel-k-runtime.git at 525f2f4a build(deps): bump quarkus-platform-version from 3.8.1 to 3.8.2 No new revisions were added by this update.
[PR] build(deps): bump quarkus-platform-version from 3.8.1 to 3.8.2 [camel-k-runtime]
dependabot[bot] opened a new pull request, #1180: URL: https://github.com/apache/camel-k-runtime/pull/1180 Bumps `quarkus-platform-version` from 3.8.1 to 3.8.2. Updates `io.quarkus.platform:quarkus-camel-bom` from 3.8.1 to 3.8.2 Commits https://github.com/quarkusio/quarkus-platform/commit/7b09b88e2a8162b11d5cd20c5b74e0a78e361445;>7b09b88 [maven-release-plugin] prepare release 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/dc597f86eb9e7815a98560af62e748d9f6e2637e;>dc597f8 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1134;>#1134 from gsmet/quarkus-3.8.2 https://github.com/quarkusio/quarkus-platform/commit/86f60cbf04baca9d1dac7be3ff7c6468701a50dc;>86f60cb Upgrade to Quarkus 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/8125659da6a094026f3b1f8611f71f29501ee625;>8125659 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1133;>#1133 from gemmellr/quarkus-qpid-jms-2.6.0-backport-3.8 https://github.com/quarkusio/quarkus-platform/commit/fde56ae324d2b6f4857e32d795a9c363ed9e73b5;>fde56ae Update to Quarkus Qpid JMS 2.6.0, uses Qpid JMS 2.5.0 against Quarkus 3.8.1 https://github.com/quarkusio/quarkus-platform/commit/e8482e0b13bb57031ae2b8176cba09fa2d6369e4;>e8482e0 [maven-release-plugin] prepare for next development iteration See full diff in https://github.com/quarkusio/quarkus-platform/compare/3.8.1...3.8.2;>compare view Updates `io.quarkus.platform:quarkus-bom` from 3.8.1 to 3.8.2 Commits https://github.com/quarkusio/quarkus-platform/commit/7b09b88e2a8162b11d5cd20c5b74e0a78e361445;>7b09b88 [maven-release-plugin] prepare release 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/dc597f86eb9e7815a98560af62e748d9f6e2637e;>dc597f8 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1134;>#1134 from gsmet/quarkus-3.8.2 https://github.com/quarkusio/quarkus-platform/commit/86f60cbf04baca9d1dac7be3ff7c6468701a50dc;>86f60cb Upgrade to Quarkus 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/8125659da6a094026f3b1f8611f71f29501ee625;>8125659 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1133;>#1133 from gemmellr/quarkus-qpid-jms-2.6.0-backport-3.8 https://github.com/quarkusio/quarkus-platform/commit/fde56ae324d2b6f4857e32d795a9c363ed9e73b5;>fde56ae Update to Quarkus Qpid JMS 2.6.0, uses Qpid JMS 2.5.0 against Quarkus 3.8.1 https://github.com/quarkusio/quarkus-platform/commit/e8482e0b13bb57031ae2b8176cba09fa2d6369e4;>e8482e0 [maven-release-plugin] prepare for next development iteration See full diff in https://github.com/quarkusio/quarkus-platform/compare/3.8.1...3.8.2;>compare view Updates `io.quarkus.platform:quarkus-maven-plugin` from 3.8.1 to 3.8.2 Commits https://github.com/quarkusio/quarkus-platform/commit/7b09b88e2a8162b11d5cd20c5b74e0a78e361445;>7b09b88 [maven-release-plugin] prepare release 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/dc597f86eb9e7815a98560af62e748d9f6e2637e;>dc597f8 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1134;>#1134 from gsmet/quarkus-3.8.2 https://github.com/quarkusio/quarkus-platform/commit/86f60cbf04baca9d1dac7be3ff7c6468701a50dc;>86f60cb Upgrade to Quarkus 3.8.2 https://github.com/quarkusio/quarkus-platform/commit/8125659da6a094026f3b1f8611f71f29501ee625;>8125659 Merge pull request https://redirect.github.com/quarkusio/quarkus-platform/issues/1133;>#1133 from gemmellr/quarkus-qpid-jms-2.6.0-backport-3.8 https://github.com/quarkusio/quarkus-platform/commit/fde56ae324d2b6f4857e32d795a9c363ed9e73b5;>fde56ae Update to Quarkus Qpid JMS 2.6.0, uses Qpid JMS 2.5.0 against Quarkus 3.8.1 https://github.com/quarkusio/quarkus-platform/commit/e8482e0b13bb57031ae2b8176cba09fa2d6369e4;>e8482e0 [maven-release-plugin] prepare for next development iteration See full diff in https://github.com/quarkusio/quarkus-platform/compare/3.8.1...3.8.2;>compare view Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed -
(camel) branch regen_bot updated (6ebd62c7f02 -> 1c2986acc89)
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 6ebd62c7f02 Typo add 1c2986acc89 CAMEL-20538: modeline for jbang DEPS should support style that jbang has, and if not found a better error message. No new revisions were added by this update. Summary of changes: .../camel/dsl/modeline/DefaultModelineParser.java | 17 + 1 file changed, 17 insertions(+)
(camel) branch camel-4.4.x updated: CAMEL-20538: modeline for jbang DEPS should support style that jbang has, and if not found a better error message.
This is an automated email from the ASF dual-hosted git repository.
davsclaus pushed a commit to branch camel-4.4.x
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/camel-4.4.x by this push:
new b296b289a64 CAMEL-20538: modeline for jbang DEPS should support style
that jbang has, and if not found a better error message.
b296b289a64 is described below
commit b296b289a6461256581b237d3499d9d8bb67710e
Author: Claus Ibsen
AuthorDate: Thu Mar 7 19:14:24 2024 +0100
CAMEL-20538: modeline for jbang DEPS should support style that jbang has,
and if not found a better error message.
---
.../camel/dsl/modeline/DefaultModelineParser.java | 17 +
1 file changed, 17 insertions(+)
diff --git
a/dsl/camel-dsl-modeline/src/main/java/org/apache/camel/dsl/modeline/DefaultModelineParser.java
b/dsl/camel-dsl-modeline/src/main/java/org/apache/camel/dsl/modeline/DefaultModelineParser.java
index 7d2569d93b4..fba0c68f6ca 100644
---
a/dsl/camel-dsl-modeline/src/main/java/org/apache/camel/dsl/modeline/DefaultModelineParser.java
+++
b/dsl/camel-dsl-modeline/src/main/java/org/apache/camel/dsl/modeline/DefaultModelineParser.java
@@ -106,6 +106,23 @@ public class DefaultModelineParser implements
ModelineParser {
// skip @pom
continue;
}
+// in case DEPS uses jbang ${ } style that refer to JVM system
properties
+if (part.contains("${") && part.contains("}")) {
+String target = StringHelper.between(part, "${", "}");
+String value = StringHelper.before(target, ":", target);
+if (target.contains(":")) {
+String def = StringHelper.after(target, ":");
+value = System.getProperty(value, def);
+} else {
+String found = System.getProperty(value);
+if (found == null) {
+throw new IllegalArgumentException(
+"Cannot find JVM system property: " +
value + " for dependency: " + part);
+}
+value = found;
+}
+part = part.replace("${" + target + "}", value);
+}
CamelContextCustomizer customizer = dep.parseTrait(resource,
part);
if (customizer != null) {
answer.add(customizer);
(camel) branch main updated (6ebd62c7f02 -> 1c2986acc89)
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 6ebd62c7f02 Typo add 1c2986acc89 CAMEL-20538: modeline for jbang DEPS should support style that jbang has, and if not found a better error message. No new revisions were added by this update. Summary of changes: .../camel/dsl/modeline/DefaultModelineParser.java | 17 + 1 file changed, 17 insertions(+)
(camel) branch regen_bot updated (4f701c5a4d9 -> 6ebd62c7f02)
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 4f701c5a4d9 CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock runtime agent for knowledge base support (#13410) add 6ebd62c7f02 Typo No new revisions were added by this update. Summary of changes: .../java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java | 5 + 1 file changed, 1 insertion(+), 4 deletions(-)
(camel) branch main updated: Typo
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 6ebd62c7f02 Typo
6ebd62c7f02 is described below
commit 6ebd62c7f02d42c0c7b3894cef0fb946d198ec55
Author: Claus Ibsen
AuthorDate: Thu Mar 7 18:40:06 2024 +0100
Typo
---
.../java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git
a/tooling/camel-tooling-maven/src/main/java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java
b/tooling/camel-tooling-maven/src/main/java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java
index 04af434b7a2..3851955f771 100644
---
a/tooling/camel-tooling-maven/src/main/java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java
+++
b/tooling/camel-tooling-maven/src/main/java/org/apache/camel/tooling/maven/MavenDownloaderImpl.java
@@ -262,11 +262,8 @@ public class MavenDownloaderImpl extends ServiceSupport
implements MavenDownload
private boolean fresh;
private boolean offline;
private RemoteArtifactDownloadListener remoteArtifactDownloadListener;
-
private boolean apacheSnapshotsIncluded;
-
private AtomicInteger customRepositoryCounter = new AtomicInteger(1);
-
private Settings settings;
public MavenDownloaderImpl() {
@@ -322,7 +319,7 @@ public class MavenDownloaderImpl extends ServiceSupport
implements MavenDownload
defaultPolicy = fresh ? POLICY_FRESH : POLICY_DEFAULT;
// process repositories - both from settings.xml and from --repos
option. All are subject to
-// mirrorring and proxying (handled by
org.eclipse.aether.RepositorySystem#newResolutionRepositories())
+// mirroring and proxying (handled by
org.eclipse.aether.RepositorySystem#newResolutionRepositories())
List originalRepositories =
configureDefaultRepositories(settings);
remoteRepositories.addAll(repositorySystem.newResolutionRepositories(repositorySystemSession,
Re: [PR] fix(traits): use Comparable matches [camel-k]
github-actions[bot] commented on PR #5230: URL: https://github.com/apache/camel-k/pull/5230#issuecomment-1983979304 :heavy_check_mark: Unit test coverage report - coverage increased from 36.8% to 37% (**+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
[PR] fix(traits): use Comparable matches [camel-k]
squakez opened a new pull request, #5230: URL: https://github.com/apache/camel-k/pull/5230 Reverting #4512 which introduced a function diverging the match from the original design This PR aims to fix a diverging design I introduced some times ago. With this one we move back to the original design making correct usage of `Comparable` interface in trait which is used by the operator to distinguish when to reuse or rebuild an Integration based only on certain IntegrationKit traits changes (ie, runtime version, builder properties). **Release Note** ```release-note fix(traits): use Comparable matches ``` -- 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-spring-boot) branch main updated: Camel-AWS-Bedrock-Starter: Regen starter
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-spring-boot.git
The following commit(s) were added to refs/heads/main by this push:
new a2435cc417e Camel-AWS-Bedrock-Starter: Regen starter
a2435cc417e is described below
commit a2435cc417e2858863a33284e0fc996d690069e8
Author: Andrea Cosentino
AuthorDate: Thu Mar 7 17:23:50 2024 +0100
Camel-AWS-Bedrock-Starter: Regen starter
Signed-off-by: Andrea Cosentino
---
.../camel/springboot/catalog/components.properties | 1 +
.../components/aws-bedrock-agent-runtime.json | 78 +++
.../springboot/catalog/components/aws-bedrock.json | 84
.../src/main/docs/aws-bedrock.json | 237 ++---
...ockAgentRuntimeComponentAutoConfiguration.java} | 22 +-
...BedrockAgentRuntimeComponentConfiguration.java} | 75 ---
.../BedrockAgentRuntimeComponentConverter.java}| 14 +-
.../BedrockComponentAutoConfiguration.java | 4 +-
.../springboot/BedrockComponentConfiguration.java | 11 +-
.../springboot/BedrockComponentConverter.java | 6 +-
...rk.boot.autoconfigure.AutoConfiguration.imports | 6 +-
.../itest/springboot/CamelAwsBedrockTest.java | 4 +-
12 files changed, 409 insertions(+), 133 deletions(-)
diff --git
a/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components.properties
b/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components.properties
index ecc7f827abd..55aac288128 100644
---
a/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components.properties
+++
b/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components.properties
@@ -9,6 +9,7 @@ atmosphere-websocket
atom
avro
aws-bedrock
+aws-bedrock-agent-runtime
aws-cloudtrail
aws-config
aws-secrets-manager
diff --git
a/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/aws-bedrock-agent-runtime.json
b/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/aws-bedrock-agent-runtime.json
new file mode 100644
index 000..f82111a4e00
--- /dev/null
+++
b/catalog/camel-catalog-provider-springboot/src/main/resources/org/apache/camel/springboot/catalog/components/aws-bedrock-agent-runtime.json
@@ -0,0 +1,78 @@
+{
+ "component": {
+"kind": "component",
+"name": "aws-bedrock-agent-runtime",
+"title": "AWS Bedrock Agent Runtime",
+"description": "Invoke Model of AWS Bedrock Agent Runtime service.",
+"deprecated": false,
+"firstVersion": "4.5.0",
+"label": "ai,cloud",
+"javaType":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeComponent",
+"supportLevel": "Preview",
+"groupId": "org.apache.camel.springboot",
+"artifactId": "camel-aws-bedrock-starter",
+"version": "4.5.0-SNAPSHOT",
+"scheme": "aws-bedrock-agent-runtime",
+"extendsScheme": "",
+"syntax": "aws-bedrock-agent-runtime:label",
+"async": false,
+"api": false,
+"consumerOnly": false,
+"producerOnly": true,
+"lenientProperties": false,
+"remote": true
+ },
+ "componentProperties": {
+"configuration": { "index": 0, "kind": "property", "displayName":
"Configuration", "group": "producer", "label": "", "required": false, "type":
"object", "javaType":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"deprecated": false, "autowired": false, "secret": false, "description":
"Component configuration" },
+"knowledgeBaseId": { "index": 1, "kind": "property", "displayName":
"Knowledge Base Id", "group": "producer", "label": "", "required": true,
"type": "string", "javaType": "java.lang.String", "deprecated": false,
"deprecationNote": "", "autowired": false, "secret": false,
"configurationClass":
"org.apache.camel.component.aws2.bedrock.agentruntime.BedrockAgentRuntimeConfiguration",
"configurationField": "configuration", "description": "Define the Knowledge
Base Id we are going to use" },
+"lazyStartProducer": { "index": 2, "kind": "property", "displayName":
"Lazy Start Producer", "group": "producer", "label": "producer", "required":
false, "type": "boolean", "javaType": "boolean", "deprecated": false,
"autowired": false, "secret": false, "defaultValue": false, "description":
"Whether the producer should be started lazy (on the first message). By
starting lazy you can use this to allow CamelContext and routes to startup in
situations where a producer may otherwise fail [...]
+"modelId": { "index": 3, "kind": "property", "displayName": "Model Id",
"group": "producer", "label": "", "required": true, "type": "string",
"javaType": "java.lang.String",
Re: [PR] fix #4948 part of #3397 [camel-k]
squakez commented on PR #5119: URL: https://github.com/apache/camel-k/pull/5119#issuecomment-1983848486 I've enabled Quarkus Native tests. However it's very likely they fail due to some infrastructure problem that we have not yet tackled. In such circumstances, please, run the Quarkus Native test manually in your local environment and report result here. -- 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] chore(e2e): Improve E2E tests [camel-k]
squakez commented on PR #5188: URL: https://github.com/apache/camel-k/pull/5188#issuecomment-1983845623 I've enabled Quarkus Native tests. However it's very likely they fail due to some infrastructure problem that we have not yet tackled. In such circumstances, please, run the Quarkus Native test manually in your local environment and report result here. -- 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/base image according to the target architecture [camel-k]
tdiesler commented on PR #5200: URL: https://github.com/apache/camel-k/pull/5200#issuecomment-1983808541 The above would only work if your buildx extensions supports both (linux/amd64, linux/arm64) platforms. Perhaps there needs to be more logic in the Makefile to account for that. -- 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 (5a0ff041f01 -> 4f701c5a4d9)
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 5a0ff041f01 CAMEL-20533: camel-rest - Binding mode json or xml should
automatically set consumes/produces on the rest endpoint according to the
binding mode. (#13407)
add 4f701c5a4d9 CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the
bedrock runtime agent for knowledge base support (#13410)
No new revisions were added by this update.
Summary of changes:
.../org/apache/camel/catalog/components.properties | 1 +
.../components/aws-bedrock-agent-runtime.json | 78
.../camel/catalog/components/aws-bedrock.json | 84
components/camel-aws/camel-aws-bedrock/pom.xml | 5 +
.../BedrockAgentRuntimeComponentConfigurer.java} | 40 ++--
.../BedrockAgentRuntimeEndpointConfigurer.java}| 32 +--
.../BedrockAgentRuntimeEndpointUriFactory.java}| 11 +-
.../{ => runtime}/BedrockComponentConfigurer.java | 14 +-
.../{ => runtime}/BedrockEndpointConfigurer.java | 6 +-
.../{ => runtime}/BedrockEndpointUriFactory.java | 2 +-
.../agentruntime/aws-bedrock-agent-runtime.json| 78
.../aws2/bedrock/{ => runtime}/aws-bedrock.json| 84
.../services/org/apache/camel/component.properties | 2 +-
.../org/apache/camel/component/aws-bedrock | 2 +-
.../camel/component/aws-bedrock-agent-runtime | 2 +
.../configurer/aws-bedrock-agent-runtime-component | 2 +
.../configurer/aws-bedrock-agent-runtime-endpoint | 2 +
.../apache/camel/configurer/aws-bedrock-component | 2 +-
.../apache/camel/configurer/aws-bedrock-endpoint | 2 +-
.../urifactory/aws-bedrock-agent-runtime-endpoint | 2 +
.../apache/camel/urifactory/aws-bedrock-endpoint | 2 +-
.../docs/aws-bedrock-agent-runtime-component.adoc | 102 ++
.../component/aws2/bedrock/BedrockModels.java | 4 +-
.../BedrockAgentRuntimeComponent.java} | 26 +--
.../BedrockAgentRuntimeConfiguration.java} | 51 +++--
.../BedrockAgentRuntimeConstants.java} | 10 +-
.../agentruntime/BedrockAgentRuntimeEndpoint.java | 86
.../BedrockAgentRuntimeOperations.java}| 10 +-
.../agentruntime/BedrockAgentRuntimeProducer.java | 134 +
.../client/BedrockAgentRuntimeClientFactory.java | 51 +
.../client/BedrockAgentRuntimeInternalClient.java} | 12 +-
...BedrockAgentRuntimeClientIAMOptimizedImpl.java} | 30 +--
...AgentRuntimeClientIAMProfileOptimizedImpl.java} | 34 ++--
...BedrockAgentRuntimeClientSessionTokenImpl.java} | 34 ++--
.../BedrockAgentRuntimeClientStandardImpl.java}| 34 ++--
.../bedrock/{ => runtime}/BedrockComponent.java| 2 +-
.../{ => runtime}/BedrockConfiguration.java| 2 +-
.../bedrock/{ => runtime}/BedrockConstants.java| 2 +-
.../bedrock/{ => runtime}/BedrockEndpoint.java | 4 +-
.../bedrock/{ => runtime}/BedrockOperations.java | 2 +-
.../bedrock/{ => runtime}/BedrockProducer.java | 2 +-
.../{ => runtime}/client/BedrockClientFactory.java | 12 +-
.../client/BedrockRuntimeInternalClient.java | 2 +-
.../impl/BedrockRuntimeClientIAMOptimizedImpl.java | 6 +-
...edrockRuntimeClientIAMProfileOptimizedImpl.java | 6 +-
.../impl/BedrockRuntimeClientSessionTokenImpl.java | 6 +-
.../impl/BedrockRuntimeClientStandardImpl.java | 6 +-
.../BedrockAgentRuntimeClientFactoryTest.java | 74 +++
.../integration/BedrockAgentRuntimeProducerIT.java | 72 +++
.../BedrockRuntimeClientFactoryTest.java | 14 +-
.../integration/BedrockProducerIT.java | 4 +-
.../org/apache/camel/main/components.properties| 1 +
.../examples/json/aws-bedrock-agent-runtime.json | 1 +
.../modules/ROOT/examples/json/aws-bedrock.json| 2 +-
docs/components/modules/ROOT/nav.adoc | 1 +
.../pages/aws-bedrock-agent-runtime-component.adoc | 1 +
.../component/ComponentsBuilderFactory.java| 13 ++
...edrockAgentRuntimeComponentBuilderFactory.java} | 183 +
.../dsl/AwsBedrockComponentBuilderFactory.java | 20 +-
.../src/generated/resources/metadata.json | 25 ++-
.../builder/endpoint/EndpointBuilderFactory.java | 1 +
.../camel/builder/endpoint/EndpointBuilders.java | 1 +
.../builder/endpoint/StaticEndpointBuilders.java | 43
...BedrockAgentRuntimeEndpointBuilderFactory.java} | 216 +++--
.../dsl/BedrockEndpointBuilderFactory.java | 6 +-
.../camel-component-known-dependencies.properties | 3 +-
.../components/AwsBedrockAgentRuntimeUriDsl.kt | 159 +++
67 files changed, 1479 insertions(+), 484 deletions(-)
create mode 100644
catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws-bedrock-agent-runtime.json
copy
Re: [PR] Install operator/base image according to the target architecture [camel-k]
tdiesler commented on PR #5200: URL: https://github.com/apache/camel-k/pull/5200#issuecomment-1983798678 Here is my next attempt for this ... You can run ``` NOTEST=1 make clean images ... ### Skipping unit test... ### Building kamel CLI for amd64 architecture... CGO_ENABLED=0 GOARCH=amd64 go build -ldflags "-X github.com/apache/camel-k/v2/pkg/util/defaults.GitCommit=853be0a46cf0d3532e33e48c2644884a7fdcaac9" -trimpath -o build/_output/bin/kamel-amd64 ./cmd/kamel/*.go ### Building kamel CLI for arm64 architecture... CGO_ENABLED=0 GOARCH=arm64 go build -ldflags "-X github.com/apache/camel-k/v2/pkg/util/defaults.GitCommit=853be0a46cf0d3532e33e48c2644884a7fdcaac9" -trimpath -o build/_output/bin/kamel-arm64 ./cmd/kamel/*.go ... ### Building Camel K operator arch amd64 container image... mkdir -p build/_maven_output docker buildx build --target base --platform=linux/amd64 --build-arg IMAGE_ARCH=amd64 -t docker.io/apache/camel-k:2.3.0-SNAPSHOT-amd64 -f build/Dockerfile --load . ### Building Camel K operator arch arm64 container image... mkdir -p build/_maven_output docker buildx build --target base --platform=linux/arm64 --build-arg IMAGE_ARCH=arm64 -t docker.io/apache/camel-k:2.3.0-SNAPSHOT-arm64 -f build/Dockerfile --load . ``` You should then find these images in your local registry ``` $ docker images REPOSITORYTAG IMAGE ID CREATED SIZE apache/camel-k2.3.0-SNAPSHOT-arm64 aea419a206ab 27 seconds ago 549MB apache/camel-k2.3.0-SNAPSHOT-amd64 509a6b88f5f5 About a minute ago 556MB ``` Which can them be combined to a multiarch image like this ``` $ make images-push docker push docker.io/apache/camel-k:2.3.0-SNAPSHOT-amd64 docker push docker.io/apache/camel-k:2.3.0-SNAPSHOT-arm64 ... docker manifest create docker.io/nessusio/camel-k:2.3.0-SNAPSHOT --amend docker.io/nessusio/camel-k:2.3.0-SNAPSHOT-amd64 --amend docker.io/nessusio/camel-k:2.3.0-SNAPSHOT-arm64 Created manifest list docker.io/nessusio/camel-k:2.3.0-SNAPSHOT docker manifest push docker.io/nessusio/camel-k:2.3.0-SNAPSHOT sha256:13f2e249b41046ddfbfbe70906736ac899dd252bc685ae459f689aa26b2ba8d8 ``` https://hub.docker.com/repository/docker/nessusio/camel-k/tags?page=1=last_updated -- 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 CAMEL-20535 deleted (was f42236c01a3)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch CAMEL-20535 in repository https://gitbox.apache.org/repos/asf/camel.git was f42236c01a3 CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock runtime agent for knowledge base support The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
Re: [PR] CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock ru… [camel]
oscerd merged PR #13410: URL: https://github.com/apache/camel/pull/13410 -- 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 quarkus-main updated (fac497e7c2 -> 3d5d391076)
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a change to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git discard fac497e7c2 Disable Infinispan tests due to #5844 discard 1715274739 Upgrade Quarkus to 3.9.0.CR1 add 4e30f6a184 Bump quarkiverse-mybatis.version from 2.2.1 to 2.2.2 (#5845) add fc01220a03 Bump org.amqphub.quarkus:quarkus-qpid-jms-bom from 2.5.0 to 2.6.0 (#5846) add ff9d9545e2 Upgrade Quarkus to 3.8.2 new 4fbc6774db Upgrade Quarkus to 3.9.0.CR1 new 3d5d391076 Disable Infinispan tests due to #5844 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 (fac497e7c2) \ N -- N -- N refs/heads/quarkus-main (3d5d391076) 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: docs/antora.yml| 2 +- pom.xml| 6 +- poms/bom/src/main/generated/flattened-full-pom.xml | 14401 ++- .../src/main/generated/flattened-reduced-pom.xml | 8 +- .../generated/flattened-reduced-verbose-pom.xml|22 +- 5 files changed, 7405 insertions(+), 7034 deletions(-)
(camel-quarkus) 02/02: Disable Infinispan tests due to #5844
This is an automated email from the ASF dual-hosted git repository.
jamesnetherton pushed a commit to branch quarkus-main
in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git
commit 3d5d3910764986af10373f94a609dca30762f44b
Author: James Netherton
AuthorDate: Wed Mar 6 16:06:31 2024 +
Disable Infinispan tests due to #5844
---
integration-tests/infinispan-quarkus-client/pom.xml | 2 ++
integration-tests/infinispan/pom.xml | 2 ++
.../apache/camel/quarkus/component/infinispan/InfinispanIT.java | 2 ++
.../camel/quarkus/component/infinispan/InfinispanTest.java | 1 +
poms/bom/pom.xml | 9 +
poms/bom/src/main/generated/flattened-full-pom.xml | 8
poms/bom/src/main/generated/flattened-reduced-pom.xml| 8
poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml| 8
8 files changed, 40 insertions(+)
diff --git a/integration-tests/infinispan-quarkus-client/pom.xml
b/integration-tests/infinispan-quarkus-client/pom.xml
index a87b90409d..d77f674203 100644
--- a/integration-tests/infinispan-quarkus-client/pom.xml
+++ b/integration-tests/infinispan-quarkus-client/pom.xml
@@ -70,6 +70,7 @@
+
virtualDependencies
diff --git a/integration-tests/infinispan/pom.xml
b/integration-tests/infinispan/pom.xml
index 90f2f30fc9..e5514af2dd 100644
--- a/integration-tests/infinispan/pom.xml
+++ b/integration-tests/infinispan/pom.xml
@@ -82,6 +82,7 @@
+
virtualDependencies
diff --git
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
index 25a4208dc0..f63f2c078d 100644
---
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
+++
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanIT.java
@@ -17,7 +17,9 @@
package org.apache.camel.quarkus.component.infinispan;
import io.quarkus.test.junit.QuarkusIntegrationTest;
+import org.junit.jupiter.api.Disabled;
+@Disabled("https://github.com/apache/camel-quarkus/issues/5844;)
@QuarkusIntegrationTest
public class InfinispanIT extends InfinispanTest {
}
diff --git
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
index 637fa714a2..b469e343c3 100644
---
a/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
+++
b/integration-tests/infinispan/src/test/java/org/apache/camel/quarkus/component/infinispan/InfinispanTest.java
@@ -26,6 +26,7 @@ import org.junit.jupiter.api.Test;
import static org.hamcrest.Matchers.is;
+@Disabled("https://github.com/apache/camel-quarkus/issues/5844;)
@QuarkusTest
@QuarkusTestResource(InfinispanServerTestResource.class)
public class InfinispanTest extends InfinispanCommonTest {
diff --git a/poms/bom/pom.xml b/poms/bom/pom.xml
index 2b6dd5ca23..30a1f81f01 100644
--- a/poms/bom/pom.xml
+++ b/poms/bom/pom.xml
@@ -1426,6 +1426,15 @@
camel-infinispan
${camel.version}
+
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+org.infinispan
+
infinispan-client-hotrod-jakarta
+
org.slf4j
jcl-over-slf4j
diff --git a/poms/bom/src/main/generated/flattened-full-pom.xml
b/poms/bom/src/main/generated/flattened-full-pom.xml
index 9aab1bc7d7..ef1499de74 100644
--- a/poms/bom/src/main/generated/flattened-full-pom.xml
+++ b/poms/bom/src/main/generated/flattened-full-pom.xml
@@ -1364,6 +1364,14 @@
camel-infinispan
4.4.0
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+org.infinispan
+infinispan-client-hotrod-jakarta
+
org.slf4j
jcl-over-slf4j
diff --git a/poms/bom/src/main/generated/flattened-reduced-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-pom.xml
index 0fa7ef209c..b0b7abfca9 100644
--- a/poms/bom/src/main/generated/flattened-reduced-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-pom.xml
@@ -1364,6 +1364,14 @@
camel-infinispan
4.4.0
+
+org.infinispan
+infinispan-commons-jakarta
+
+
+
(camel-quarkus) 01/02: Upgrade Quarkus to 3.9.0.CR1
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a commit to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git commit 4fbc6774dbf1f116e530a0cf7ae3881500604ae5 Author: James Netherton AuthorDate: Wed Feb 28 08:13:46 2024 + Upgrade Quarkus to 3.9.0.CR1 --- extensions/google-bigquery/deployment/pom.xml | 4 extensions/google-bigquery/runtime/pom.xml| 4 extensions/google-pubsub/deployment/pom.xml | 4 extensions/google-pubsub/runtime/pom.xml | 4 extensions/grpc/deployment/pom.xml| 4 extensions/grpc/runtime/pom.xml | 8 extensions/salesforce/deployment/pom.xml | 4 extensions/salesforce/runtime/pom.xml | 4 .../jdbc/db2/src/test/resources/container-license-acceptance.txt | 2 +- pom.xml | 8 poms/bom/src/main/generated/flattened-full-pom.xml| 2 +- poms/bom/src/main/generated/flattened-reduced-pom.xml | 2 +- poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml | 2 +- .../java/org/apache/camel/quarkus/test/AfterEachCallback.java | 1 + .../java/org/apache/camel/quarkus/test/BeforeEachCallback.java| 1 + .../test/userTestCases/AdviceInDoBeforeEachMethodsTest.java | 6 +++--- 16 files changed, 49 insertions(+), 11 deletions(-) diff --git a/extensions/google-bigquery/deployment/pom.xml b/extensions/google-bigquery/deployment/pom.xml index e4cb3b702f..193d2935dc 100644 --- a/extensions/google-bigquery/deployment/pom.xml +++ b/extensions/google-bigquery/deployment/pom.xml @@ -54,6 +54,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/google-bigquery/runtime/pom.xml b/extensions/google-bigquery/runtime/pom.xml index 15a4b77477..9dfe0da15d 100644 --- a/extensions/google-bigquery/runtime/pom.xml +++ b/extensions/google-bigquery/runtime/pom.xml @@ -90,6 +90,10 @@ io.grpc grpc-netty + +com.google.code.findbugs +jsr305 + com.google.android annotations diff --git a/extensions/google-pubsub/deployment/pom.xml b/extensions/google-pubsub/deployment/pom.xml index 31b8b31739..3820cbf30b 100644 --- a/extensions/google-pubsub/deployment/pom.xml +++ b/extensions/google-pubsub/deployment/pom.xml @@ -58,6 +58,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/google-pubsub/runtime/pom.xml b/extensions/google-pubsub/runtime/pom.xml index 9161883ace..437a7ebb91 100644 --- a/extensions/google-pubsub/runtime/pom.xml +++ b/extensions/google-pubsub/runtime/pom.xml @@ -113,6 +113,10 @@ com.google.android annotations + +com.google.code.findbugs +jsr305 + commons-logging commons-logging diff --git a/extensions/grpc/deployment/pom.xml b/extensions/grpc/deployment/pom.xml index db1b712e67..e606eeb03e 100644 --- a/extensions/grpc/deployment/pom.xml +++ b/extensions/grpc/deployment/pom.xml @@ -50,6 +50,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/grpc/runtime/pom.xml b/extensions/grpc/runtime/pom.xml index bc4e5e3974..b58d4f005d 100644 --- a/extensions/grpc/runtime/pom.xml +++ b/extensions/grpc/runtime/pom.xml @@ -39,6 +39,10 @@ io.quarkus quarkus-grpc-common + +com.google.code.findbugs +jsr305 + io.quarkus quarkus-vertx @@ -55,6 +59,10 @@ io.vertx vertx-grpc-server + +org.checkerframework +checker-qual + diff --git a/extensions/salesforce/deployment/pom.xml b/extensions/salesforce/deployment/pom.xml index 2b2e620592..cd5e5989c7 100644 ---
[PR] [3.8.x] Upgrade Quarkus to 3.8.2 [camel-quarkus]
jamesnetherton opened a new pull request, #5852: URL: https://github.com/apache/camel-quarkus/pull/5852 (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] chore(traits): store executed traits [camel-k]
squakez closed pull request #5216: chore(traits): store executed traits URL: https://github.com/apache/camel-k/pull/5216 -- 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] chore(traits): store executed traits [camel-k]
squakez commented on PR #5216: URL: https://github.com/apache/camel-k/pull/5216#issuecomment-1983735186 I realized there are more inconsistencies that need to be tackled before this one. Closing for now and I'll open it again once the rest of problems are fixed. -- 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(runtime): get rid off camel k runtime dependency - WIP [camel-k]
squakez closed pull request #5090: feat(runtime): get rid off camel k runtime dependency - WIP URL: https://github.com/apache/camel-k/pull/5090 -- 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: Upgrade Quarkus to 3.8.2
This is an automated email from the ASF dual-hosted git repository.
jamesnetherton 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 ff9d9545e2 Upgrade Quarkus to 3.8.2
ff9d9545e2 is described below
commit ff9d9545e267d6481e5315d6dc9645c8162d65e5
Author: James Netherton
AuthorDate: Thu Mar 7 13:23:26 2024 +
Upgrade Quarkus to 3.8.2
---
docs/antora.yml | 2 +-
pom.xml | 4 ++--
poms/bom/src/main/generated/flattened-full-pom.xml| 2 +-
poms/bom/src/main/generated/flattened-reduced-pom.xml | 2 +-
poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml | 2 +-
5 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/docs/antora.yml b/docs/antora.yml
index 6ccd463a0a..3bb969c940 100644
--- a/docs/antora.yml
+++ b/docs/antora.yml
@@ -31,7 +31,7 @@ asciidoc:
camel-version: 4.4.0 # replace ${camel.version}
camel-docs-version: 4.4.x
camel-quarkus-version: 3.9.0 # replace ${camel-quarkus.version}
-quarkus-version: 3.8.1 # replace ${quarkus.version}
+quarkus-version: 3.8.2 # replace ${quarkus.version}
graalvm-version: 23.0.1 # replace ${graalvm.version}
graalvm-docs-version: jdk21 # replace ${graalvm-docs.version}
mapstruct-version: 1.5.5.Final # replace ${mapstruct.version}
diff --git a/pom.xml b/pom.xml
index f799b8b548..6f0197b7c4 100644
--- a/pom.xml
+++ b/pom.xml
@@ -61,7 +61,7 @@
2.2.2
2.3.1
2.0.2
-3.8.1
+3.8.2
4.0.0
2.6.0
@@ -148,7 +148,7 @@
3.4.34
${reactor-netty-version}
2.9.0
-
4.16.2
+
4.18.0
2.2
${snakeyaml-engine-version}
${spring-version}
diff --git a/poms/bom/src/main/generated/flattened-full-pom.xml
b/poms/bom/src/main/generated/flattened-full-pom.xml
index f99b46207f..7d91ee6e1e 100644
--- a/poms/bom/src/main/generated/flattened-full-pom.xml
+++ b/poms/bom/src/main/generated/flattened-full-pom.xml
@@ -6452,7 +6452,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
diff --git a/poms/bom/src/main/generated/flattened-reduced-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-pom.xml
index 18c4c0b601..079603546f 100644
--- a/poms/bom/src/main/generated/flattened-reduced-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-pom.xml
@@ -6407,7 +6407,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
diff --git a/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
b/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
index 96ee06ac96..bbd9fcdea0 100644
--- a/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
+++ b/poms/bom/src/main/generated/flattened-reduced-verbose-pom.xml
@@ -6407,7 +6407,7 @@
io.smallrye.reactive
smallrye-reactive-messaging-camel
-4.16.2
+4.18.0
io.swagger.core.v3
Re: [PR] Upgrade Quarkus to 3.8.2 [camel-quarkus]
jamesnetherton merged PR #5848: URL: https://github.com/apache/camel-quarkus/pull/5848 -- 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-k) branch main updated (1f51a53c9 -> 727e295ab)
This is an automated email from the ASF dual-hosted git repository. cdeppisch pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git from 1f51a53c9 chore(deps): bump github.com/cloudevents/sdk-go/v2 from 2.13.0 to 2.15.2 add 322c4b750 chore(lint): Error nil check: use require.NoError add 149738df8 chore(lint): Error not nil check: use require.Error add 1ce6e8efe chore(lint): assert NoError check: use require.NoError add 491e7d0b1 chore(lint): assert Error check: use require.Error add 88eed782d chore(lint): Use golangci-lint version matching go version 1.21 add 33a5ed2ef chore(lint): Fix lint errors add fc8d9176d chore(lint): Ignore tests add 45f2c1aa5 chore(lint): Fix lint errors add 41713d06a chore(lint): Skip test_support go files add 727e295ab chore(lint): Fix lint errors No new revisions were added by this update. Summary of changes: .github/workflows/validate.yml | 2 +- .golangci.yml | 6 +- addons/keda/keda_test.go | 39 ++--- addons/strimzi/strimzi_test.go | 7 +- addons/telemetry/discovery/locator.go | 2 +- addons/telemetry/telemetry_test.go | 11 +- addons/threescale/3scale_test.go | 15 +- addons/tracing/discovery/locator.go| 2 +- addons/tracing/tracing_test.go | 7 +- addons/vault/aws/aws_secrets_manager_test.go | 19 ++- addons/vault/azure/azure_key_vault_test.go | 15 +- addons/vault/gcp/gcp_secret_manager_test.go| 11 +- addons/vault/hashicorp/hashicorp_vault_test.go | 15 +- e2e/common/cli/config_test.go | 3 +- e2e/common/misc/client_test.go | 13 +- e2e/common/misc/registry_maven_wagon_test.go | 10 +- e2e/common/misc/rest_test.go | 5 +- e2e/common/misc/scale_binding_test.go | 4 +- e2e/common/traits/jvm_test.go | 6 +- e2e/common/traits/openapi_test.go | 4 +- e2e/common/traits/prometheus_test.go | 3 +- e2e/common/traits/route_test.go| 7 +- e2e/install/cli/install_test.go| 8 +- e2e/install/olm/olm_install_test.go| 4 +- e2e/install/upgrade/olm_upgrade_test.go| 4 +- e2e/knative/knative_platform_test.go | 7 +- e2e/knative/openapi_test.go| 4 +- e2e/support/test_support.go| 4 +- e2e/support/test_util.go | 5 +- e2e/support/util/structured_log_test.go| 3 +- pkg/apis/camel/v1/build_type_support_test.go | 6 +- pkg/apis/camel/v1/common_types_support_test.go | 2 +- .../camel/v1/integration_types_support_test.go | 4 +- pkg/apis/camel/v1/knative/types_support_test.go| 3 +- pkg/apis/camel/v1/maven_types_support_test.go | 21 +-- pkg/apis/camel/v1/pipe_types_support_test.go | 5 +- .../v1alpha1/kamelet_binding_types_support_test.go | 5 +- pkg/builder/builder_test.go| 3 +- pkg/builder/image_test.go | 7 +- pkg/builder/project_test.go| 35 ++-- pkg/builder/quarkus_test.go| 41 ++--- pkg/builder/types.go | 2 +- pkg/client/client.go | 2 +- pkg/cmd/bind_test.go | 16 +- pkg/cmd/builder_test.go| 7 +- pkg/cmd/config_test.go | 25 +-- pkg/cmd/delete_test.go | 7 +- pkg/cmd/install.go | 4 - pkg/cmd/install_test.go| 155 + pkg/cmd/kamelet_add_repo_test.go | 28 +-- pkg/cmd/kamelet_remove_repo_test.go| 11 +- pkg/cmd/kit_create_test.go | 17 +- pkg/cmd/modeline_test.go | 123 +++--- pkg/cmd/operator_test.go | 9 +- pkg/cmd/promote_test.go| 21 +-- pkg/cmd/rebuild_test.go| 7 +- pkg/cmd/run_support_test.go| 15 +- pkg/cmd/run_test.go| 188 ++--- pkg/cmd/source/content_test.go | 29 ++-- pkg/cmd/source/util_test.go| 23 +-- pkg/cmd/uninstall_test.go | 7 +- pkg/cmd/version_test.go| 37 ++-- pkg/controller/build/build_monitor_test.go | 19 ++- pkg/controller/build/build_pod_test.go | 3 +- pkg/controller/integration/health_test.go | 5 +-
Re: [PR] chore(lint): Update golangci-lint version to 1.55 [camel-k]
christophd merged PR #5227: URL: https://github.com/apache/camel-k/pull/5227 -- 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(trait): Add documentation and test on multiple sources [camel-k]
gansheer opened a new pull request, #5229: URL: https://github.com/apache/camel-k/pull/5229 Closes #5158 **Release Note** ```release-note fix(trait): Add documentation and test on multiple sources for cron trait ``` -- 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 (a03b66fa24 -> dc6508b333)
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a change to branch camel-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git discard a03b66fa24 Upgrade Camel to 4.5.0 new dc6508b333 Upgrade Camel to 4.5.0 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 (a03b66fa24) \ N -- N -- N refs/heads/camel-main (dc6508b333) 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 1 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: .../camel/quarkus/core/deployment/catalog/SchemaResource.java | 2 +- extensions-jvm/azure-cosmosdb/deployment/pom.xml| 4 extensions-jvm/azure-cosmosdb/runtime/pom.xml | 4 .../test/java/org/apache/camel/quarkus/js/JavaScriptDslTest.java| 2 +- .../java/org/apache/camel/quarkus/main/CoreMainXmlJaxbTest.java | 1 - integration-tests/ssh/pom.xml | 2 ++ .../test/java/org/apache/camel/quarkus/component/ssh/it/SshIT.java | 2 ++ .../java/org/apache/camel/quarkus/component/ssh/it/SshTest.java | 2 ++ .../apache/camel/quarkus/component/velocity/it/VelocityTest.java| 6 +++--- 9 files changed, 19 insertions(+), 6 deletions(-)
Re: [PR] feat(jbang): camel-k agent sub-command [camel]
lburgazzoli commented on PR #13408: URL: https://github.com/apache/camel/pull/13408#issuecomment-1983688129 > is this Camel K specific? I wonder if this could go into the core Camel JBang instead of the k plugin? I do not know, at this stage I'm mainly focus to be able to replicate what camel-k needs so I added this to the camel-k subcommand, but could become more generic at some point. -- 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(jbang): camel-k agent sub-command [camel]
lburgazzoli commented on code in PR #13408:
URL: https://github.com/apache/camel/pull/13408#discussion_r1516294071
##
dsl/camel-jbang/camel-jbang-plugin-k/src/main/java/org/apache/camel/dsl/jbang/core/commands/k/support/SourceHelper.java:
##
@@ -0,0 +1,71 @@
+/*
+ * 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.dsl.jbang.core.commands.k.support;
+
+import java.io.File;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+import org.apache.camel.main.KameletMain;
+
+public final class SourceHelper {
+
+private SourceHelper() {
+//prevent instantiation of utility class.
+}
+
+public static Set dependencies(List sources) throws
Exception {
+List files = new ArrayList<>();
+
+for (Source s : sources) {
+if (s.local() && !s.path().startsWith("classpath:")) {
+// get the absolute path for a local file
+files.add("file://" + new File(s.path()).getAbsolutePath());
+} else {
+files.add(s.path());
+}
+}
+
+KameletMain main = new KameletMain();
+DependenciesDownloadListener downloadListener = new
DependenciesDownloadListener(sources);
Review Comment:
thx, this is pretty much a leftover that I was supposed to remove, Initially
I did try to use the download listeners but realized that in fact we do no need
to download anything to perform route inspection.
--
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(jbang): camel-k agent sub-command [camel]
claudio4j commented on code in PR #13408:
URL: https://github.com/apache/camel/pull/13408#discussion_r1516287383
##
dsl/camel-jbang/camel-jbang-plugin-k/src/main/java/org/apache/camel/dsl/jbang/core/commands/k/support/SourceHelper.java:
##
@@ -0,0 +1,71 @@
+/*
+ * 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.dsl.jbang.core.commands.k.support;
+
+import java.io.File;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Set;
+
+import org.apache.camel.main.KameletMain;
+
+public final class SourceHelper {
+
+private SourceHelper() {
+//prevent instantiation of utility class.
+}
+
+public static Set dependencies(List sources) throws
Exception {
+List files = new ArrayList<>();
+
+for (Source s : sources) {
+if (s.local() && !s.path().startsWith("classpath:")) {
+// get the absolute path for a local file
+files.add("file://" + new File(s.path()).getAbsolutePath());
+} else {
+files.add(s.path());
+}
+}
+
+KameletMain main = new KameletMain();
+DependenciesDownloadListener downloadListener = new
DependenciesDownloadListener(sources);
Review Comment:
just a note, there is a [PR to
revert](https://github.com/apache/camel/pull/13404) 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] CAMEL-20095 : add langchain component [camel]
zbendhiba commented on code in PR #13341:
URL: https://github.com/apache/camel/pull/13341#discussion_r1516284211
##
components/camel-ai/camel-langchain/src/main/java/org/apache/camel/component/langchain/service/Langchain4jChatHandler.java:
##
@@ -0,0 +1,132 @@
+/*
+ * 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.langchain.service;
+
+import java.util.List;
+import java.util.Map;
+
+import dev.langchain4j.data.message.AiMessage;
+import dev.langchain4j.data.message.ChatMessage;
+import dev.langchain4j.data.message.ChatMessageType;
+import dev.langchain4j.data.message.SystemMessage;
+import dev.langchain4j.data.message.UserMessage;
+import dev.langchain4j.model.chat.ChatLanguageModel;
+import dev.langchain4j.model.input.Prompt;
+import dev.langchain4j.model.input.PromptTemplate;
+import dev.langchain4j.model.output.Response;
+
+public class Langchain4jChatHandler {
+
+ChatLanguageModel chatLanguageModel;
+
+public Langchain4jChatHandler(ChatLanguageModel chatLanguageModel) {
+this.chatLanguageModel = chatLanguageModel;
+}
+
+/**
+ * Send one simple message
+ *
+ * @param message
+ * @return
+ */
+public String sendMessage(String message) {
+return this.chatLanguageModel.generate(message);
+}
+
+public String sendWithPromptTemplate(String promptTemplate, Map variables) {
+PromptTemplate template = PromptTemplate.from(promptTemplate);
+Prompt prompt = template.apply(variables);
+return this.sendMessage(prompt.text());
+}
+
+public String sendWithPromptTemplate(
+String promptTemplate, Map variables,
ChatMessageType chatMessageType) {
+PromptTemplate template = PromptTemplate.from(promptTemplate);
+Prompt prompt = template.apply(variables);
+return this.sendMessage(prompt.text(), chatMessageType);
+}
+
+/**
+ * Send a simple message, for a specific Chat Message Type
+ *
+ * @param message
+ * @param type
+ * @return
+ */
+public String sendMessage(String message, ChatMessageType type) {
+ChatMessage[] chatMessages = { convertChatMessages(type, message) };
+return this.sendMessage(chatMessages);
+}
+
+/**
+ * Send a List of Messages
+ *
+ * @param type
+ * @param messages
+ * @return
+ */
+public String sendMessages(ChatMessageType type, List messages) {
+
+var chatMessages = messages.stream()
+.map(message -> (ChatMessage) convertChatMessages(type,
message))
+.toArray(ChatMessage[]::new);
+
+return sendMessage(chatMessages);
+}
+
+/**
+ * Create the messages in the format of Langchain4j ChatMessage
+ *
+ * @param type
+ * @param message
+ * @return
+ */
+private ChatMessage convertChatMessages(ChatMessageType type, String
message) {
+return switch (type) {
+case USER -> new UserMessage(message);
+case AI -> new AiMessage(message);
+case SYSTEM -> new SystemMessage(message);
+case TOOL_EXECUTION_RESULT -> throw new
IllegalArgumentException("Tools are currently not supported");
+};
+
+}
+
+/**
+ * Send And Array of ChatMessages
+ *
+ * @param messages
+ * @return
+ */
+public String sendMessage(ChatMessage... messages) {
Review Comment:
I removed this one.
--
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-20535 - Camel-AWS-Bedrock: Add the component for the bedrock ru… [camel]
github-actions[bot] commented on PR #13410: URL: https://github.com/apache/camel/pull/13410#issuecomment-1983657907 :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: [PR] CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock ru… [camel]
oscerd commented on PR #13410: URL: https://github.com/apache/camel/pull/13410#issuecomment-1983656748 /component-test aws -- 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-20535 - Camel-AWS-Bedrock: Add the component for the bedrock ru… [camel]
github-actions[bot] commented on PR #13410: URL: https://github.com/apache/camel/pull/13410#issuecomment-1983656178 :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-20535 - Camel-AWS-Bedrock: Add the component for the bedrock ru… [camel]
oscerd opened a new pull request, #13410: URL: https://github.com/apache/camel/pull/13410 …ntime agent for knowledge base support # 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 - [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). # 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
(camel) branch CAMEL-20535 created (now f42236c01a3)
This is an automated email from the ASF dual-hosted git repository. acosentino pushed a change to branch CAMEL-20535 in repository https://gitbox.apache.org/repos/asf/camel.git at f42236c01a3 CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock runtime agent for knowledge base support This branch includes the following new commits: new f42236c01a3 CAMEL-20535 - Camel-AWS-Bedrock: Add the component for the bedrock runtime agent for knowledge base support The 1 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.
[I] [camel-main] SSH tests fail due to NoSuchMethodError: boolean org.apache.sshd.client.future.ConnectFuture.await(long) [camel-quarkus]
jamesnetherton opened a new issue, #5850: URL: https://github.com/apache/camel-quarkus/issues/5850 ### Bug description Camel upgraded to SSHD 2.12.1 but Quarkus 3.8.x is currently aligned with 2.12.0. There were changes in 2.12.1 in `VerifiableFuture` & `WaitableFuture`. https://github.com/apache/mina-sshd/compare/sshd-2.12.0...sshd-2.12.1 ``` Caused by: java.lang.NoSuchMethodError: 'boolean org.apache.sshd.client.future.ConnectFuture.await(long)' at org.apache.camel.component.ssh.SshHelper.sendExecCommand(SshHelper.java:69) at org.apache.camel.component.ssh.SshProducer.process(SshProducer.java:82) at org.apache.camel.support.AsyncProcessorConverterHelper$ProcessorToAsyncProcessorBridge.process(AsyncProcessorConverterHelper.java:65) at org.apache.camel.impl.engine.SharedCamelInternalProcessor.processNonTransacted(SharedCamelInternalProcessor.java:199) at org.apache.camel.impl.engine.SharedCamelInternalProcessor.process(SharedCamelInternalProcessor.java:173) at org.apache.camel.impl.engine.SharedCamelInternalProcessor$1.process(SharedCamelInternalProcessor.java:109) at org.apache.camel.impl.engine.DefaultAsyncProcessorAwaitManager.process(DefaultAsyncProcessorAwaitManager.java:82) at org.apache.camel.impl.engine.SharedCamelInternalProcessor.process(SharedCamelInternalProcessor.java:106) at org.apache.camel.support.cache.DefaultProducerCache.send(DefaultProducerCache.java:178) at org.apache.camel.impl.engine.DefaultProducerTemplate.send(DefaultProducerTemplate.java:176) at org.apache.camel.impl.engine.DefaultProducerTemplate.send(DefaultProducerTemplate.java:172) at org.apache.camel.impl.engine.DefaultProducerTemplate.send(DefaultProducerTemplate.java:153) at org.apache.camel.impl.engine.DefaultProducerTemplate.sendBody(DefaultProducerTemplate.java:187) at org.apache.camel.impl.engine.DefaultProducerTemplate.sendBody(DefaultProducerTemplate.java:195) at org.apache.camel.quarkus.component.ssh.it.SshResource.writeToFile(SshResource.java:57) at org.apache.camel.quarkus.component.ssh.it.SshResource_ClientProxy.writeToFile(Unknown Source) ``` -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[I] [camel-main] Kamelet tests fail at build time due to StackOverflowError [camel-quarkus]
jamesnetherton opened a new issue, #5849: URL: https://github.com/apache/camel-quarkus/issues/5849 ### Bug description Also seen on the Kamelet Chuck Norris example project. ``` [INFO] Running org.apache.camel.quarkus.component.kamelet.it.KameletTest 2024-03-07 14:08:00,823 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (build-4) Apache Camel 4.5.0-SNAPSHOT (camel-2) is shutting down (timeout:45s) 2024-03-07 14:08:00,828 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (build-4) Routes stopped (total:0) 2024-03-07 14:08:00,832 INFO [org.apa.cam.imp.eng.AbstractCamelContext] (build-4) Apache Camel 4.5.0-SNAPSHOT (camel-2) shutdown in 11ms (uptime:) [ERROR] Tests run: 7, Failures: 0, Errors: 1, Skipped: 6, Time elapsed: 1.885 s <<< FAILURE! -- in org.apache.camel.quarkus.component.kamelet.it.KameletTest [ERROR] org.apache.camel.quarkus.component.kamelet.it.KameletTest.testKameletWithProperties -- Time elapsed: 0.003 s <<< ERROR! java.lang.RuntimeException: java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors [error]: Build step io.quarkus.deployment.steps.MainClassBuildStep#build threw an exception: java.lang.StackOverflowError at java.base/java.lang.Class.privateGetDeclaredConstructors(Class.java:3362) at java.base/java.lang.Class.getConstructors(Class.java:2060) at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadComplexObject(BytecodeRecorderImpl.java:1173) at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstanceImpl(BytecodeRecorderImpl.java:914) at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstance(BytecodeRecorderImpl.java:586) at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadComplexObject(BytecodeRecorderImpl.java:1357) ``` -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(camel-quarkus) branch camel-main updated (ec87692c2c -> a03b66fa24)
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a change to branch camel-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git discard ec87692c2c Upgrade Camel to 4.5.0 new a03b66fa24 Upgrade Camel to 4.5.0 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 (ec87692c2c) \ N -- N -- N refs/heads/camel-main (a03b66fa24) 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 1 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: .../src/main/java/org/apache/camel/quarkus/core/FastCamelContext.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
