[GitHub] [camel-k] astefanutti merged pull request #2754: Fix #2638: fix condition for external kit and add test
astefanutti merged pull request #2754: URL: https://github.com/apache/camel-k/pull/2754 -- 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: Fix #2638: fix condition for external kit and add test
This is an automated email from the ASF dual-hosted git repository. astefanutti pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git The following commit(s) were added to refs/heads/main by this push: new 44618f2 Fix #2638: fix condition for external kit and add test 44618f2 is described below commit 44618f24704caa12afb57ac7b07b43fcc7a9e7b2 Author: nicolaferraro AuthorDate: Wed Nov 17 13:09:30 2021 +0100 Fix #2638: fix condition for external kit and add test --- e2e/common/scale_integration_test.go | 21 + pkg/trait/container.go | 4 ++-- 2 files changed, 23 insertions(+), 2 deletions(-) diff --git a/e2e/common/scale_integration_test.go b/e2e/common/scale_integration_test.go index 7863f9f..5c70901 100644 --- a/e2e/common/scale_integration_test.go +++ b/e2e/common/scale_integration_test.go @@ -23,6 +23,7 @@ limitations under the License. package common import ( + "fmt" "testing" . "github.com/onsi/gomega" @@ -119,6 +120,26 @@ func TestIntegrationScale(t *testing.T) { Should(gstruct.PointTo(BeNumerically("==", 1))) }) + t.Run("Scale integration with external image", func(t *testing.T) { + RegisterTestingT(t) + + image := IntegrationPodImage(ns, name)() + Expect(image).NotTo(BeEmpty()) + // Save resources by deleting the integration + Expect(Kamel("delete", name, "-n", ns).Execute()).To(Succeed()) + + Expect(Kamel("run", "-n", ns, "files/Java.java", "--name", "pre-built", "-t", fmt.Sprintf("container.image=%s", image)).Execute()).To(Succeed()) + Eventually(IntegrationPhase(ns, "pre-built"), TestTimeoutShort).Should(Equal(v1.IntegrationPhaseRunning)) + Eventually(IntegrationPodPhase(ns, "pre-built"), TestTimeoutMedium).Should(Equal(corev1.PodRunning)) + Expect(ScaleIntegration(ns, "pre-built", 0)).To(Succeed()) + Eventually(IntegrationPod(ns, "pre-built"), TestTimeoutMedium).Should(BeNil()) + Expect(ScaleIntegration(ns, "pre-built", 1)).To(Succeed()) + Eventually(IntegrationPhase(ns, "pre-built"), TestTimeoutShort).Should(Equal(v1.IntegrationPhaseRunning)) + Eventually(IntegrationPodPhase(ns, "pre-built"), TestTimeoutMedium).Should(Equal(corev1.PodRunning)) + + Expect(Kamel("delete", "pre-built", "-n", ns).Execute()).To(Succeed()) + }) + // Clean up RegisterTestingT(t) Expect(Kamel("delete", "--all", "-n", ns).Execute()).To(Succeed()) diff --git a/pkg/trait/container.go b/pkg/trait/container.go index a12b261..b0f5bf5 100644 --- a/pkg/trait/container.go +++ b/pkg/trait/container.go @@ -167,8 +167,8 @@ func isValidPullPolicy(policy corev1.PullPolicy) bool { } func (t *containerTrait) Apply(e *Environment) error { - if e.IntegrationInPhase(v1.IntegrationPhaseInitialization) { - return t.configureImageIntegrationKit(e) + if err := t.configureImageIntegrationKit(e); err != nil { + return err } return t.configureContainer(e) }
[GitHub] [camel-k] astefanutti commented on pull request #2754: Fix #2638: fix condition for external kit and add test
astefanutti commented on pull request #2754: URL: https://github.com/apache/camel-k/pull/2754#issuecomment-977620690 Let's merge this. The CI failure are unrelated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] astefanutti commented on a change in pull request #2766: gosec
astefanutti commented on a change in pull request #2766: URL: https://github.com/apache/camel-k/pull/2766#discussion_r755779181 ## File path: .golangci.yml ## @@ -18,54 +18,46 @@ linters-settings: lll: line-length: 170 + goconst: +ignore-tests: true linters: - disable-all: true - enable: -- asciicheck -- bodyclose -- deadcode -- depguard -- dogsled -- durationcheck -- errcheck -- errname -- errorlint -- exportloopref -- forcetypeassert -- gocritic -- gofmt + enable-all: true Review comment: On one hand with `disable-all`, it's less work to upgrade, but we would lag behind what would be the "default" / de-facto standard set of linters, and we would create a "debt" over time that would have to be pay by some larger one-time efforts, on the other hand with `enable-all`, there is some extra work possible each time we upgrade, which may biased us from not upgrading, but we would inherit from the "default", possibly better set of linters. I'd be inclined toward trusting `golangci-lint` and trying with `enable-all`, so we dilute the possible overhead, keeping in mind that we can always revert to `disable-all` if that does not work. -- 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 (7031f97 -> 367cc34)
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 7031f97 (chores) camel-salesforce: fix a potential NPE (#6466) add 25b49c1a Update SpringTransactionPolicy.java (#6467) add b162e79 Upgrade AWS SDK v2 to version 2.17.88 add 367cc34 Sync deps No new revisions were added by this update. Summary of changes: camel-dependencies/pom.xml | 2 +- .../org/apache/camel/spring/spi/SpringTransactionPolicy.java | 12 parent/pom.xml | 2 +- 3 files changed, 14 insertions(+), 2 deletions(-)
[GitHub] [camel-k] astefanutti closed issue #2753: Add 1.7.x documentation tag
astefanutti closed issue #2753: URL: https://github.com/apache/camel-k/issues/2753 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] astefanutti commented on issue #2753: Add 1.7.x documentation tag
astefanutti commented on issue #2753: URL: https://github.com/apache/camel-k/issues/2753#issuecomment-977606269 It should be fixed with #2757. -- 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] 01/01: Upgrade Quarkus to 2.6.0.CR1
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git commit ae9a3a2516cf348d19fafeffe9d8e07c83b27578 Author: James Netherton AuthorDate: Thu Nov 11 14:04:24 2021 + Upgrade Quarkus to 2.6.0.CR1 --- docs/antora.yml | 2 +- .../camel/quarkus/component/csimple/deployment/CSimpleProcessor.java | 5 +++-- pom.xml | 4 ++-- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/docs/antora.yml b/docs/antora.yml index bc1bbc6..b5c2ecc 100644 --- a/docs/antora.yml +++ b/docs/antora.yml @@ -27,7 +27,7 @@ asciidoc: min-maven-version: 3.6.2 # replace ${min-maven-version} target-maven-version: 3.8.1 # replace ${target-maven-version} camel-version: 3.13.0 # replace ${camel.version} -quarkus-version: 2.5.0.Final # replace ${quarkus.version} +quarkus-version: 999-SNAPSHOT # replace ${quarkus.version} # attributes used in xrefs to other Antora components cq-camel-components: 3.13.x@components # replace ${camel.docs.components.xref} quarkus-examples-version: latest diff --git a/extensions/csimple/deployment/src/main/java/org/apache/camel/quarkus/component/csimple/deployment/CSimpleProcessor.java b/extensions/csimple/deployment/src/main/java/org/apache/camel/quarkus/component/csimple/deployment/CSimpleProcessor.java index 72229a3..c5266fd 100644 --- a/extensions/csimple/deployment/src/main/java/org/apache/camel/quarkus/component/csimple/deployment/CSimpleProcessor.java +++ b/extensions/csimple/deployment/src/main/java/org/apache/camel/quarkus/component/csimple/deployment/CSimpleProcessor.java @@ -310,8 +310,9 @@ class CSimpleProcessor { csimpleClassesDir.toFile(), StandardCharsets.UTF_8.name(), Collections.emptyList(), -"1.8", -"1.8", +null, +"11", +"11", Collections.emptyList(), Collections.emptyList()); } diff --git a/pom.xml b/pom.xml index 113ab49..937808d 100644 --- a/pom.xml +++ b/pom.xml @@ -50,7 +50,7 @@ 8.9.1.Final 0.3.0 2.3.0 -2.5.0.Final +999-SNAPSHOT 2.0.0 0.30.0 @@ -105,7 +105,7 @@ 2.4.7 2.8.1 ${kudu-version} -1.5.31 +1.6.0 ${libthrift-version} ${metrics-version} ${mvel-version}
[camel-quarkus] branch quarkus-main updated (8654a17 -> ae9a3a2)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch quarkus-main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git. discard 8654a17 Upgrade Quarkus to 2.6.0.CR1 add 2e47c10 Updated CHANGELOG.md add 886b58a Document that vertx-websocket consumers run on the Quarkus Vert.x web server add 7dc8268 Repalce reference to quarkus-vertx-web with quarkus-vertx-http add 1b6a587 Fix MicroProfileHealthTest.testFailureThreshold test failure new ae9a3a2 Upgrade Quarkus to 2.6.0.CR1 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 (8654a17) \ N -- N -- N refs/heads/quarkus-main (ae9a3a2) 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: CHANGELOG.md | 8 +++ .../pages/reference/extensions/platform-http.adoc | 2 +- .../reference/extensions/vertx-websocket.adoc | 38 +- .../platform-http/runtime/src/main/doc/intro.adoc | 2 +- .../runtime/src/main/doc/configuration.adoc| 12 + .../runtime/src/main/doc/intro.adoc| 3 ++ .../runtime/src/main/doc/usage.adoc| 13 + .../vertx/websocket/VertxWebsocketRecorder.java| 2 +- ...Check.java => FailureThresholdHealthCheck.java} | 24 +++-- .../it/health/MicroProfileHealthResource.java | 42 ++- .../it/health/MicroProfileHealthRouteBuilder.java | 3 -- .../it/health/MicroProfileHealthTest.java | 60 -- 12 files changed, 168 insertions(+), 41 deletions(-) create mode 100644 extensions/vertx-websocket/runtime/src/main/doc/configuration.adoc create mode 100644 extensions/vertx-websocket/runtime/src/main/doc/intro.adoc create mode 100644 extensions/vertx-websocket/runtime/src/main/doc/usage.adoc copy integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/{FailingHealthCheck.java => FailureThresholdHealthCheck.java} (65%)
[GitHub] [camel] orpiske commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977596211 > I do not have strong opinion, but to me, having the camel-dependencies to be regenerated is not a limitation or a problem. Maybe externalize would work, but don't we have the properties duplicated anyway? In the parent pom and in the externalized properties file? This build system is stable from a while, eventhough it's not perfect. Yeah. I think the build works just fine. It can certainly be improved, of course, but it does not seem to me like there's a big problem. I think the biggest problem may be for vendoring, when there's any sort of checking of the sources during the build. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977594555 > > camel-dependencies -> camel-catalog -> camel-dependencies (sync-properties) > > No I dont think this is correct - its > > > camel-catalog -> camel-dependencies (sync-properties) > Actually, we do seem to build the [camel-dependencies before the catalog](https://github.com/apache/camel/blob/main/pom.xml#L72). > In the camel-catalog there is a maven plugin that sync that camel-dependencies. > > However what is the "problem" is that running camel-catalog should be done after all the components are build etc, which was tricky in the past with maven as it can build in "whatever order it thinks its best" - so there is this camel-allcomponents "trick". I understand. But it seems to me that we are using the catalog build only to refresh the dependency pom, no? BTW, I'm not trying to push to an specific solution here ... but it does seem odd that we have this cyclic dependency (but, as you know, I may be missing previous context 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
[camel] branch main updated: Polished
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 bd364fe8 Polished bd364fe8 is described below commit bd364fe8cac16dedb45bc45dc5bdd29024610640 Author: Claus Ibsen AuthorDate: Wed Nov 24 08:07:09 2021 +0100 Polished --- catalog/camel-catalog/pom.xml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/catalog/camel-catalog/pom.xml b/catalog/camel-catalog/pom.xml index d434a7e..d6c5982 100644 --- a/catalog/camel-catalog/pom.xml +++ b/catalog/camel-catalog/pom.xml @@ -160,14 +160,13 @@ - + org.apache.camel sync-properties-maven-plugin ${project.version} - sync-properties
[GitHub] [camel] davsclaus commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
davsclaus commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977589743 So at the end of the day, what we can try is to move that generation of camel-dependencies out of catalog, and to some place where its run at the very end. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] davsclaus commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
davsclaus commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977589080 > camel-dependencies -> camel-catalog -> camel-dependencies (sync-properties) No I dont think this is correct - its > camel-catalog -> camel-dependencies (sync-properties) In the camel-catalog there is a maven plugin that sync that camel-dependencies. However what is the "problem" is that running camel-catalog should be done after all the components are build etc, which was tricky in the past with maven as it can build in "whatever order it thinks its best" - so there is this camel-allcomponents "trick". -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] oscerd commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
oscerd commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977588411 I do not have strong opinion, but to me, having the camel-dependencies to be regenerated is not a limitation or a problem. Maybe externalize would work, but don't we have the properties duplicated anyway? In the parent pom and in the externalized properties file? This build system is stable from a while, eventhough it's not perfect. -- 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 (25b49c1a -> 7031f97)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch regen_bot in repository https://gitbox.apache.org/repos/asf/camel.git. omit 25b49c1a Update SpringTransactionPolicy.java (#6467) This update removed existing revisions from the reference, leaving the reference pointing at a previous point in the repository history. * -- * -- N refs/heads/regen_bot (7031f97) \ O -- O -- O (25b49c1a) Any revisions marked "omit" are not gone; other references still refer to them. Any revisions marked "discard" are gone forever. No new revisions were added by this update. Summary of changes: .../org/apache/camel/spring/spi/SpringTransactionPolicy.java | 12 1 file changed, 12 deletions(-)
[GitHub] [camel] orpiske commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977586176 > The properties maven plugin won't write a full POM like we need for the downstream projects. This was something that was evaluated at the time, but in the need wasn't good for the job. The idea is not to rewrite the pom. The idea is to externalize only the properties in the pom. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] oscerd commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
oscerd commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977585678 The properties maven plugin won't write a full POM like we need for the downstream projects. This was something that was evaluated at the time, but in the need wasn't good for the job. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] orpiske edited a comment on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske edited a comment on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977583530 Can't we solve this by separating the concerns? I mean like separating refreshing the dependencies from the dependency management? We could use the [properties-maven plugin](http://www.mojohaus.org/properties-maven-plugin/) to write and read a properties file. One potential benefit is that, with this way we could remove the all duplicated dependencies which are also currently listed in the parent pom. IMHO it could make dependency maintenance much simpler and the build code smaller. Currently we have: `camel-dependencies` -> `camel-catalog` -> `camel-dependencies` (sync-properties) Instead, we could refresh the dependencies as part of the the `camel-parent` (or `camel-dependencies` ?) build. That way, it would become: `camel-parent` (or `camel-dependencies`) -> `camel-catalog` -> likely no need to regenerate anything related to the camel-dependencies itself. **Granted**: this is a much bigger proposition, I know, but may lead to a cleaner build and remove the cyclic dependency. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] orpiske edited a comment on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske edited a comment on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977583530 Can't we solve this by separating the concerns? I mean like separating refreshing the dependencies from the dependency management? We could use the [properties-maven plugin](http://www.mojohaus.org/properties-maven-plugin/) to write and read a properties file. One potential benefit is that, with this way we could remove the all duplicated dependencies which are also currently listed in the parent pom. IMHO it could make dependency maintenance much simpler and the build code smaller. Currently we have: `camel-dependencies` -> `camel-catalog` -> `camel-dependencies` (sync-properties) Instead, we could refresh the dependencies as part of the the `camel-parent` (or `camel-dependencies` ?) build. That way, it would become: `camel-parent` (or `camel-dependencies`) -> `camel-catalog` -> likely no need to regenerate anything. **Granted**: this is a much bigger proposition, I know, but may lead to a cleaner build and remove the cyclic dependency. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] orpiske commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
orpiske commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977583530 Can't we solve this by separating the concerns? I mean like separating refreshing the dependencies from the dependency management? We could use the [properties-maven plugin](http://www.mojohaus.org/properties-maven-plugin/) to write and read a properties file. One potential benefit is that, with this way we could remove the all duplicated dependencies which are also currently listed in the parent pom. IMHO it could make dependency maintenance much simpler and the build code smaller. Currently we have: `camel-dependencies` -> `camel-catalog` -> `camel-dependencies` (sync-properties) Instead, we could refresh the dependencies as part of the the `camel-parent` (or `camel-dependencies` ?) build. Granted: this is a much bigger proposition, but may lead to a cleaner build and remove the cyclic dependency. -- 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 (b7144c1 -> 25b49c1a)
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 b7144c1 CAMEL-17226 Add namespace to path used for leadership election when using Zookeeper with Master component (#6463) add 7031f97 (chores) camel-salesforce: fix a potential NPE (#6466) add 25b49c1a Update SpringTransactionPolicy.java (#6467) No new revisions were added by this update. Summary of changes: .../salesforce/internal/streaming/SubscriptionHelper.java| 9 - .../org/apache/camel/spring/spi/SpringTransactionPolicy.java | 12 2 files changed, 16 insertions(+), 5 deletions(-)
[GitHub] [camel] oscerd commented on pull request #6462: CAMEL-17062 Move camel-dependencies inside of camel-catalog to avoid deployment issues
oscerd commented on pull request #6462: URL: https://github.com/apache/camel/pull/6462#issuecomment-977580896 I think having a camel-dependencies pom evident in the source tree, it's a good choice. I don't see it as a limitation. Maybe we could find a way to build it later in the build, after the catalog. -- 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 (25b49c1a -> 367cc34)
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 25b49c1a Update SpringTransactionPolicy.java (#6467) new b162e79 Upgrade AWS SDK v2 to version 2.17.88 new 367cc34 Sync deps 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: camel-dependencies/pom.xml | 2 +- parent/pom.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
[camel] 01/02: Upgrade AWS SDK v2 to version 2.17.88
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.git commit b162e79cdbc7f9467ce5dbd7a30a184236b76033 Author: Andrea Cosentino AuthorDate: Wed Nov 24 06:58:00 2021 +0100 Upgrade AWS SDK v2 to version 2.17.88 --- parent/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parent/pom.xml b/parent/pom.xml index 603170a..a028cd3 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -80,7 +80,7 @@ 1.10.2 1.10.2 4.1.1 -2.17.87 +2.17.88 2.6.1 1.2.14 2.11.3
[camel] 02/02: Sync deps
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.git commit 367cc341b2ba5dd0c676c62a40910590db2620cf Author: Andrea Cosentino AuthorDate: Wed Nov 24 07:44:30 2021 +0100 Sync deps --- camel-dependencies/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/camel-dependencies/pom.xml b/camel-dependencies/pom.xml index 462d45f..04b28a3 100644 --- a/camel-dependencies/pom.xml +++ b/camel-dependencies/pom.xml @@ -65,7 +65,7 @@ 1.10.2 1.10.2 4.1.1 -2.17.87 +2.17.88 2.6.1 1.2.14 4.20.0
[camel] branch regen_bot updated (495de76 -> b7144c1)
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 495de76 Revert "Upgrade formatter-maven-plugin to 2.17.0" add b7144c1 CAMEL-17226 Add namespace to path used for leadership election when using Zookeeper with Master component (#6463) No new revisions were added by this update. Summary of changes: .../component/zookeeper/cluster/ZooKeeperClusterView.java| 12 +--- .../modules/ROOT/pages/camel-3x-upgrade-guide-3_14.adoc | 3 +++ 2 files changed, 12 insertions(+), 3 deletions(-)
[GitHub] [camel-k] lburgazzoli commented on a change in pull request #2766: gosec
lburgazzoli commented on a change in pull request #2766: URL: https://github.com/apache/camel-k/pull/2766#discussion_r755737923 ## File path: .golangci.yml ## @@ -18,54 +18,46 @@ linters-settings: lll: line-length: 170 + goconst: +ignore-tests: true linters: - disable-all: true - enable: -- asciicheck -- bodyclose -- deadcode -- depguard -- dogsled -- durationcheck -- errcheck -- errname -- errorlint -- exportloopref -- forcetypeassert -- gocritic -- gofmt + enable-all: true Review comment: I can revert to the `disable-all` mode, I don't have any strong opinion here. The reason I did it the other way is to be sure we are not missing any important lint (as example. the `gosec` one was missing) so I did try to run it with all the possible lints and then disabling those that are just noise. @astefanutti @squakez @tadayosi since I'm not contributing to the project very often now, I let you decide -- 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: Fix xref links, remove them (its hard to understand how to link between sub projects)
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-quarkus.git The following commit(s) were added to refs/heads/main by this push: new 506b229 Fix xref links, remove them (its hard to understand how to link between sub projects) 506b229 is described below commit 506b229dbe21dc5fa8a119606801e0e9a515d1e8 Author: Claus Ibsen AuthorDate: Wed Nov 24 07:07:55 2021 +0100 Fix xref links, remove them (its hard to understand how to link between sub projects) --- docs/modules/ROOT/pages/migration-guide/2.6.0.adoc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/modules/ROOT/pages/migration-guide/2.6.0.adoc b/docs/modules/ROOT/pages/migration-guide/2.6.0.adoc index 81dd32c..e32d379 100644 --- a/docs/modules/ROOT/pages/migration-guide/2.6.0.adoc +++ b/docs/modules/ROOT/pages/migration-guide/2.6.0.adoc @@ -4,11 +4,11 @@ The following guide outlines how to adapt your code to changes that were made in == Graceful shutdown strategy is used by default -In previous releases, graceful shutdown strategy (default strategy in Camel, see https://camel.apache.org/manual/graceful-shutdown.html[documentation] ) wasn't used by default. +In previous releases, graceful shutdown strategy (default strategy in Camel) wasn't used by default. Shutdown wasn't controlled by any strategy. This is no longer the case. -Graceful shutdown strategy is enabled by default (see https://camel.apache.org/manual/graceful-shutdown.html[documentation]) with one exception only. +Graceful shutdown strategy is enabled by default with one exception only. If timeout for graceful shutdown is not set and application runs in a development mode, no shutdown strategy is used (behavior for the development mode without timeout for graceful timeout wasn't changed). `DefaultShutdownStrategy` could be configured via `application.properties`.
[camel] branch main updated: Update SpringTransactionPolicy.java (#6467)
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 25b49c1a Update SpringTransactionPolicy.java (#6467) 25b49c1a is described below commit 25b49c1ae970b4bc2cf6bbfb9e02a59895e4d2c0 Author: Benjamin Graf AuthorDate: Wed Nov 24 06:59:07 2021 +0100 Update SpringTransactionPolicy.java (#6467) --- .../org/apache/camel/spring/spi/SpringTransactionPolicy.java | 12 1 file changed, 12 insertions(+) diff --git a/components/camel-spring/src/main/java/org/apache/camel/spring/spi/SpringTransactionPolicy.java b/components/camel-spring/src/main/java/org/apache/camel/spring/spi/SpringTransactionPolicy.java index d264ad4..505d0fa 100644 --- a/components/camel-spring/src/main/java/org/apache/camel/spring/spi/SpringTransactionPolicy.java +++ b/components/camel-spring/src/main/java/org/apache/camel/spring/spi/SpringTransactionPolicy.java @@ -38,6 +38,7 @@ import org.springframework.transaction.support.TransactionTemplate; public class SpringTransactionPolicy implements TransactedPolicy { private static final Logger LOG = LoggerFactory.getLogger(SpringTransactionPolicy.class); private TransactionTemplate template; +private String name; private String propagationBehaviorName; private PlatformTransactionManager transactionManager; @@ -133,6 +134,9 @@ public class SpringTransactionPolicy implements TransactedPolicy { if (template == null) { ObjectHelper.notNull(transactionManager, "transactionManager"); template = new TransactionTemplate(transactionManager); +if (name != null) { +template.setName(name); +} if (propagationBehaviorName != null) { template.setPropagationBehaviorName(propagationBehaviorName); } @@ -152,6 +156,14 @@ public class SpringTransactionPolicy implements TransactedPolicy { return transactionManager; } +public void setName(String name) { +this.name = name; +} + +public String getName() { +return name; +} + public void setPropagationBehaviorName(String propagationBehaviorName) { this.propagationBehaviorName = propagationBehaviorName; }
[GitHub] [camel] davsclaus merged pull request #6467: CAMEL-17227: camel-spring - add name property for TransactionTemplate
davsclaus merged pull request #6467: URL: https://github.com/apache/camel/pull/6467 -- 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 (b7144c1 -> 7031f97)
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 b7144c1 CAMEL-17226 Add namespace to path used for leadership election when using Zookeeper with Master component (#6463) add 7031f97 (chores) camel-salesforce: fix a potential NPE (#6466) No new revisions were added by this update. Summary of changes: .../salesforce/internal/streaming/SubscriptionHelper.java| 9 - 1 file changed, 4 insertions(+), 5 deletions(-)
[camel] branch main updated (495de76 -> b7144c1)
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 495de76 Revert "Upgrade formatter-maven-plugin to 2.17.0" add b7144c1 CAMEL-17226 Add namespace to path used for leadership election when using Zookeeper with Master component (#6463) No new revisions were added by this update. Summary of changes: .../component/zookeeper/cluster/ZooKeeperClusterView.java| 12 +--- .../modules/ROOT/pages/camel-3x-upgrade-guide-3_14.adoc | 3 +++ 2 files changed, 12 insertions(+), 3 deletions(-)
[GitHub] [camel] davsclaus merged pull request #6466: (chores) camel-salesforce: fix a potential NPE
davsclaus merged pull request #6466: URL: https://github.com/apache/camel/pull/6466 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] davsclaus merged pull request #6463: CAMEL-17226 Add namespace to path used for leadership election when using Zookeeper with Master component
davsclaus merged pull request #6463: URL: https://github.com/apache/camel/pull/6463 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] tadayosi commented on a change in pull request #2766: gosec
tadayosi commented on a change in pull request #2766: URL: https://github.com/apache/camel-k/pull/2766#discussion_r755685485 ## File path: .golangci.yml ## @@ -18,54 +18,46 @@ linters-settings: lll: line-length: 170 + goconst: +ignore-tests: true linters: - disable-all: true - enable: -- asciicheck -- bodyclose -- deadcode -- depguard -- dogsled -- durationcheck -- errcheck -- errname -- errorlint -- exportloopref -- forcetypeassert -- gocritic -- gofmt + enable-all: true Review comment: I changed in a previous commit here from enable-all (denylisting) to disable-all (allowlisting), because otherwise every time we try to upgrade golangci-lint it will introduce a new set of linters which weren't checked before and thus upgrading golangci-lint will become painful. I thought using `disable-all` and allowlisting makes upgrading the golangci-lint action easier. So, do you think it's not a good idea and we should catch up with new additions to linters along with upgrade of the action? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] tadayosi commented on a change in pull request #2775: fix(build): default jitpack to main
tadayosi commented on a change in pull request #2775: URL: https://github.com/apache/camel-k/pull/2775#discussion_r755681491 ## File path: pkg/util/jitpack/jitpack.go ## @@ -24,10 +24,13 @@ import ( ) const ( - RepoURL = "https://jitpack.io"; - LatestVersion = "master-SNAPSHOT" + // RepoURL is the Jitpack repository url + RepoURL = "https://jitpack.io"; + // DefaultVersion is the default branch/version to use + DefaultVersion = "main-SNAPSHOT" ) +// ToDependency converts a jitpack dependency into Depdendency struct Review comment: Not a big deal, but a small typo: `Depdendency` -> `Dependency` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which uses JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 Affected tests - activemq - atlasmap - git - nsq - openapi-java - optaplanner and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- 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: Updated CHANGELOG.md
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git The following commit(s) were added to refs/heads/main by this push: new c646d34 Updated CHANGELOG.md c646d34 is described below commit c646d3416aa4c8d4c5b48742c9f4f94e804a7fd2 Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> AuthorDate: Wed Nov 24 03:11:12 2021 + Updated CHANGELOG.md --- CHANGELOG.md | 4 1 file changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0497638..f9f70b2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,11 +6,15 @@ **Closed issues:** +- Document that `vertx-websocket` consumers run on the Quarkus Vert.x web server [\#3318](https://github.com/apache/camel-quarkus/issues/3318) - FOP integration test failed in native mode [\#3280](https://github.com/apache/camel-quarkus/issues/3280) +- \[Camel 3.13.0\] `MicroProfileHealthTest.testFailureThreshold` test failure [\#3277](https://github.com/apache/camel-quarkus/issues/3277) - `quarkus.camel.main.shutdown.timeout` doesn't work as intended [\#3179](https://github.com/apache/camel-quarkus/issues/3179) **Merged pull requests:** +- Fix `MicroProfileHealthTest.testFailureThreshold` test failure [\#3322](https://github.com/apache/camel-quarkus/pull/3322) ([jamesnetherton](https://github.com/jamesnetherton)) +- Document that vertx-websocket consumers run on the Quarkus Vert.x web server [\#3320](https://github.com/apache/camel-quarkus/pull/3320) ([jamesnetherton](https://github.com/jamesnetherton)) - Next is 2.6.0 [\#3317](https://github.com/apache/camel-quarkus/pull/3317) ([zbendhiba](https://github.com/zbendhiba)) - Disable FOP integration test failed in native mode on Mac OS \#3280 [\#3314](https://github.com/apache/camel-quarkus/pull/3314) ([ffang](https://github.com/ffang)) - Make building possible with Java 11 through 17 [\#3313](https://github.com/apache/camel-quarkus/pull/3313) ([ppalaga](https://github.com/ppalaga))
[camel-k] branch main updated: Updated CHANGELOG.md
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git The following commit(s) were added to refs/heads/main by this push: new e012129 Updated CHANGELOG.md e012129 is described below commit e012129ae178c1a68a4d885a844e58de899c396c Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> AuthorDate: Wed Nov 24 03:09:31 2021 + Updated CHANGELOG.md --- CHANGELOG.md | 4 1 file changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 6806109..3676332 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,8 @@ **Closed issues:** +- Dry run kamel run [\#2769](https://github.com/apache/camel-k/issues/2769) +- Is there any resources requirements for camel-k operator Deployment? [\#2768](https://github.com/apache/camel-k/issues/2768) - How can I find the log file location in the integration pod? [\#2762](https://github.com/apache/camel-k/issues/2762) - Could not pull base image - Build kit failed \(No matching credentials were found, falling back on anonymous\) [\#2748](https://github.com/apache/camel-k/issues/2748) - Cannot deploy XML integration with undertow [\#2746](https://github.com/apache/camel-k/issues/2746) @@ -24,6 +26,8 @@ **Merged pull requests:** +- doc\(install\): suggested resource configuration [\#2774](https://github.com/apache/camel-k/pull/2774) ([squakez](https://github.com/squakez)) +- chore\(release\): Update latest released version and OLM base CSV manifest [\#2770](https://github.com/apache/camel-k/pull/2770) ([astefanutti](https://github.com/astefanutti)) - chore\(build\): Upgrade Maven to version 3.8.4 [\#2767](https://github.com/apache/camel-k/pull/2767) ([astefanutti](https://github.com/astefanutti)) - chore\(release\): Remove staging repository [\#2750](https://github.com/apache/camel-k/pull/2750) ([astefanutti](https://github.com/astefanutti)) - chore\(e2e\): disable object truncation on test results [\#2745](https://github.com/apache/camel-k/pull/2745) ([tadayosi](https://github.com/tadayosi))
[GitHub] [camel-quarkus] github-actions[bot] commented on issue #2927: [CI] - Camel Main Branch Build Failure
github-actions[bot] commented on issue #2927: URL: https://github.com/apache/camel-quarkus/issues/2927#issuecomment-977456564 The [camel-main](https://github.com/apache/camel-quarkus/tree/camel-main) branch build has failed: * Build ID: 1497193378-476-61bf38a1-3782-4336-bcb6-31f0c289c7d4 * Camel Quarkus Commit: 2104129f1c4ee948250e65e061f5e89fc7a4dabf * Camel Main Commit: 1b6a587fdebb1282b42c7befd8a41d324d202475 * Link to build: https://github.com/apache/camel-quarkus/actions/runs/1497193378 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which uses JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 Affected tests - activemq - atlasmap - git - nsq - openapi-java and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman edited a comment on issue #2776: Error: unknown flag: --trait on local build
scodeman edited a comment on issue #2776: URL: https://github.com/apache/camel-k/issues/2776#issuecomment-977381236 Note that the dry-run (cf:#2769) ``` kamel run IntegrationFile.java -o yaml``` call works as expected providing me the following output ``` Modeline options have been loaded from source files Full command: kamel run IntegrationFile.java -o yaml --trait=container.enabled=true --trait=container.image=localhost:5000/camelk_test:latest ``` ```yaml apiVersion: camel.apache.org/v1 kind: Integration metadata: creationTimestamp: null name: integration-file namespace: default spec: sources: - content: | // camel-k: language=java // camel-k: trait=container.enabled=true // camel-k: trait=container.image=localhost:5000/camelk_test:latest import org.apache.camel.builder.RouteBuilder; public class IntegrationFile extends RouteBuilder { @Override public void configure() throws Exception { // Write your routes here, for example: from("timer:java?period=1000") .routeId("java") .setBody() .simple("Hello Camel K from ${routeId}") .to("log:info"); } } name: IntegrationFile.java traits: container: configuration: enabled: true image: localhost:5000/camelk_test:latest status: {} ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman edited a comment on issue #2776: Error: unknown flag: --trait on local build
scodeman edited a comment on issue #2776: URL: https://github.com/apache/camel-k/issues/2776#issuecomment-977381236 Note that the dry-run (cf:#2769) ``` kamel run IntegrationFile.java -o yaml``` call works as expected providing me the following output ``` Modeline options have been loaded from source files Full command: kamel run IntegrationFile.java -o yaml --trait=container.image=localhost:5000/integration-base-image:latest ``` ```yaml apiVersion: camel.apache.org/v1 kind: Integration metadata: creationTimestamp: null name: integration-file namespace: default spec: sources: - content: | // camel-k: language=java // camel-k: trait=container.enabled=true // camel-k: trait=container.image=localhost:5000/camelk_test:latest import org.apache.camel.builder.RouteBuilder; public class IntegrationFile extends RouteBuilder { @Override public void configure() throws Exception { // Write your routes here, for example: from("timer:java?period=1000") .routeId("java") .setBody() .simple("Hello Camel K from ${routeId}") .to("log:info"); } } name: IntegrationFile.java traits: container: configuration: enabled: true image: localhost:5000/camelk_test:latest status: {} ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman edited a comment on issue #2776: Error: unknown flag: --trait on local build
scodeman edited a comment on issue #2776: URL: https://github.com/apache/camel-k/issues/2776#issuecomment-977381236 Note that the dry-run (cf:#2769) ``` kamel run IntegrationFile.java -o yaml``` call works as expected providing me the following output ``` Modeline options have been loaded from source files Full command: kamel run IntegrationFile.java -o yaml --trait=container.image=localhost:5000/integration-base-image:latest ``` ```yaml apiVersion: camel.apache.org/v1 kind: Integration metadata: creationTimestamp: null name: integration-file namespace: default spec: sources: - content: | // camel-k: language=java // camel-k: trait=container.enabled=true // camel-k: trait=container.image=localhost:5000/camelk_test:latest import org.apache.camel.builder.RouteBuilder; public class IntegrationFile extends RouteBuilder { @Override public void configure() throws Exception { // Write your routes here, for example: from("timer:java?period=1000") .routeId("java") .setBody() .simple("Hello Camel K from ${routeId}") .to("log:info"); } } name: IntegrationFile.java traits: container: configuration: image: localhost:5000/camelk_test:latest status: {} ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman commented on issue #2776: Error: unknown flag: --trait on local build
scodeman commented on issue #2776: URL: https://github.com/apache/camel-k/issues/2776#issuecomment-977381236 Note that the dry-run (cf:#2769) ``` kamel run IntegrationFile.java -o yaml``` call works as expected providing me the following output ``` Modeline options have been loaded from source files Full command: kamel run IntegrationFile.java -o yaml --trait=container.image=localhost:5000/integration-base-image:latest ``` ```yaml apiVersion: camel.apache.org/v1 kind: Integration metadata: creationTimestamp: null name: integration-file namespace: default spec: sources: - content: | // camel-k: language=java // camel-k: trait=container.image=localhost:5000/integration-base-image:latest import org.apache.camel.builder.RouteBuilder; public class IntegrationFile extends RouteBuilder { @Override public void configure() throws Exception { // Write your routes here, for example: from("timer:java?period=1000") .routeId("java") .setBody() .simple("Hello Camel K from ${routeId}") .to("log:info"); } } name: IntegrationFile.java traits: container: configuration: image: localhost:5000/integration-base-image:latest status: {} ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman opened a new issue #2776: Error: unknown flag: --trait on local build
scodeman opened a new issue #2776: URL: https://github.com/apache/camel-k/issues/2776 As per #2769 I am trying to build a local integration image and use it in for my integration manifest. I have the following IntegrationFile.java simple java DSL example with modelines: ```java // camel-k: language=java // camel-k: trait=container.image=localhost:5000/integration-base-image:latest import org.apache.camel.builder.RouteBuilder; public class IntegrationFile extends RouteBuilder { @Override public void configure() throws Exception { // Write your routes here, for example: from("timer:java?period=1000") .routeId("java") .setBody() .simple("Hello Camel K from ${routeId}") .to("log:info"); } } ``` But it fails to local build ```kamel local build IntegrationFile.java``` with the following error reported ``` Modeline options have been loaded from source files Full command: kamel local build IntegrationFile.java --trait=container.image=localhost:5000/integration-base-image:latest Error: unknown flag: --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
[GitHub] [camel-k] scodeman commented on issue #2769: Dry run kamel run
scodeman commented on issue #2769: URL: https://github.com/apache/camel-k/issues/2769#issuecomment-977362451 Yes, perfect that does it. Thank you. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] scodeman closed issue #2769: Dry run kamel run
scodeman closed issue #2769: URL: https://github.com/apache/camel-k/issues/2769 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which uses JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 Affected tests - activemq - atlasmap - git - nsq and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which uses JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 Affected tests 1. activemq 2.atlasmap 3.git 4.nsq and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-karavan] mgubaidullin commented on issue #116: JBang - run in JVM debug mode so you can do JVM remote debugging
mgubaidullin commented on issue #116: URL: https://github.com/apache/camel-karavan/issues/116#issuecomment-977266050 I think this feature outside Karavan for now. If we have yaml route debugger in VSCode perhaps we could try to visualize it on the diagram. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-karavan] mgubaidullin commented on issue #112: JBang live reload
mgubaidullin commented on issue #112: URL: https://github.com/apache/camel-karavan/issues/112#issuecomment-977257707 @davsclaus route does not start after stop if change log parameters. ex. https://youtu.be/lAVERmz8tu4 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-website] mgubaidullin opened a new pull request #685: [Blog] Karavan Serverless Mode
mgubaidullin opened a new pull request #685: URL: https://github.com/apache/camel-website/pull/685 A new blog post about new feature in Karavan preview release 0.0.8: Karavan Serverless mode -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3323: [JDK17]integration test failed in native mode on Mac which depends on quarkus-awt feature
ffang edited a comment on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976823947 affected tests azure-grouped main-xml-jaxb dozer hazelcast infinispan jaxb -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3323: [JDK17]integration test failed in native mode on Mac which depends on quarkus-awt feature
ffang edited a comment on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976823947 affected tests azure-grouped main-xml-jaxb dozer hazelcast infinispan -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3323: [JDK17]integration test failed in native mode on Mac which depends on quarkus-awt feature
ffang edited a comment on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976823947 affected tests azure-grouped main-xml-jaxb dozer hazelcast -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which using JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 Affected tests activemq atlasmap git and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3326: [JDK17]integration test failed in native mode caused by unresolved type during parsing: sun.security.krb5.Credentials
ffang edited a comment on issue #3326: URL: https://github.com/apache/camel-quarkus/issues/3326#issuecomment-977030352 affected tests debezium We can fix it by adding property -J--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED in native profile Also, this is reported and fixed at quarkus by https://github.com/quarkusio/quarkus/issues/21359 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang commented on issue #3326: [JDK17]integration test failed in native mode caused by unresolved type during parsing: sun.security.krb5.Credentials
ffang commented on issue #3326: URL: https://github.com/apache/camel-quarkus/issues/3326#issuecomment-977030352 affected tests debezium We can fix it by adding property -J--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED in native profile -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang opened a new issue #3326: [JDK17]integration test failed in native mode caused by unresolved type during parsing: sun.security.krb5.Credentials
ffang opened a new issue #3326: URL: https://github.com/apache/camel-quarkus/issues/3326 ``` Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: sun.security.krb5.Credentials. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time. Detailed message: Trace: at parsing org.postgresql.util.internal.Unsafe.credentialCacheExists(Unsafe.java:18) Call path from entry point to org.postgresql.util.internal.Unsafe.credentialCacheExists(): at org.postgresql.util.internal.Unsafe.credentialCacheExists(Unsafe.java:17) at org.postgresql.core.v3.ConnectionFactoryImpl.credentialCacheExists(ConnectionFactoryImpl.java:413) at org.postgresql.core.v3.ConnectionFactoryImpl.enableGSSEncrypted(ConnectionFactoryImpl.java:430) at org.postgresql.core.v3.ConnectionFactoryImpl.tryConnect(ConnectionFactoryImpl.java:146) at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:245) at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:51) at org.postgresql.jdbc.PgConnection.(PgConnection.java:225) at org.postgresql.Driver.makeConnection(Driver.java:466) at org.postgresql.Driver.access$100(Driver.java:63) at org.postgresql.Driver$ConnectThread.run(Driver.java:376) at java.lang.Thread.run(Thread.java:833) at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:596) at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192) at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0) com.oracle.svm.core.util.UserError$UserException: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: sun.security.krb5.Credentials. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time. Detailed message: Trace: at parsing org.postgresql.util.internal.Unsafe.credentialCacheExists(Unsafe.java:18) Call path from entry point to org.postgresql.util.internal.Unsafe.credentialCacheExists(): at org.postgresql.util.internal.Unsafe.credentialCacheExists(Unsafe.java:17) at org.postgresql.core.v3.ConnectionFactoryImpl.credentialCacheExists(ConnectionFactoryImpl.java:413) at org.postgresql.core.v3.ConnectionFactoryImpl.enableGSSEncrypted(ConnectionFactoryImpl.java:430) at org.postgresql.core.v3.ConnectionFactoryImpl.tryConnect(ConnectionFactoryImpl.java:146) at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:245) at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:51) at org.postgresql.jdbc.PgConnection.(PgConnection.java:225) at org.postgresql.Driver.makeConnection(Driver.java:466) at org.postgresql.Driver.access$100(Driver.java:63) at org.postgresql.Driver$ConnectThread.run(Driver.java:376) at java.lang.Thread.run(Thread.java:833) at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:596) at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192) at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0) at com.oracle.svm.core.util.UserError.abort(UserError.java:87) at com.oracle.svm.hosted.FallbackFeature.reportAsFallback(FallbackFeature.java:233) at com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:759) at com.oracle.svm.hosted.NativeImageGenerator.doRun(NativeImageGenerator.java:529) at com.oracle.svm.hosted.NativeImageGenerator.run(NativeImageGenerator.java:488) at com.oracle.svm.hosted.NativeImageGeneratorRunner.buildImage(NativeImageGeneratorRunner.java:403) at com.oracle.svm.hosted.NativeImageGeneratorRunner.build(NativeImageGeneratorRunner.java:569) at com.oracle.svm.hosted.NativeImageGeneratorRunner.main(NativeImageGeneratorRunner.java:122) at com.oracle.svm.hosted.NativeImageGeneratorRunner$JDK9Plus.main(NativeImageGeneratorRunner.java:599) ``` This is JDK17 specifiic and should occur on all OS -- 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.
[GitHub] [camel-quarkus] ffang edited a comment on issue #3323: [JDK17]integration test failed in native mode on Mac which depends on quarkus-awt feature
ffang edited a comment on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976823947 affected tests azure-grouped main-xml-jaxb dozer -- 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 (ee6a51b -> 495de76)
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 ee6a51b CAMEL-17225: camel-core - Using predicates in Java DSL with ValueBuilder should init as predicate add 495de76 Revert "Upgrade formatter-maven-plugin to 2.17.0" No new revisions were added by this update. Summary of changes: camel-dependencies/pom.xml | 2 +- parent/pom.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which using JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 This also affects other extensions(atlasmap, activemq) native test which using jmx and we can add property ``` -J--add-exports=java.management/sun.management=ALL-UNNAMED ``` to native profile before bumping the quarkus version as temporary fix -- 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: Revert "Upgrade formatter-maven-plugin to 2.17.0"
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.git The following commit(s) were added to refs/heads/main by this push: new 495de76 Revert "Upgrade formatter-maven-plugin to 2.17.0" 495de76 is described below commit 495de7645a1ed6139f650a7a371cd4ce5bdf9890 Author: Andrea Cosentino AuthorDate: Tue Nov 23 18:56:12 2021 +0100 Revert "Upgrade formatter-maven-plugin to 2.17.0" This reverts commit c4b5681ef5e5ee5bd5b33aae96dcaba846ae3125. Formatter-maven-plugin 2.17.0 doesn't work with JDK 8. We need to postpone to after 3.14.0 --- camel-dependencies/pom.xml | 2 +- parent/pom.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/camel-dependencies/pom.xml b/camel-dependencies/pom.xml index 0a41f7f..462d45f 100644 --- a/camel-dependencies/pom.xml +++ b/camel-dependencies/pom.xml @@ -199,7 +199,7 @@ 4.0.5 1.14.0 2.6 -2.17.0 +2.16.0 2.3.31 1.1.1 1.1.1 diff --git a/parent/pom.xml b/parent/pom.xml index f51345e..603170a 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -181,7 +181,7 @@ 4.0.5 1.14.0 2.6 -2.17.0 +2.16.0 1.1.1 2.3.31 1.1.1
[GitHub] [camel-karavan] mgubaidullin merged pull request #120: Serverless demo
mgubaidullin merged pull request #120: URL: https://github.com/apache/camel-karavan/pull/120 -- 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-karavan] branch main updated: Serverless demo (#120)
This is an automated email from the ASF dual-hosted git repository. marat pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-karavan.git The following commit(s) were added to refs/heads/main by this push: new b841511 Serverless demo (#120) b841511 is described below commit b841511b54cf8496a42319911b09324f5dbfd6df Author: Marat Gubaidullin AuthorDate: Tue Nov 23 12:33:59 2021 -0500 Serverless demo (#120) --- karavan-app/.mvn/wrapper/maven-wrapper.jar | Bin 0 -> 50710 bytes karavan-app/src/main/webapp/package-lock.json | 4 +-- karavan-demo/openshift/README.md | 2 +- .../openshift/apps/karavan/deployment.yaml | 2 +- karavan-demo/openshift/gitops.png | Bin 0 -> 341934 bytes karavan-demo/openshift/karavan-cloud.png | Bin 197201 -> 0 bytes karavan-demo/serverless/README.md | 35 +++-- karavan-demo/serverless/karavan/access.yaml| 35 + karavan-demo/serverless/karavan/deployment.yaml| 32 +++ karavan-demo/serverless/karavan/kustomization.yaml | 7 + karavan-demo/serverless/karavan/service.yaml | 17 ++ karavan-demo/serverless/serverless.png | Bin 0 -> 316398 bytes 12 files changed, 128 insertions(+), 6 deletions(-) diff --git a/karavan-app/.mvn/wrapper/maven-wrapper.jar b/karavan-app/.mvn/wrapper/maven-wrapper.jar new file mode 100644 index 000..2cc7d4a Binary files /dev/null and b/karavan-app/.mvn/wrapper/maven-wrapper.jar differ diff --git a/karavan-app/src/main/webapp/package-lock.json b/karavan-app/src/main/webapp/package-lock.json index 8ef74f0..4b2f446 100644 --- a/karavan-app/src/main/webapp/package-lock.json +++ b/karavan-app/src/main/webapp/package-lock.json @@ -1,12 +1,12 @@ { "name": "karavan", - "version": "0.0.7", + "version": "0.0.8", "lockfileVersion": 2, "requires": true, "packages": { "": { "name": "karavan", - "version": "0.0.7", + "version": "0.0.8", "dependencies": { "@patternfly/patternfly": "^4.132.2", "@patternfly/react-code-editor": "^4.3.42", diff --git a/karavan-demo/openshift/README.md b/karavan-demo/openshift/README.md index e2f5522..ec0e9be 100644 --- a/karavan-demo/openshift/README.md +++ b/karavan-demo/openshift/README.md @@ -6,7 +6,7 @@ Required: Openshift 4.7 installed ### Architecture -![Architecture](karavan-cloud.png) +![Architecture](gitops.png) ### Deploy Operators Deploy GitOps and Camel-K operators diff --git a/karavan-demo/openshift/apps/karavan/deployment.yaml b/karavan-demo/openshift/apps/karavan/deployment.yaml index e098eeb..758f943 100644 --- a/karavan-demo/openshift/apps/karavan/deployment.yaml +++ b/karavan-demo/openshift/apps/karavan/deployment.yaml @@ -25,7 +25,7 @@ spec: image: ghcr.io/apache/camel-karavan:latest env: - name: KARAVAN_MODE - value: "cloud" + value: "gitops" - name: KARAVAN_GIT_URI value: "http://gitea:3000/git/karavan.git"; - name: KARAVAN_GIT_USERNAME diff --git a/karavan-demo/openshift/gitops.png b/karavan-demo/openshift/gitops.png new file mode 100644 index 000..c97a0a8 Binary files /dev/null and b/karavan-demo/openshift/gitops.png differ diff --git a/karavan-demo/openshift/karavan-cloud.png b/karavan-demo/openshift/karavan-cloud.png deleted file mode 100644 index 9bb458b..000 Binary files a/karavan-demo/openshift/karavan-cloud.png and /dev/null differ diff --git a/karavan-demo/serverless/README.md b/karavan-demo/serverless/README.md index f071401..f956f55 100644 --- a/karavan-demo/serverless/README.md +++ b/karavan-demo/serverless/README.md @@ -1,8 +1,39 @@ # Karavan Serverless Mode demo on Kubernetes +Required: Minukube v1.24.0 installed ### Demo [Video of Karavan Serverless Mode demo on Kubernetes](https://youtu.be/m8BCqwHWHCo) - ### Architecture -**TODO** +![Architecture](serverless.png) + +### Install Camel-K +See the Apache Camel K installation page for details: (https://camel.apache.org/camel-k/next/installation/installation.html). + +```shell +minikube start +minikube addons enable registry +minikube start --addons registry +kamel install +``` + +### Install Karavan +```shell +kubectl apply -k karavan -n default +``` + +### Open Karavan +Get Karavan URL +```shell +minikube service karavan --url +``` +Result should be like +```shell +🏃 Starting tunnel for service camel-karavan. +|---|---|-|| +| NAMESPACE | NAME | TARGET PORT | URL | +|---|---|-|| +| default | camel-karavan | | http://127.0.0.1:60708 | +|---|---|-|| +``` +Open url `http://127.0.0.1:60708` in browser diff --git a/karavan-demo/serverless/karavan/access.
[GitHub] [camel-quarkus] ppalaga commented on issue #3321: camel-quarkus-support-* source JARs should have manifests
ppalaga commented on issue #3321: URL: https://github.com/apache/camel-quarkus/issues/3321#issuecomment-976874843 The manifests should contain at least “Implementation-Version” and “Specification-Version” entries. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton closed issue #3277: [Camel 3.13.0] `MicroProfileHealthTest.testFailureThreshold` test failure
jamesnetherton closed issue #3277: URL: https://github.com/apache/camel-quarkus/issues/3277 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] jamesnetherton merged pull request #3322: Fix `MicroProfileHealthTest.testFailureThreshold` test failure
jamesnetherton merged pull request #3322: URL: https://github.com/apache/camel-quarkus/pull/3322 -- 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 (7dc8268 -> 1b6a587)
This is an automated email from the ASF dual-hosted git repository. jamesnetherton pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/camel-quarkus.git. from 7dc8268 Repalce reference to quarkus-vertx-web with quarkus-vertx-http add 1b6a587 Fix MicroProfileHealthTest.testFailureThreshold test failure No new revisions were added by this update. Summary of changes: ...Check.java => FailureThresholdHealthCheck.java} | 24 +++-- .../it/health/MicroProfileHealthResource.java | 42 ++- .../it/health/MicroProfileHealthRouteBuilder.java | 3 -- .../it/health/MicroProfileHealthTest.java | 60 -- 4 files changed, 92 insertions(+), 37 deletions(-) copy integration-tests/microprofile/src/main/java/org/apache/camel/quarkus/component/microprofile/it/health/{FailingHealthCheck.java => FailureThresholdHealthCheck.java} (65%)
[GitHub] [camel-quarkus] ffang commented on issue #3323: [JDK17]integration test failed in native mode on Mac which depends on quarkus-awt feature
ffang commented on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976823947 affected tests azure-grouped main-xml-jaxb -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] squakez opened a new pull request #2775: fix(build): default jitpack to main
squakez opened a new pull request #2775: URL: https://github.com/apache/camel-k/pull/2775 * Moved default from master to main * Added some meaningful example Closes #2764 **Release Note** ```release-note NONE ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] squakez merged pull request #2774: doc(install): suggested resource configuration
squakez merged pull request #2774: URL: https://github.com/apache/camel-k/pull/2774 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-k] squakez closed issue #2768: Is there any resources requirements for camel-k operator Deployment?
squakez closed issue #2768: URL: https://github.com/apache/camel-k/issues/2768 -- 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: doc(install): suggested resource configuration
This is an automated email from the ASF dual-hosted git repository. pcongiusti pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git The following commit(s) were added to refs/heads/main by this push: new d0a5fdd doc(install): suggested resource configuration d0a5fdd is described below commit d0a5fdde320bf19d01885d767295e64114edf58c Author: Pasquale Congiusti AuthorDate: Tue Nov 23 15:16:40 2021 +0100 doc(install): suggested resource configuration Closes #2768 --- .../ROOT/pages/installation/scheduling.adoc| 26 +- 1 file changed, 21 insertions(+), 5 deletions(-) diff --git a/docs/modules/ROOT/pages/installation/scheduling.adoc b/docs/modules/ROOT/pages/installation/scheduling.adoc index 98253f0..a288584 100644 --- a/docs/modules/ROOT/pages/installation/scheduling.adoc +++ b/docs/modules/ROOT/pages/installation/scheduling.adoc @@ -1,7 +1,7 @@ [[scheduling-infra-pod]] = Scheduling infrastructure Pods and Resource managment -During the installation procedure you will be able to provide information on how to best "operationalize" your infrastructure. Through the configuration of `--node-selector`, `--toleration` and `--operator-resources` you will be able to drive the operator `Pod`s scheduling and to be able to assign resources. +During the installation procedure you will be able to provide information on how to best "operationalize" your infrastructure. Through the configuration of `--node-selector`, `--toleration` and `--operator-resources` you will be able to drive the operator `Pods` scheduling and to be able to assign resources. The usage of these advanced properties assumes you're familiar with the https://kubernetes.io/docs/concepts/scheduling-eviction/[Kubernetes Scheduling] concepts and configurations. @@ -11,7 +11,7 @@ NOTE: the aforementioned flags setting will work both with `OLM` installation an == Scheduling === Node Selectors -The most basic operation we provide is to let you assign Camel K operator `Pod`s to a specific cluster `Node` via `--node-selector` option. The functionality is based on https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/[`NodeSelector` Kubernetes feature]. As an example, you can schedule Camel K infra `Pod`s to a specific `Node` of your cluster: +The most basic operation we provide is to let you assign Camel K operator `Pods` to a specific cluster `Node` via `--node-selector` option. The functionality is based on https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/[`NodeSelector` Kubernetes feature]. As an example, you can schedule Camel K infra `Pods` to a specific `Node` of your cluster: ``` kamel install --node-selector kubernetes.io/hostname=ip-172-20-114-199.ec2.internal ... @@ -20,13 +20,13 @@ kamel install --node-selector kubernetes.io/hostname=ip-172-20-114-199.ec2.inter The option will accept any `key=value` pair as specified and supported by Kubernetes. You can specify more than one `node-selector`. === Tolerations -The `--toleration` option will let you tolerate a Camel K infra `Pod` to support any matching `Taint` according the https://kubernetes.io/docs/concepts/scheduling-eviction/taint-and-toleration/[`Taint` and `Toleration` Kubernetes feature]. As an example, let's suppose we have a node tainted as "dedicated=camel-k:NoSchedule". In order to allow the infra `Pod`s to be scheduled on that `Node` we can provide the following option during installation procedure: +The `--toleration` option will let you tolerate a Camel K infra `Pod` to support any matching `Taint` according the https://kubernetes.io/docs/concepts/scheduling-eviction/taint-and-toleration/[`Taint` and `Toleration` Kubernetes feature]. As an example, let's suppose we have a node tainted as "dedicated=camel-k:NoSchedule". In order to allow the infra `Pods` to be scheduled on that `Node` we can provide the following option during installation procedure: ``` kamel install --toleration dedicated=camel-k:NoSchedule ... ``` -The option accept a value in the following format `Key[=Value]:Effect[:Seconds]` where the values in squared bracket are considered optional and `Effect` must admit any of the `Taint`s accepted values: `NoSchedule`, `PreferNoSchedule`, `NoExecute`. You can specify more than one `toleration`. +The option accept a value in the following format `Key[=Value]:Effect[:Seconds]` where the values in squared bracket are considered optional and `Effect` must admit any of the `Taints` accepted values: `NoSchedule`, `PreferNoSchedule`, `NoExecute`. You can specify more than one `toleration`. [[scheduling-infra-pod-resources]] == Resources @@ -39,4 +39,20 @@ kamel install --operator-resources requests.cpu=256m --operator-resources limits The value expected by the option are in the form `requestType.requestResource=value` where `requestType` must be either `requests` or `limits`,
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]integration tests failed in native mode which using JMX
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 This also affects other extensions(atlasmap, activemq) native test which using jmx -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] orpiske opened a new pull request #6466: (chores) camel-salesforce: fix a potential NPE
orpiske opened a new pull request #6466: URL: https://github.com/apache/camel/pull/6466 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]activemq integration test failed in native mode
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 This also affects other extensions(atlasmap) native test which using jmx -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang edited a comment on issue #3325: [JDK17]activemq integration test failed in native mode
ffang edited a comment on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 This also affects other extensions native test which using jmx -- 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 (3b8710f -> ee6a51b)
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 3b8710f Polished add ee6a51b CAMEL-17225: camel-core - Using predicates in Java DSL with ValueBuilder should init as predicate No new revisions were added by this update. Summary of changes: .../ChoiceCompoundPredicateSimpleTest.java | 132 + .../camel/support/builder/PredicateBuilder.java| 14 +-- .../apache/camel/support/builder/ValueBuilder.java | 12 ++ 3 files changed, 151 insertions(+), 7 deletions(-) create mode 100644 core/camel-core/src/test/java/org/apache/camel/processor/ChoiceCompoundPredicateSimpleTest.java
[GitHub] [camel-quarkus] ffang commented on issue #3325: [JDK17]activemq integration test failed in native mode
ffang commented on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976748103 This also affects atlasmap native test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang commented on issue #3325: [JDK17]activemq integration test failed in native mode
ffang commented on issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325#issuecomment-976738611 Ah, nice, we have the similar issue reported and fixed at quarkus already https://github.com/quarkusio/quarkus/issues/21372 And this commit https://github.com/quarkusio/quarkus/pull/21408/commits/dc9a6ec5593928ac58681d21de8bc6a8a3de483f fixed it from quarkus-core when building the native image -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel] colin-mullikin commented on pull request #6463: Add namespace to path used for leadership election when using Zookeeper with Master component
colin-mullikin commented on pull request #6463: URL: https://github.com/apache/camel/pull/6463#issuecomment-976731094 @davsclaus Yep, not a problem. Should hopefully get to that today 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
[GitHub] [camel-quarkus] ffang opened a new issue #3325: [JDK17]activemq integration test failed in native mode
ffang opened a new issue #3325: URL: https://github.com/apache/camel-quarkus/issues/3325 get error like ``` Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: sun.management.Util. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time. Trace: at parsing com.oracle.svm.core.genscavenge.CompleteGarbageCollectorMXBean.getObjectName(CompleteGarbageCollectorMXBean.java:81) Call path from entry point to com.oracle.svm.core.genscavenge.CompleteGarbageCollectorMXBean.getObjectName(): at com.oracle.svm.core.genscavenge.CompleteGarbageCollectorMXBean.getObjectName(CompleteGarbageCollectorMXBean.java:81) at com.oracle.svm.core.jdk.management.ManagementSupport.addMXBean(ManagementSupport.java:267) at com.oracle.svm.core.jdk.management.ManagementSupport.getPlatformMBeanServer(ManagementSupport.java:259) at com.oracle.svm.core.jdk.management.Target_java_lang_management_ManagementFactory.getPlatformMBeanServer(Target_java_lang_management_ManagementFactory.java:45) at org.apache.commons.pool2.impl.BaseGenericObjectPool.jmxUnregister(BaseGenericObjectPool.java:969) at org.apache.commons.pool2.impl.GenericKeyedObjectPool.close(GenericKeyedObjectPool.java:695) at com.oracle.svm.core.posix.NativeSecureRandomFilesCloserShutdownHook.close(PosixSunSecuritySubstitutions.java:86) at com.oracle.svm.core.posix.NativeSecureRandomFilesCloserShutdownHook.run(PosixSunSecuritySubstitutions.java:70) at java.lang.Thread.run(Thread.java:833) at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:596) at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192) at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0) Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: sun.management.Util. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time. Trace: at parsing com.oracle.svm.core.genscavenge.IncrementalGarbageCollectorMXBean.getObjectName(IncrementalGarbageCollectorMXBean.java:81) Call path from entry point to com.oracle.svm.core.genscavenge.IncrementalGarbageCollectorMXBean.getObjectName(): at com.oracle.svm.core.genscavenge.IncrementalGarbageCollectorMXBean.getObjectName(IncrementalGarbageCollectorMXBean.java:81) at com.oracle.svm.core.jdk.management.ManagementSupport.addMXBean(ManagementSupport.java:267) at com.oracle.svm.core.jdk.management.ManagementSupport.getPlatformMBeanServer(ManagementSupport.java:259) at com.oracle.svm.core.jdk.management.Target_java_lang_management_ManagementFactory.getPlatformMBeanServer(Target_java_lang_management_ManagementFactory.java:45) at org.apache.commons.pool2.impl.BaseGenericObjectPool.jmxUnregister(BaseGenericObjectPool.java:969) at org.apache.commons.pool2.impl.GenericKeyedObjectPool.close(GenericKeyedObjectPool.java:695) at com.oracle.svm.core.posix.NativeSecureRandomFilesCloserShutdownHook.close(PosixSunSecuritySubstitutions.java:86) at com.oracle.svm.core.posix.NativeSecureRandomFilesCloserShutdownHook.run(PosixSunSecuritySubstitutions.java:70) at java.lang.Thread.run(Thread.java:833) at com.oracle.svm.core.thread.JavaThreads.threadStartRoutine(JavaThreads.java:596) at com.oracle.svm.core.posix.thread.PosixJavaThreads.pthreadStartRoutine(PosixJavaThreads.java:192) at com.oracle.svm.core.code.IsolateEnterStub.PosixJavaThreads_pthreadStartRoutine_e1f4a8c0039f8337338252cd8734f63a79b5e3df(generated:0) Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved type during parsing: sun.management.Util. To diagnose the issue you can use the --allow-incomplete-classpath option. The missing type is then reported at run time when it is accessed the first time. Trace: at parsing com.oracle.svm.core.heap.AbstractMemoryMXBean.getObjectName(AbstractMemoryMXBean.java:66) Call path from entry point to com.oracle.svm.core.heap.AbstractMemoryMXBean.getObjectName(): at com.oracle.svm.core.heap.AbstractMemoryMXBean.getObjectName(AbstractMemoryMXBean.java:66) at com.oracle.svm.core.jdk.management.ManagementSupport.addMXBean(ManagementSupport.java:267) at com.oracle.svm.core.jdk.management.ManagementSupport.getPlatformMBeanServer(ManagementSupport.java:259) at com.oracle.svm.core.jdk.management.Target_java_lang_management_ManagementFactory.getPlatformMBeanServer(Target_java_lang_management_Man
[GitHub] [camel-k] astefanutti merged pull request #2770: chore(release): Update latest released version and OLM base CSV manifest
astefanutti merged pull request #2770: URL: https://github.com/apache/camel-k/pull/2770 -- 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: chore(release): Update latest released version and OLM base CSV manifest
This is an automated email from the ASF dual-hosted git repository. astefanutti pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git The following commit(s) were added to refs/heads/main by this push: new d368f7a chore(release): Update latest released version and OLM base CSV manifest d368f7a is described below commit d368f7a1f2be83fee1796be7f3b6f594dbe341d3 Author: Antonin Stefanutti AuthorDate: Tue Nov 23 12:16:17 2021 +0100 chore(release): Update latest released version and OLM base CSV manifest --- config/manifests/bases/camel-k.clusterserviceversion.yaml | 2 +- script/Makefile | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/config/manifests/bases/camel-k.clusterserviceversion.yaml b/config/manifests/bases/camel-k.clusterserviceversion.yaml index 3e8dbc2..03b7414 100644 --- a/config/manifests/bases/camel-k.clusterserviceversion.yaml +++ b/config/manifests/bases/camel-k.clusterserviceversion.yaml @@ -30,7 +30,7 @@ metadata: operators.operatorframework.io/project_layout: go.kubebuilder.io/v2 repository: https://github.com/apache/camel-k support: Camel - name: camel-k.v1.7.0 + name: camel-k.v1.8.0 namespace: placeholder spec: apiservicedefinitions: {} diff --git a/script/Makefile b/script/Makefile index ed8b878..6e2fa27 100644 --- a/script/Makefile +++ b/script/Makefile @@ -16,7 +16,7 @@ VERSIONFILE := pkg/util/defaults/defaults.go VERSION ?= 1.8.0-SNAPSHOT OPERATOR_VERSION := $(subst -SNAPSHOT,,$(VERSION)) -LAST_RELEASED_VERSION := 1.6.1 +LAST_RELEASED_VERSION := 1.7.0 RUNTIME_VERSION := 1.10.0 BUILDAH_VERSION := 1.14.0 KANIKO_VERSION := 0.17.1
[GitHub] [camel-quarkus] JiriOndrusek opened a new pull request #3324: Sql test using derby doesn't start dev service and stored procedure is not working on derby db
JiriOndrusek opened a new pull request #3324: URL: https://github.com/apache/camel-quarkus/pull/3324 fixes https://github.com/apache/camel-quarkus/issues/3260 - I used a workaround for an issue in quarkus - https://github.com/quarkusio/quarkus/issues/21639 - Test still can't be executed with external derby db (hance the condition on the stored procedure test). Problem is caused by the fact, that stored procedure is a java class, which has to be uploaded to the external derby instance. With devServices (which uses in-memory db) this is not a problem. - I used different approach for the stored procedure test - creation of temp file. Which is sufficient for the test. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang commented on issue #3323: [JDK17]main-xml-jaxb integration test failed in native mode on Mac
ffang commented on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976674166 I tried to add class sun.lwawt.macosx.CInputMethod as RuntimeInitializedClassBuildItem for camel-xml-jaxb and this can fix it. But this commit https://github.com/quarkusio/quarkus/pull/21566/commits/918d4c36be669b8c110c24490b73a041436e77d8 can fix it from quarkus-awt extension, so next time once we upgrade the quarkus version to pick up the fix, this test in camel-quarkus should be also ok. I will leave this issue open until we pick up the fix from future quarkus version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[camel] branch main updated: CAMEL-17225: camel-core - Using predicates in Java DSL with ValueBuilder should init as predicate
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 ee6a51b CAMEL-17225: camel-core - Using predicates in Java DSL with ValueBuilder should init as predicate ee6a51b is described below commit ee6a51b9118361ac7a44d6f9eef9de6c842655cc Author: Claus Ibsen AuthorDate: Tue Nov 23 15:55:47 2021 +0100 CAMEL-17225: camel-core - Using predicates in Java DSL with ValueBuilder should init as predicate --- .../ChoiceCompoundPredicateSimpleTest.java | 132 + .../camel/support/builder/PredicateBuilder.java| 14 +-- .../apache/camel/support/builder/ValueBuilder.java | 12 ++ 3 files changed, 151 insertions(+), 7 deletions(-) diff --git a/core/camel-core/src/test/java/org/apache/camel/processor/ChoiceCompoundPredicateSimpleTest.java b/core/camel-core/src/test/java/org/apache/camel/processor/ChoiceCompoundPredicateSimpleTest.java new file mode 100644 index 000..83d4ca2 --- /dev/null +++ b/core/camel-core/src/test/java/org/apache/camel/processor/ChoiceCompoundPredicateSimpleTest.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.processor; + +import java.util.ArrayList; +import java.util.List; + +import org.apache.camel.ContextTestSupport; +import org.apache.camel.builder.RouteBuilder; +import org.junit.jupiter.api.Test; + +import static org.apache.camel.builder.PredicateBuilder.or; + +public class ChoiceCompoundPredicateSimpleTest extends ContextTestSupport { + +@Test +public void testNull() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(2); +getMockEndpoint("mock:data").expectedMessageCount(0); + +template.sendBody("direct:simple", null); +template.sendBody("direct:or", null); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testEmpty() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(2); +getMockEndpoint("mock:data").expectedMessageCount(0); + +template.sendBody("direct:simple", new ArrayList<>()); +template.sendBody("direct:or", new ArrayList<>()); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testEmptyOr() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(1); +getMockEndpoint("mock:data").expectedMessageCount(0); + +template.sendBody("direct:or", new ArrayList<>()); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testEmptySimple() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(1); +getMockEndpoint("mock:data").expectedMessageCount(0); + +template.sendBody("direct:simple", new ArrayList<>()); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testData() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(0); +getMockEndpoint("mock:data").expectedMessageCount(2); + +List list = new ArrayList<>(); +list.add("Hello Camel"); +template.sendBody("direct:simple", list); +template.sendBody("direct:or", list); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testDataOr() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(0); +getMockEndpoint("mock:data").expectedMessageCount(1); + +List list = new ArrayList<>(); +list.add("Hello Camel"); +template.sendBody("direct:or", list); + +assertMockEndpointsSatisfied(); +} + +@Test +public void testDataSimple() throws Exception { +getMockEndpoint("mock:empty").expectedMessageCount(0); +getMockEndpoint("mock:data").expectedMessageCount(1); + +List list = new ArrayList<>(); +list.add("Hello Camel"); +template.sendBody("direct:simple", list); + +assertMockEndpointsSatisfied(); +} + +@Override +protected RouteBuilder createRouteBuilder() { +return new RouteBuilder() { +
[GitHub] [camel-quarkus] ffang commented on issue #3323: [JDK17]main-xml-jaxb integration test failed in native mode on Mac
ffang commented on issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323#issuecomment-976663297 The same issue reported on quarkus side very recently https://github.com/quarkusio/quarkus-quickstarts/issues/989 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [camel-quarkus] ffang opened a new issue #3323: [JDK17]main-xml-jaxb integration test failed in native mode on Mac
ffang opened a new issue #3323: URL: https://github.com/apache/camel-quarkus/issues/3323 Errors like ``` Error: Class initialization of sun.lwawt.macosx.CInputMethod failed. Use the option --initialize-at-run-time=sun.lwawt.macosx.CInputMethod to explicitly request delayed initialization of this class. Detailed message: com.oracle.svm.core.util.UserError$UserException: Class initialization of sun.lwawt.macosx.CInputMethod failed. Use the option --initialize-at-run-time=sun.lwawt.macosx.CInputMethod to explicitly request delayed initialization of this class. Detailed message: at com.oracle.svm.core.util.UserError.abort(UserError.java:87) at com.oracle.svm.hosted.FallbackFeature.reportAsFallback(FallbackFeature.java:233) at com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:759) at com.oracle.svm.hosted.NativeImageGenerator.doRun(NativeImageGenerator.java:529) at com.oracle.svm.hosted.NativeImageGenerator.run(NativeImageGenerator.java:488) at com.oracle.svm.hosted.NativeImageGeneratorRunner.buildImage(NativeImageGeneratorRunner.java:403) at com.oracle.svm.hosted.NativeImageGeneratorRunner.build(NativeImageGeneratorRunner.java:569) at com.oracle.svm.hosted.NativeImageGeneratorRunner.main(NativeImageGeneratorRunner.java:122) at com.oracle.svm.hosted.NativeImageGeneratorRunner$JDK9Plus.main(NativeImageGeneratorRunner.java:599) Caused by: com.oracle.graal.pointsto.constraints.UnsupportedFeatureException: Class initialization of sun.lwawt.macosx.CInputMethod failed. Use the option --initialize-at-run-time=sun.lwawt.macosx.CInputMethod to explicitly request delayed initialization of this class. Detailed message: at com.oracle.graal.pointsto.constraints.UnsupportedFeatures.report(UnsupportedFeatures.java:126) at com.oracle.svm.hosted.NativeImageGenerator.runPointsToAnalysis(NativeImageGenerator.java:756) ... 6 more Caused by: java.lang.UnsatisfiedLinkError: 'void sun.lwawt.macosx.CInputMethod.nativeInit()' at java.desktop/sun.lwawt.macosx.CInputMethod.nativeInit(Native Method) at java.desktop/sun.lwawt.macosx.CInputMethod.(CInputMethod.java:87) at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized0(Native Method) at java.base/jdk.internal.misc.Unsafe.ensureClassInitialized(Unsafe.java:1155) at jdk.unsupported/sun.misc.Unsafe.ensureClassInitialized(Unsafe.java:743) at jdk.internal.vm.compiler/org.graalvm.compiler.serviceprovider.GraalUnsafeAccess.ensureClassInitialized(GraalUnsafeAccess.java:77) at com.oracle.svm.hosted.classinitialization.ConfigurableClassInitialization.ensureClassInitialized(ConfigurableClassInitialization.java:178) at com.oracle.svm.hosted.classinitialization.ConfigurableClassInitialization.computeInitKindAndMaybeInitializeClass(ConfigurableClassInitialization.java:648) at com.oracle.svm.hosted.classinitialization.ConfigurableClassInitialization.computeInitKindAndMaybeInitializeClass(ConfigurableClassInitialization.java:131) at com.oracle.svm.hosted.classinitialization.ConfigurableClassInitialization.shouldInitializeAtRuntime(ConfigurableClassInitialization.java:159) at com.oracle.svm.hosted.SVMHost.isInitialized(SVMHost.java:308) at com.oracle.graal.pointsto.meta.AnalysisType.isInitialized(AnalysisType.java:723) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.maybeEagerlyInitialize(BytecodeParser.java:4435) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.genInvokeStatic(BytecodeParser.java:1659) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.genInvokeStatic(BytecodeParser.java:1652) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.processBytecode(BytecodeParser.java:5419) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.iterateBytecodesForBlock(BytecodeParser.java:3477) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.handleBytecodeBlock(BytecodeParser.java:3437) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.processBlock(BytecodeParser.java:3282) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.build(BytecodeParser.java:1145) at jdk.internal.vm.compiler/org.graalvm.compiler.java.BytecodeParser.buildRootMethod(BytecodeParser.java:1030) at jdk.internal.vm.compiler/org.graalvm.compiler.java.GraphBuilderPhase$Instance.run(GraphBuilderPhase.java:84) at com.oracle.svm.hosted.phases.SharedGraphBuilderPhase.run(SharedGraphBuilderPhase.java:81) at jdk.internal.vm.compiler/org.graalvm.compiler.phases.Phase.run(Phase.java:49) at jdk.internal.vm.compiler/org.graalvm.compiler.phases.BasePhase.apply(BasePhase.java:212)
[GitHub] [camel-k] squakez opened a new pull request #2774: doc(install): suggested resource configuration
squakez opened a new pull request #2774: URL: https://github.com/apache/camel-k/pull/2774 @astefanutti I've took the liberty to copy paste your comment from the referred issue Closes #2768 **Release Note** ```release-note doc(install): suggested resource configuration ``` -- 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-kamelets] branch regen_bot updated (254d77b -> d06e999)
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-kamelets.git. from 254d77b Do not use kamelet-reify anymore - Elasticsearch Index Sink Kamelet add cb75633 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet add 423b4c6 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet add d06e999 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet No new revisions were added by this update. Summary of changes: docs/modules/ROOT/pages/elasticsearch-search-source.adoc | 1 - elasticsearch-search-source.kamelet.yaml | 15 --- .../kamelets/elasticsearch-search-source.kamelet.yaml | 15 --- 3 files changed, 8 insertions(+), 23 deletions(-)
[GitHub] [camel-kamelets] oscerd merged pull request #581: Elasticsearch Search Source Kamelet: Do not use kamelet-reify anymore
oscerd merged pull request #581: URL: https://github.com/apache/camel-kamelets/pull/581 -- 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-kamelets] 03/03: Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet
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-kamelets.git commit d06e99956aafd13168becbc0778d6f195dea4c18 Author: Andrea Cosentino AuthorDate: Tue Nov 23 15:11:26 2021 +0100 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet --- docs/modules/ROOT/pages/elasticsearch-search-source.adoc | 1 - 1 file changed, 1 deletion(-) diff --git a/docs/modules/ROOT/pages/elasticsearch-search-source.adoc b/docs/modules/ROOT/pages/elasticsearch-search-source.adoc index f30777b..638ce40 100644 --- a/docs/modules/ROOT/pages/elasticsearch-search-source.adoc +++ b/docs/modules/ROOT/pages/elasticsearch-search-source.adoc @@ -34,7 +34,6 @@ At runtime, the `elasticsearch-search-source` Kamelet relies upon the presence o - camel:core - camel:kamelet - camel:timer -- mvn:org.apache.camel.k:camel-k-kamelet-reify - camel:elasticsearch-rest - camel:gson
[camel-kamelets] 02/03: Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet
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-kamelets.git commit 423b4c6dbf1e13d23a7f8a0533d4e9aae0b37e32 Author: Andrea Cosentino AuthorDate: Tue Nov 23 15:10:37 2021 +0100 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet --- .../kamelets/elasticsearch-search-source.kamelet.yaml | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/library/camel-kamelets/src/main/resources/kamelets/elasticsearch-search-source.kamelet.yaml b/library/camel-kamelets/src/main/resources/kamelets/elasticsearch-search-source.kamelet.yaml index 1cb7856..34b5431 100644 --- a/library/camel-kamelets/src/main/resources/kamelets/elasticsearch-search-source.kamelet.yaml +++ b/library/camel-kamelets/src/main/resources/kamelets/elasticsearch-search-source.kamelet.yaml @@ -11,15 +11,6 @@ metadata: camel.apache.org/provider: "Apache Software Foundation" camel.apache.org/kamelet.icon: "data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KPHN2ZyBlbmFibGUtYmFja2dyb3VuZD0ibmV3IDAgMCA1NTAgMjUwIiB2aWV3Qm94PSI3Nyw4Myw0MTAsOTAiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+CiA8cGF0aCBkPSJtMTc4IDEwMi41aDMxM3Y0MWgtMzEzeiIgZmlsbD0ibm9uZSIvPgogPGcgZW5hYmxlLWJhY2tncm91bmQ9Im5ldyI+CiAgPHBhdGggZD0ibTE5My4zIDEzOS40Yy00IDAtNy4yLTEuMi05LjUtMy41cy0zLjQtNS42LTMuNC05LjdjMC00LjIgMS4xLTcuNiAzLjItMTBzNS0zLjYgOC43LTMuNmMzLjQgMCA2Lj [...] spec: - sources: -- content: |- -public class ESWorkaround extends org.apache.camel.builder.RouteBuilder { - @Override - public void configure() throws Exception { -getContext().setAutowiredEnabled(false); - } -} - name: ESWorkaround.java definition: title: "ElasticSearch Index Source" description: |- @@ -80,10 +71,12 @@ spec: - "camel:core" - "camel:kamelet" - "camel:timer" - - "mvn:org.apache.camel.k:camel-k-kamelet-reify" - "camel:elasticsearch-rest" - "camel:gson" flow: +beans: + - name: local-es +type: "#class:org.apache.camel.component.elasticsearch.ElasticsearchComponent" from: uri: timer:tick parameters: @@ -95,7 +88,7 @@ spec: name: "Content-Type" constant: "text/plain" - to: -uri: "kamelet-reify:elasticsearch-rest:{{clusterName}}" +uri: "{{local-es}}:{{clusterName}}" parameters: operation: "SEARCH" indexName: "{{indexName}}"
[camel-kamelets] branch main updated (254d77b -> d06e999)
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-kamelets.git. from 254d77b Do not use kamelet-reify anymore - Elasticsearch Index Sink Kamelet new cb75633 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet new 423b4c6 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet new d06e999 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet The 3 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/modules/ROOT/pages/elasticsearch-search-source.adoc | 1 - elasticsearch-search-source.kamelet.yaml | 15 --- .../kamelets/elasticsearch-search-source.kamelet.yaml | 15 --- 3 files changed, 8 insertions(+), 23 deletions(-)
[camel-kamelets] 01/03: Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet
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-kamelets.git commit cb75633085511b3b89eef5456741eee5cf92720c Author: Andrea Cosentino AuthorDate: Tue Nov 23 15:09:32 2021 +0100 Do not use kamelet-reify anymore - Elasticsearch Search Source Kamelet --- elasticsearch-search-source.kamelet.yaml | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/elasticsearch-search-source.kamelet.yaml b/elasticsearch-search-source.kamelet.yaml index 1cb7856..34b5431 100644 --- a/elasticsearch-search-source.kamelet.yaml +++ b/elasticsearch-search-source.kamelet.yaml @@ -11,15 +11,6 @@ metadata: camel.apache.org/provider: "Apache Software Foundation" camel.apache.org/kamelet.icon: "data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KPHN2ZyBlbmFibGUtYmFja2dyb3VuZD0ibmV3IDAgMCA1NTAgMjUwIiB2aWV3Qm94PSI3Nyw4Myw0MTAsOTAiIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3N2ZyI+CiA8cGF0aCBkPSJtMTc4IDEwMi41aDMxM3Y0MWgtMzEzeiIgZmlsbD0ibm9uZSIvPgogPGcgZW5hYmxlLWJhY2tncm91bmQ9Im5ldyI+CiAgPHBhdGggZD0ibTE5My4zIDEzOS40Yy00IDAtNy4yLTEuMi05LjUtMy41cy0zLjQtNS42LTMuNC05LjdjMC00LjIgMS4xLTcuNiAzLjItMTBzNS0zLjYgOC43LTMuNmMzLjQgMCA2Lj [...] spec: - sources: -- content: |- -public class ESWorkaround extends org.apache.camel.builder.RouteBuilder { - @Override - public void configure() throws Exception { -getContext().setAutowiredEnabled(false); - } -} - name: ESWorkaround.java definition: title: "ElasticSearch Index Source" description: |- @@ -80,10 +71,12 @@ spec: - "camel:core" - "camel:kamelet" - "camel:timer" - - "mvn:org.apache.camel.k:camel-k-kamelet-reify" - "camel:elasticsearch-rest" - "camel:gson" flow: +beans: + - name: local-es +type: "#class:org.apache.camel.component.elasticsearch.ElasticsearchComponent" from: uri: timer:tick parameters: @@ -95,7 +88,7 @@ spec: name: "Content-Type" constant: "text/plain" - to: -uri: "kamelet-reify:elasticsearch-rest:{{clusterName}}" +uri: "{{local-es}}:{{clusterName}}" parameters: operation: "SEARCH" indexName: "{{indexName}}"
[GitHub] [camel-kamelets] oscerd opened a new pull request #581: Elasticsearch Search Source Kamelet: Do not use kamelet-reify anymore
oscerd opened a new pull request #581: URL: https://github.com/apache/camel-kamelets/pull/581 Related to #554 -- 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