[GitHub] nicolaferraro closed pull request #2: Add bash completion

2018-09-07 Thread GitBox
nicolaferraro closed pull request #2: Add bash completion URL: https://github.com/apache/camel-k/pull/2 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] nicolaferraro closed pull request #4: Minor refactor of client cmd

2018-09-10 Thread GitBox
nicolaferraro closed pull request #4: Minor refactor of client cmd URL: https://github.com/apache/camel-k/pull/4 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] lburgazzoli closed pull request #5: Added Property placeholder for commons-lang3 version

2018-09-10 Thread GitBox
lburgazzoli closed pull request #5: Added Property placeholder for commons-lang3 version URL: https://github.com/apache/camel-k/pull/5 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] oscerd opened a new pull request #5: Added Property placeholder for commons-lang3 version

2018-09-10 Thread GitBox
oscerd opened a new pull request #5: Added Property placeholder for commons-lang3 version URL: https://github.com/apache/camel-k/pull/5 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] nicolaferraro closed pull request #3: swith to full in-memory java compiler and some minro cleanup

2018-09-09 Thread GitBox
nicolaferraro closed pull request #3: swith to full in-memory java compiler and some minro cleanup URL: https://github.com/apache/camel-k/pull/3 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] nicolaferraro opened a new pull request #6: Adding install and get commands

2018-09-10 Thread GitBox
nicolaferraro opened a new pull request #6: Adding install and get commands URL: https://github.com/apache/camel-k/pull/6 This adds the "kamel install" command. It tells you to login as admin the first time you install. Once you do the first install, you can install it also with

[GitHub] dmvolod opened a new pull request #8: Add go get dependecies before build

2018-09-10 Thread GitBox
dmvolod opened a new pull request #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8 Also need to add notes to the docs about $GOPATH env variable be set before make This is an automated

[GitHub] nicolaferraro opened a new pull request #7: Fix operator role to have lower privileges than the standard user and standardize command options

2018-09-10 Thread GitBox
nicolaferraro opened a new pull request #7: Fix operator role to have lower privileges than the standard user and standardize command options URL: https://github.com/apache/camel-k/pull/7 This is an automated message from

[GitHub] nicolaferraro commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
nicolaferraro commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419953442 We can change it safely to use https. Until I port the change to the upstream repo.

[GitHub] dmvolod edited a comment on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
dmvolod edited a comment on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-419971680 @oscerd , @nicolaferraro what do you think about to add Gopkg.lock to .gitignore as it autogenerated?

[GitHub] dmvolod commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
dmvolod commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419948285 @nicolaferraro another one issue: I didn't success with dep ensure and source = "g...@github.com:nicolaferraro/operator-sdk.git". When I changed to

[GitHub] nicolaferraro commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
nicolaferraro commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419927393 Thank @dmvolod for joining :+1: We're using `dep` as dependency management mechanism, that works differently from "go get" and actually the

[GitHub] nicolaferraro commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
nicolaferraro commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-419945536 What version of `dep` are you using? Other than the hash, also the syntax seems a bit different sometimes (e.g. trailing comma). Did you run `dep ensure

[GitHub] nicolaferraro closed pull request #10: Use https instead of git for github.com/nicolaferraro/operator-sdk.git source

2018-09-10 Thread GitBox
nicolaferraro closed pull request #10: Use https instead of git for github.com/nicolaferraro/operator-sdk.git source URL: https://github.com/apache/camel-k/pull/10 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] nicolaferraro commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
nicolaferraro commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419928458 See https://github.com/golang/dep, we'll evaluate later a switch to vgo. This is an

[GitHub] oscerd opened a new pull request #9: Regen gopkg.lock

2018-09-10 Thread GitBox
oscerd opened a new pull request #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
oscerd commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-419945939 ``` dep: version : v0.5.0 build date : 2018-07-26 git hash: 224a564 go version : go1.10.3 go compiler : gc platform: linux/amd64

[GitHub] oscerd commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
oscerd commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-419946475 The Pr is updated with dep ensure -update This is an automated message from the Apache Git Service. To

[GitHub] dmvolod opened a new pull request #10: Use https instead of git for github.com/nicolaferraro/operator-sdk.git source

2018-09-10 Thread GitBox
dmvolod opened a new pull request #10: Use https instead of git for github.com/nicolaferraro/operator-sdk.git source URL: https://github.com/apache/camel-k/pull/10 …t source This is an automated message from the Apache Git

[GitHub] dmvolod closed pull request #8: Add go get dependecies before build

2018-09-10 Thread GitBox
dmvolod closed pull request #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] dmvolod commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
dmvolod commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419941618 @nicolaferraro thanks for explanation. Undestood, why camel-k is using dep rather than 'go get' and it's realy slow. I will close this PR and will try to

[GitHub] dmvolod commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
dmvolod commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-419971680 @oscerd , @nicolaferraro what do you think about to add Gopkg.lock to .gitignore? This is an automated

[GitHub] lburgazzoli closed pull request #7: Fix operator role to have lower privileges than the standard user and standardize command options

2018-09-10 Thread GitBox
lburgazzoli closed pull request #7: Fix operator role to have lower privileges than the standard user and standardize command options URL: https://github.com/apache/camel-k/pull/7 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] nicolaferraro commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
nicolaferraro commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419944713 Yes, last time I tried vgo there was lack of support in IDE (lot of compilation errors in intellij). If this changed I'm happy to switch.

[GitHub] dmvolod commented on issue #8: Add go get dependecies before build

2018-09-10 Thread GitBox
dmvolod commented on issue #8: Add go get dependecies before build URL: https://github.com/apache/camel-k/pull/8#issuecomment-419954646 Ok, I will change. Looks like we can use forked repo every time we need to change something faster than upstream.

[GitHub] nicolaferraro commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
nicolaferraro commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-420062146 No @dmvolod, the lock file establishes the exact version of the libraries that the project is using, while the toml file describes high level constraints on them that

[GitHub] valdar opened a new pull request #11: Added build documentation.

2018-09-10 Thread GitBox
valdar opened a new pull request #11: Added build documentation. URL: https://github.com/apache/camel-k/pull/11 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] onderson opened a new pull request #12: Update readme fix typos

2018-09-10 Thread GitBox
onderson opened a new pull request #12: Update readme fix typos URL: https://github.com/apache/camel-k/pull/12 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] nicolaferraro closed pull request #11: Added build documentation.

2018-09-10 Thread GitBox
nicolaferraro closed pull request #11: Added build documentation. URL: https://github.com/apache/camel-k/pull/11 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] lburgazzoli commented on issue #13: Add go 1.11 module files so that you can build outside the GOPATH.

2018-09-10 Thread GitBox
lburgazzoli commented on issue #13: Add go 1.11 module files so that you can build outside the GOPATH. URL: https://github.com/apache/camel-k/pull/13#issuecomment-420144478 In my env it fails with: ``` go:

[GitHub] dmvolod commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
dmvolod commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-420149199 Thanks, @nicolaferraro . But update of the Gopkg.lock look like related to the version of the dep. When I used a dep from fedora package all was good, but installed via go

[GitHub] oscerd closed pull request #9: Regen gopkg.lock

2018-09-10 Thread GitBox
oscerd closed pull request #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] oscerd closed pull request #12: Update readme fix typos

2018-09-10 Thread GitBox
oscerd closed pull request #12: Update readme fix typos URL: https://github.com/apache/camel-k/pull/12 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd commented on issue #9: Regen gopkg.lock

2018-09-10 Thread GitBox
oscerd commented on issue #9: Regen gopkg.lock URL: https://github.com/apache/camel-k/pull/9#issuecomment-420151007 This has been already updated. Closing. This is an automated message from the Apache Git Service. To respond

[GitHub] nicolaferraro closed pull request #1: Add support for .js, .java and classes

2018-09-07 Thread GitBox
nicolaferraro closed pull request #1: Add support for .js, .java and classes URL: https://github.com/apache/camel-k/pull/1 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] oscerd closed pull request #15: Dep version Readme

2018-09-10 Thread GitBox
oscerd closed pull request #15: Dep version Readme URL: https://github.com/apache/camel-k/pull/15 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd commented on issue #16: README update

2018-09-10 Thread GitBox
oscerd commented on issue #16: README update URL: https://github.com/apache/camel-k/pull/16#issuecomment-420156153 ok to test This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] oscerd closed pull request #16: README update

2018-09-10 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] oscerd opened a new pull request #16: README update

2018-09-10 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] asfgit commented on issue #16: README update

2018-09-10 Thread GitBox
asfgit commented on issue #16: README update URL: https://github.com/apache/camel-k/pull/16#issuecomment-420156050 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond

[GitHub] asfgit commented on issue #16: README update

2018-09-11 Thread GitBox
asfgit commented on issue #16: README update URL: https://github.com/apache/camel-k/pull/16#issuecomment-420156835 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Camel-k-pr/1/

[GitHub] oscerd closed pull request #18: Installation and run works

2018-09-11 Thread GitBox
oscerd closed pull request #18: Installation and run works URL: https://github.com/apache/camel-k/pull/18 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] nicolaferraro opened a new pull request #22: CAMEL-12789: complete install command

2018-09-11 Thread GitBox
nicolaferraro opened a new pull request #22: CAMEL-12789: complete install command URL: https://github.com/apache/camel-k/pull/22 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] oscerd opened a new pull request #16: README update

2018-09-10 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd closed pull request #16: README update

2018-09-10 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] nicolaferraro opened a new pull request #18: Installation and run works

2018-09-11 Thread GitBox
nicolaferraro opened a new pull request #18: Installation and run works URL: https://github.com/apache/camel-k/pull/18 This embed java artifacts in the operator image, so that it can execute embedded builds and s2i push (other strategies in the future). Now Camel K can be tested

[GitHub] dmvolod opened a new pull request #21: Add labels to all resources

2018-09-11 Thread GitBox
dmvolod opened a new pull request #21: Add labels to all resources URL: https://github.com/apache/camel-k/pull/21 Add labels to resources. It will be useful if we will add an 'uninstall' option in the future and remove or update can easy with one command, i.e. oc delete

[GitHub] oscerd opened a new pull request #16: README update

2018-09-11 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd closed pull request #16: README update

2018-09-11 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] nicolaferraro commented on issue #21: Add labels to all resources

2018-09-11 Thread GitBox
nicolaferraro commented on issue #21: Add labels to all resources URL: https://github.com/apache/camel-k/pull/21#issuecomment-420217342 Yes, sounds good! This is an automated message from the Apache Git Service. To respond to

[GitHub] oscerd opened a new pull request #15: Dep version Readme

2018-09-10 Thread GitBox
oscerd opened a new pull request #15: Dep version Readme URL: https://github.com/apache/camel-k/pull/15 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] oscerd opened a new pull request #16: README update

2018-09-10 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd closed pull request #16: README update

2018-09-10 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] oscerd opened a new pull request #16: README update

2018-09-10 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd closed pull request #16: README update

2018-09-11 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] oscerd opened a new pull request #16: README update

2018-09-11 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] oscerd closed pull request #16: README update

2018-09-11 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] nicolaferraro closed pull request #14: add clean and dep target to Makefile

2018-09-11 Thread GitBox
nicolaferraro closed pull request #14: add clean and dep target to Makefile URL: https://github.com/apache/camel-k/pull/14 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] nicolaferraro closed pull request #17: t code

2018-09-11 Thread GitBox
nicolaferraro closed pull request #17: t code URL: https://github.com/apache/camel-k/pull/17 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the

[GitHub] lburgazzoli closed pull request #20: CAMEL-12798: Add getting started guide

2018-09-11 Thread GitBox
lburgazzoli closed pull request #20: CAMEL-12798: Add getting started guide URL: https://github.com/apache/camel-k/pull/20 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] oscerd closed pull request #16: README update

2018-09-11 Thread GitBox
oscerd closed pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] oscerd opened a new pull request #16: README update

2018-09-11 Thread GitBox
oscerd opened a new pull request #16: README update URL: https://github.com/apache/camel-k/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] asfgit commented on issue #16: README update

2018-09-11 Thread GitBox
asfgit commented on issue #16: README update URL: https://github.com/apache/camel-k/pull/16#issuecomment-420157152 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Camel-k-pr/3/

[GitHub] asfgit commented on issue #17: t code

2018-09-11 Thread GitBox
asfgit commented on issue #17: t code URL: https://github.com/apache/camel-k/pull/17#issuecomment-420156916 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Camel-k-pr/2/

[GitHub] nicolaferraro closed pull request #19: runtime: add support for groovy

2018-09-11 Thread GitBox
nicolaferraro closed pull request #19: runtime: add support for groovy URL: https://github.com/apache/camel-k/pull/19 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] nicolaferraro opened a new pull request #20: CAMEL-12798: Add getting started guide

2018-09-11 Thread GitBox
nicolaferraro opened a new pull request #20: CAMEL-12798: Add getting started guide URL: https://github.com/apache/camel-k/pull/20 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] nicolaferraro closed pull request #21: Add labels to all resources

2018-09-11 Thread GitBox
nicolaferraro closed pull request #21: Add labels to all resources URL: https://github.com/apache/camel-k/pull/21 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] nicolaferraro commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216656331 ## File path: deploy/operator-service.yaml ## @@ -0,0 +1,19 @@ +apiVersion: v1 Review comment: I mean the service

[GitHub] nicolaferraro commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216657892 ## File path: deploy/resources.go ## @@ -86,7 +86,6 @@ spec: metadata: labels: name: camel-k-operator -

[GitHub] nicolaferraro commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216656035 ## File path: deploy/resources.go ## @@ -86,7 +86,6 @@ spec: metadata: labels: name: camel-k-operator -

[GitHub] nicolaferraro commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216646302 ## File path: README.md ## @@ -46,6 +46,12 @@ kamel install This will configure the cluster with the Camel K custom

[GitHub] nicolaferraro commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216646603 ## File path: deploy/operator-service.yaml ## @@ -0,0 +1,19 @@ +apiVersion: v1 Review comment: Ok, is this required for

[GitHub] dmvolod commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
dmvolod commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216676928 ## File path: deploy/operator-service.yaml ## @@ -0,0 +1,19 @@ +apiVersion: v1 Review comment: Yes, this is for metrics. It

[GitHub] dmvolod commented on a change in pull request #23: Add label to service

2018-09-11 Thread GitBox
dmvolod commented on a change in pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23#discussion_r216677017 ## File path: README.md ## @@ -46,6 +46,12 @@ kamel install This will configure the cluster with the Camel K custom resource

[GitHub] dmvolod opened a new pull request #23: Add label to service

2018-09-11 Thread GitBox
dmvolod opened a new pull request #23: Add label to service URL: https://github.com/apache/camel-k/pull/23 Missing to add label to service. This requires to separate service creation in single file operator-service.yaml.

[GitHub] valdar opened a new pull request #25: Changed log message in Sample.java in order to really rocks!

2018-09-11 Thread GitBox
valdar opened a new pull request #25: Changed log message in Sample.java in order to really rocks! URL: https://github.com/apache/camel-k/pull/25 This is an automated message from the Apache Git Service. To respond to the

[GitHub] lburgazzoli closed pull request #22: CAMEL-12789: complete install command

2018-09-11 Thread GitBox
lburgazzoli closed pull request #22: CAMEL-12789: complete install command URL: https://github.com/apache/camel-k/pull/22 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] nicolaferraro commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216658402 ## File path: pkg/apis/camel/v1alpha1/types.go ## @@ -36,6 +36,7 @@ type Integration struct {

[GitHub] nicolaferraro commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216658930 ## File path: pkg/apis/camel/v1alpha1/types.go ## @@ -61,3 +62,53 @@ const ( IntegrationPhaseRunning

[GitHub] nicolaferraro commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216660873 ## File path: pkg/client/cmd/context_edit.go ## @@ -0,0 +1,101 @@ +/* +Licensed to the Apache Software

[GitHub] nicolaferraro commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216661986 ## File path: pkg/stub/action/context/monitor.go ## @@ -0,0 +1,46 @@ +/* +Licensed to the Apache Software

[GitHub] nicolaferraro commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
nicolaferraro commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216658126 ## File path: Gopkg.lock ## @@ -2,230 +2,179 @@ [[projects]] - digest =

[GitHub] lburgazzoli commented on a change in pull request #24: initial code for integration context

2018-09-11 Thread GitBox
lburgazzoli commented on a change in pull request #24: initial code for integration context URL: https://github.com/apache/camel-k/pull/24#discussion_r216677379 ## File path: pkg/apis/camel/v1alpha1/types.go ## @@ -61,3 +62,53 @@ const ( IntegrationPhaseRunning

[GitHub] lburgazzoli closed pull request #25: Changed log message in Sample.java in order to really rocks!

2018-09-11 Thread GitBox
lburgazzoli closed pull request #25: Changed log message in Sample.java in order to really rocks! URL: https://github.com/apache/camel-k/pull/25 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] nicolaferraro commented on issue #40: Support set integration name for kamel run

2018-09-11 Thread GitBox
nicolaferraro commented on issue #40: Support set integration name for kamel run URL: https://github.com/apache/camel-k/issues/40#issuecomment-420441434 I'll add also that the current mapping is trivial, e.g. `MyIntegration.java` -> `myintegration`. A better name would be

[GitHub] nicolaferraro opened a new issue #37: Improve user guide

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #37: Improve user guide URL: https://github.com/apache/camel-k/issues/37 We should document what options and scenarios are available for a end user (from running "kamel run Source.java" on)

[GitHub] lburgazzoli commented on issue #40: Support set integration name for kamel run

2018-09-11 Thread GitBox
lburgazzoli commented on issue #40: Support set integration name for kamel run URL: https://github.com/apache/camel-k/issues/40#issuecomment-420442166 Thre is also a problem when the file is located on a different directory than the current one i,e `runtime/examples/Sample.java` becomes

[GitHub] nicolaferraro opened a new issue #43: Unable to reinstall the operator

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #43: Unable to reinstall the operator URL: https://github.com/apache/camel-k/issues/43 If trying to execute `kamel install` twice, the second installation fails for a issue while trying to update the service: ``` Service "camel-k-operator" is

[GitHub] nicolaferraro opened a new issue #36: Document architecture

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #36: Document architecture URL: https://github.com/apache/camel-k/issues/36 We should document the architecture: - CRDs - Operator and client, how they interact - Build modes - Publishing modes - The deployment flow

[GitHub] nicolaferraro opened a new issue #42: Add "make install-minishift" command

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #42: Add "make install-minishift" command URL: https://github.com/apache/camel-k/issues/42 As explained in the readme, the procedure to do a local build and deploy is complex. We should add a `make install-minishift` command to simplify it (and update

[GitHub] oscerd commented on issue #44: How to know a Component status at a time

2018-09-11 Thread GitBox
oscerd commented on issue #44: How to know a Component status at a time URL: https://github.com/apache/camel-k/issues/44#issuecomment-420512774 Please ask on the users mailing list users at camel.apache.org This is an

[GitHub] oscerd closed issue #44: How to know a Component status at a time

2018-09-11 Thread GitBox
oscerd closed issue #44: How to know a Component status at a time URL: https://github.com/apache/camel-k/issues/44 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] oscerd commented on issue #40: Support set integration name for kamel run

2018-09-11 Thread GitBox
oscerd commented on issue #40: Support set integration name for kamel run URL: https://github.com/apache/camel-k/issues/40#issuecomment-420512966 Sure This is an automated message from the Apache Git Service. To respond to

[GitHub] lburgazzoli opened a new issue #45: Support for multiple integration definitions

2018-09-11 Thread GitBox
lburgazzoli opened a new issue #45: Support for multiple integration definitions URL: https://github.com/apache/camel-k/issues/45 We can now deploy our routes by defining them in a single file and the using the `kamel` command like: kamel run MyRoutes.js But it would be

[GitHub] nicolaferraro opened a new issue #38: Add uninstall command

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #38: Add uninstall command URL: https://github.com/apache/camel-k/issues/38 It's would be nice to have an uninstall command which will remove all camel-k artifacts (cluster-wide and project related). This will be useful for development and update

[GitHub] lburgazzoli edited a comment on issue #40: Support set integration name for kamel run

2018-09-11 Thread GitBox
lburgazzoli edited a comment on issue #40: Support set integration name for kamel run URL: https://github.com/apache/camel-k/issues/40#issuecomment-420442166 Thre is also a problem when the file is located on a different directory than the current one i.e. `runtime/examples/Sample.java`

[GitHub] nicolaferraro closed pull request #39: add support for non java integrations

2018-09-11 Thread GitBox
nicolaferraro closed pull request #39: add support for non java integrations URL: https://github.com/apache/camel-k/pull/39 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] nicolaferraro opened a new issue #41: Cannot deploy integration if not in the "kamel" package

2018-09-11 Thread GitBox
nicolaferraro opened a new issue #41: Cannot deploy integration if not in the "kamel" package URL: https://github.com/apache/camel-k/issues/41 Integrations not using the "kamel" package (see `Sample.java` in the repo root) cannot be run. The reason is that the package is in some way

[GitHub] kyleabcha opened a new issue #44: How to know a Component status at a time

2018-09-11 Thread GitBox
kyleabcha opened a new issue #44: How to know a Component status at a time URL: https://github.com/apache/camel-k/issues/44 Hi, I am having difficulty how to differ when a Component (DefaultComponent) is deployed (NEW) or when it is just restarted or undeployed (REMOVED completely).

[GitHub] nicolaferraro commented on issue #30: Manage dependencies

2018-09-11 Thread GitBox
nicolaferraro commented on issue #30: Manage dependencies URL: https://github.com/apache/camel-k/issues/30#issuecomment-420440725 I think this is a problem we have in common with FaaS platforms, we should investigate what they do to add dependencies to a function. I'm thinking that

  1   2   3   4   5   6   7   8   9   10   >