You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2022/08/15 18:46:43 UTC
[GitHub] [tinkerpop] L0Lmaker opened a new pull request, #1782: TINKERPOP-2779: Setting Cucumber timeout to 10s for Floating Errors on Windows on GitHub Actions
L0Lmaker opened a new pull request, #1782:
URL: https://github.com/apache/tinkerpop/pull/1782
Temporary fix for the [TINKERPOP-2779](https://issues.apache.org/jira/browse/TINKERPOP-2779).
Slowdown appears to be coming from the Gremlin Server that is spun up to run the Cucumber tests. On logging some information from the server, I was able to verify that the server receives and processes the Connected Component Traversal ([Link to log line](https://github.com/L0Lmaker/tinkerpop/runs/7755756716?check_suite_focus=true#step:4:11942)).
Breakdown of what happens in the order it happens:
1. Client sends traversal to the server from Cucumber Step (Note: Cucumber Steps currently have a default timeout of 5s).
2. Server receives the request and starts processing the traversal.
3. Slowdown on the Server causes traversal to be processed much longer than usual.
4. 5s threshold on Cucumber Step is passed and Cucumber fails the scenario with a timeout error.
5. Gremlin-JS on Windows Fails on GitHub Actions.
As of now, without any server changes, the only way to mitigate the occasional GitHub Actions failure for Gremlin-JS Windows would be to increase the Cucumber timeout to 10s. If there is anyone that has encountered similar issues and could add some information that would be helpful. [TINKERPOP-2779](https://issues.apache.org/jira/browse/TINKERPOP-2779) was created specifically with `gremlin-javascript` in mind. Since then, ConnectedComponent's occasional timeout failures have been observed in the `Gremlin Dotnet` Driver as well. I have created a new ticket ([TINKERPOP-2784](https://issues.apache.org/jira/browse/TINKERPOP-2784)) documenting specifics of the `Gremlin Server` issue to keep track of 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.
To unsubscribe, e-mail: commits-unsubscribe@tinkerpop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tinkerpop] spmallette merged pull request #1782: TINKERPOP-2779: Setting Cucumber timeout to 10s for Floating Errors on Windows on GitHub Actions
Posted by GitBox <gi...@apache.org>.
spmallette merged PR #1782:
URL: https://github.com/apache/tinkerpop/pull/1782
--
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.
To unsubscribe, e-mail: commits-unsubscribe@tinkerpop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tinkerpop] codecov-commenter commented on pull request #1782: TINKERPOP-2779: Setting Cucumber timeout to 10s for Floating Errors on Windows on GitHub Actions
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #1782:
URL: https://github.com/apache/tinkerpop/pull/1782#issuecomment-1215612285
# [Codecov](https://codecov.io/gh/apache/tinkerpop/pull/1782?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#1782](https://codecov.io/gh/apache/tinkerpop/pull/1782?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (63d7bd1) into [3.5-dev](https://codecov.io/gh/apache/tinkerpop/commit/5ca0447eab86658882f4805210c746a702838bc6?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (5ca0447) will **not change** coverage.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## 3.5-dev #1782 +/- ##
========================================
Coverage 63.58% 63.58%
========================================
Files 23 23
Lines 3636 3636
========================================
Hits 2312 2312
Misses 1145 1145
Partials 179 179
```
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
--
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.
To unsubscribe, e-mail: commits-unsubscribe@tinkerpop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org