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

2018-09-10 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 end-

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

2018-09-10 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-10 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 dif

[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 g

[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 abov

[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 g

[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 g

[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-420157152 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/Camel-k-pr/3/ --

[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 abov

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

2018-09-10 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] 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-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 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 abov

[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 g

[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 G

[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 t

[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 g

[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 abov

[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 g

[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 abov

[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 g

[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 abov

[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 abov

[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 above

[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 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 t

[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] 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 g

[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.com/operator-framework/operator-sdk@v0.0.0-20180907071707-1f054f148

[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 (f

[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 use

Re: R: [camel] branch master updated: Revert "Upgrade TestContainers to version 1.9.0", because "org.testcontainers:testcontainers:jar:1.9.0" is not in maven central yet.

2018-09-10 Thread Pascal Schumacher
No problem, I just wanted to fix the build on builds.apache.org: e.g.: https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/master/623/console org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal on project camel-testcontainers: Could not resolve depend

R: [camel] branch master updated: Revert "Upgrade TestContainers to version 1.9.0", because "org.testcontainers:testcontainers:jar:1.9.0" is not in maven central yet.

2018-09-10 Thread Andrea Cosentino
My fault. Probably it comes from a different repo. Sorry. I tested with camel-nats. I'll double check tomorrow Il lun, 10 set, 2018 alle 23:07, pascalschumac...@apache.org ha scritto: This is an automated email from the ASF dual-hosted git repository. pascalschumacher pushed a commit to

[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] camel pull request #2516: Replace string concatenation in logging with place...

2018-09-10 Thread PascalSchumacher
GitHub user PascalSchumacher opened a pull request: https://github.com/apache/camel/pull/2516 Replace string concatenation in logging with placeholders. Done by regex search and replace. Then reviewed by hand. I excluded test classes because the changes is already big enough

[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 messa

[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] 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] 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 Se

[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 #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 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 source

[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 respond

[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 fe

[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 -update`

[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] 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 abo

[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] 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] 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 t

[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 bel

[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 mes

[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 th

[GitHub] camel pull request #2515: CAMEL-12733. Check if directory exists via ls, so ...

2018-09-10 Thread evilball
GitHub user evilball opened a pull request: https://github.com/apache/camel/pull/2515 CAMEL-12733. Check if directory exists via ls, so we don't need to re… …turn back to original dir later. You can merge this pull request into a Git repository by running: $ git pull https:

[GitHub] lburgazzoli closed pull request #6: Adding install and get commands

2018-09-10 Thread GitBox
lburgazzoli closed pull request #6: Adding install and get commands URL: https://github.com/apache/camel-k/pull/6 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 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 standard

[GitHub] camel pull request #2514: Update aws-kinesis-component.adoc

2018-09-10 Thread oscerd
Github user oscerd closed the pull request at: https://github.com/apache/camel/pull/2514 ---

[GitHub] camel pull request #2514: Update aws-kinesis-component.adoc

2018-09-10 Thread lebenlechzer
GitHub user lebenlechzer opened a pull request: https://github.com/apache/camel/pull/2514 Update aws-kinesis-component.adoc The AWS Kinesis Producer actually requires a byte[] instead of a ByteBuffer. You can merge this pull request into a Git repository by running: $ git pull