You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2020/11/23 11:53:06 UTC
[GitHub] [camel-k] bouskaJ opened a new pull request #1822: Split tests by language
bouskaJ opened a new pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822
<!-- Description -->
<!--
Enter your extended release note in the below block. If the PR requires
additional action from users switching to the new release, include the string
"action required". If no release note is required, write "NONE".
You can (optionally) mark this PR with labels "kind/bug" or "kind/feature" to make sure
the text is added to the right section of the release notes.
-->
**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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] bouskaJ commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-732993569
It seems that `go test -timeout 60m -v ./e2e/common/... -tags=integration` does not print it's output in progress but it prits everything at the end of the test execution.
Travis CI seems to have an issue with it:
```
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated
```
Should I change the command to `go test -timeout 60m -v ./e2e/common -tags=integration && go test -timeout 60m -v ./e2e/common/languages -tags=integration`? What do you thing about it?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] nicolaferraro merged pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
nicolaferraro merged pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] bouskaJ commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-734751716
@nicolaferraro I am done. Can you recheck/merge this PR?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] bouskaJ commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-734145372
Damn! The https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received is not working in the script that runs the integration test (https://travis-ci.com/github/apache/camel-k/builds/204584739). Then I would go with ` go test -timeout 60m -v ./e2e/common -tags=integration && go test -timeout 60m -v ./e2e/common/languages -tags=integration`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] bouskaJ commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-733800268
@nicolaferraro WDYT?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] bouskaJ commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
bouskaJ commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-734132884
> > @nicolaferraro WDYT?
>
> I was thinking about this but did not have time to look carefully.
> I remember there was an issue in travis that happened when an execution did not produce any output for a long time. But don't know why this change triggers that issue.
>
> This may be something completely silly, but I remember some issue about the `...` notation here: https://github.com/apache/camel-k/pull/1822/files#diff-9aecbcc15d64c3e7a146ee5776a35e65b45fb72461e370c4e295dd90f6d45262R151
>
> I remember a bug about that and maybe the go version in Travis is hitting that. Do you mind trying to run tests on explicit dirs or hack with versions?
I hope this could help https://github.com/apache/camel-k/pull/1822/files#diff-b2c08647d24f7df76cd7a5f2adc119ab93451ad2f662ded422e55943fd889b04R86
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] nicolaferraro commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
nicolaferraro commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-733810982
> @nicolaferraro WDYT?
I was thinking about this but did not have time to look carefully.
I remember there was an issue in travis that happened when an execution did not produce any output for a long time. But don't know why this change triggers that issue.
This may be something completely silly, but I remember some issue about the `...` notation here: https://github.com/apache/camel-k/pull/1822/files#diff-9aecbcc15d64c3e7a146ee5776a35e65b45fb72461e370c4e295dd90f6d45262R151
I remember a bug about that and maybe the go version in Travis is hitting that. Do you mind trying to run tests on explicit dirs or hack with versions?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] nicolaferraro commented on pull request #1822: Split tests by language
Posted by GitBox <gi...@apache.org>.
nicolaferraro commented on pull request #1822:
URL: https://github.com/apache/camel-k/pull/1822#issuecomment-734755681
Thanks
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org