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 user oscerd closed the pull request at:
https://github.com/apache/camel/pull/2513
---
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 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
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 user oscerd closed the pull request at:
https://github.com/apache/camel/pull/2514
---
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
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 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
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
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.
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?
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
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 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
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
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
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
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
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
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
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:
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
50 matches
Mail list logo