[GitHub] [camel-k-runtime] lburgazzoli merged pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli merged pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-k-runtime] lburgazzoli closed issue #80: Support full yaml dsl

2019-06-20 Thread GitBox
lburgazzoli closed issue #80: Support full yaml dsl URL: https://github.com/apache/camel-k-runtime/issues/80 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [VOTE] Release Apache Camel 2.24.1

2019-06-20 Thread Francois Papon
+1 (non-binding) Thanks! regards, François fpa...@apache.org Le 20/06/2019 à 23:04, Gregor Zurowski a écrit : > Hi Everyone: > > This is a vote to release Apache Camel 2.24.1, the first patch release > for the camel-2.24.x branch with 22 improvements and bug fixes. > > Release notes: >

Re: [VOTE] Release Apache Camel 2.24.1

2019-06-20 Thread Claus Ibsen
+1 (binding) On Thu, Jun 20, 2019 at 9:05 PM Gregor Zurowski wrote: > > Hi Everyone: > > This is a vote to release Apache Camel 2.24.1, the first patch release > for the camel-2.24.x branch with 22 improvements and bug fixes. > > Release notes: >

Re: [VOTE] Release Apache Camel 2.24.1

2019-06-20 Thread Freeman Fang
+1 Thanks! - Freeman(Yue) Fang Red Hat, Inc. > On Jun 20, 2019, at 3:04 PM, Gregor Zurowski wrote: > > Hi Everyone: > > This is a vote to release Apache Camel 2.24.1, the first patch release > for the camel-2.24.x branch with 22 improvements and bug fixes. > > Release

[GitHub] [camel-k-runtime] valdar commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
valdar commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295981173 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/model/Definitions.java ## @@ -0,0 +1,106 @@

Re: [VOTE] Release Apache Camel 2.24.1

2019-06-20 Thread Alex Dettinger
+1 (binding) Thanks Gregor, Alex On Thu, Jun 20, 2019 at 9:05 PM Gregor Zurowski wrote: > Hi Everyone: > > This is a vote to release Apache Camel 2.24.1, the first patch release > for the camel-2.24.x branch with 22 improvements and bug fixes. > > Release notes: >

[GitHub] [camel-k-runtime] valdar commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
valdar commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295975560 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/model/Definitions.java ## @@ -0,0 +1,106 @@

[VOTE] Release Apache Camel 2.24.1

2019-06-20 Thread Gregor Zurowski
Hi Everyone: This is a vote to release Apache Camel 2.24.1, the first patch release for the camel-2.24.x branch with 22 improvements and bug fixes. Release notes: https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12345495=12311211 Staging repository:

[GitHub] [camel-k-runtime] davsclaus commented on issue #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on issue #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#issuecomment-504047067 Yeah StepParser is better than Converter This is an automated message from the Apache

[GitHub] [camel-k-runtime] lburgazzoli commented on issue #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli commented on issue #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#issuecomment-504045425 ok, working on renaming things to StepParser @valdar @davsclaus do you mind doing a second review ?

[GitHub] [camel-k-runtime] lburgazzoli commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295826949 ## File path: camel-k-runtime-yaml/README.adoc ## @@ -0,0 +1,204 @@ +YAML DSL + + +This artifact provides a

[GitHub] [camel-k-runtime] lburgazzoli commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295825199 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/converter/ChoiceConverter.java ## @@ -0,0

[GitHub] [camel-k-runtime] lburgazzoli commented on issue #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli commented on issue #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#issuecomment-504041294 @davsclaus @valdar yeah, the things have originally started as plain jackson converters so named according but it turned out it was not possible to

[GitHub] [camel-k-runtime] valdar edited a comment on issue #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
valdar edited a comment on issue #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#issuecomment-504038971 +1 on the converter naming, some proposals: - StepParser - StepHandler This

[GitHub] [camel-k-runtime] valdar commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
valdar commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295817180 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/model/Definitions.java ## @@ -0,0 +1,172 @@

[GitHub] [camel-k-runtime] davsclaus commented on issue #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on issue #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#issuecomment-504035662 Looks really good. Only found cosmetic issues in the comments. However I do think we should think about the naming of `converter` as it reminds

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295799257 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/Yaml.java ## @@ -0,0 +1,82 @@ +/** + *

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295799726 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/Yaml.java ## @@ -0,0 +1,82 @@ +/** + *

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295798644 ## File path: camel-k-runtime-yaml/README.adoc ## @@ -0,0 +1,204 @@ +YAML DSL + + +This artifact provides a

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295798355 ## File path: camel-k-runtime-yaml/README.adoc ## @@ -0,0 +1,204 @@ +YAML DSL + + +This artifact provides a

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295800824 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/converter/ChoiceConverter.java ## @@ -0,0

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295798105 ## File path: camel-k-runtime-yaml/README.adoc ## @@ -0,0 +1,204 @@ +YAML DSL + + +This artifact provides a

[GitHub] [camel-k-runtime] davsclaus commented on a change in pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
davsclaus commented on a change in pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84#discussion_r295800538 ## File path: camel-k-runtime-yaml/src/main/java/org/apache/camel/k/yaml/YamlRoutesLoader.java ## @@ -0,0 +1,103 @@

[GitHub] [camel-k] nicolaferraro merged pull request #757: Service trait: confusing behavior

2019-06-20 Thread GitBox
nicolaferraro merged pull request #757: Service trait: confusing behavior URL: https://github.com/apache/camel-k/pull/757 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-k] nicolaferraro closed issue #753: Service trait: confusing behavior

2019-06-20 Thread GitBox
nicolaferraro closed issue #753: Service trait: confusing behavior URL: https://github.com/apache/camel-k/issues/753 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [camel-k-runtime] lburgazzoli merged pull request #85: chore(deps): update kotlin to v1.3.40

2019-06-20 Thread GitBox
lburgazzoli merged pull request #85: chore(deps): update kotlin to v1.3.40 URL: https://github.com/apache/camel-k-runtime/pull/85 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [camel-k] nicolaferraro opened a new pull request #758: Add e2e tests

2019-06-20 Thread GitBox
nicolaferraro opened a new pull request #758: Add e2e tests URL: https://github.com/apache/camel-k/pull/758 This is built on top of #756 Fixes #693 This adds a series of e2e tests that should verify (if travis works) the most important functionalities, e.g.: - installation

[GitHub] [camel-k] lburgazzoli opened a new pull request #757: Service trait: confusing behavior

2019-06-20 Thread GitBox
lburgazzoli opened a new pull request #757: Service trait: confusing behavior URL: https://github.com/apache/camel-k/pull/757 Fixes #753 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [camel-k-runtime] lburgazzoli opened a new pull request #84: Support full yaml dsl #80

2019-06-20 Thread GitBox
lburgazzoli opened a new pull request #84: Support full yaml dsl #80 URL: https://github.com/apache/camel-k-runtime/pull/84 Fixes #80 This is the first iteration for an YAMLesque DSL so that you can define routes as: ```yaml - from: uri: "direct:start"

[GitHub] [camel-k] oscerd closed issue #745: Add a --global install option

2019-06-20 Thread GitBox
oscerd closed issue #745: Add a --global install option URL: https://github.com/apache/camel-k/issues/745 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [camel-k] oscerd merged pull request #752: feat: Add a --global install option

2019-06-20 Thread GitBox
oscerd merged pull request #752: feat: Add a --global install option URL: https://github.com/apache/camel-k/pull/752 This is an automated message from the Apache Git Service. To respond to the message, please log on to