Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase timeout for node connection in e2e tests #3079

Merged

Conversation

piotr-iohk
Copy link
Contributor

@piotr-iohk piotr-iohk commented Jan 12, 2022

  • Increase timeout for node connection as replaying blocks on the node may take even 1h more than 1h on node version bumps

Comments

https://github.com/input-output-hk/cardano-wallet/runs/4788838771?check_suite_focus=true

Issue Number

@piotr-iohk piotr-iohk self-assigned this Jan 12, 2022
@piotr-iohk piotr-iohk mentioned this pull request Jan 12, 2022
6 tasks
…may take more than 1h on node version bumps
@piotr-iohk piotr-iohk force-pushed the piotr/increase-timeout-for-node-connection-in-e2e-tests branch from 0e827f0 to 3f09a7c Compare January 12, 2022 11:58
@piotr-iohk piotr-iohk merged commit d60a49a into master Jan 12, 2022
@piotr-iohk piotr-iohk deleted the piotr/increase-timeout-for-node-connection-in-e2e-tests branch January 12, 2022 15:31
iohk-bors bot added a commit that referenced this pull request Jan 14, 2022
3081: Increase timeouts for fetching node in benchmarks and e2e tests r=piotr-iohk a=piotr-iohk

- [x] I have increased timeouts in restoration benchmarks and e2e tests

### Comments

Follow up to #3079.
 - e2e tests 120 mins was not enough for [MacOS](https://github.com/input-output-hk/cardano-wallet/actions/runs/1689464914). 
   - 🧪 [Testing](https://github.com/input-output-hk/cardano-wallet/actions/runs/1691714591). 180mins ✔️ 
 - hopefully the same issue in [benchmarks](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1216#68ecaf3e-2649-4d9e-804e-bef3bad97fc). 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1218)  180mins ❌ 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1219) 240mins ✔️ (build failed due to cancellation signal 🤔 , but passed through connecting to the node)

### Issue Number

<!-- Reference the Jira/GitHub issue that this PR relates to, and which requirements it tackles.
  Note: Jira issues of the form ADP- will be auto-linked. -->


Co-authored-by: Piotr Stachyra <[email protected]>
iohk-bors bot added a commit that referenced this pull request Jan 14, 2022
3081: Increase timeouts for fetching node in benchmarks and e2e tests r=piotr-iohk a=piotr-iohk

- [x] I have increased timeouts in restoration benchmarks and e2e tests

### Comments

Follow up to #3079.
 - e2e tests 120 mins was not enough for [MacOS](https://github.com/input-output-hk/cardano-wallet/actions/runs/1689464914). 
   - 🧪 [Testing](https://github.com/input-output-hk/cardano-wallet/actions/runs/1691714591). 180mins ✔️ 
 - hopefully the same issue in [benchmarks](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1216#68ecaf3e-2649-4d9e-804e-bef3bad97fc). 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1218)  180mins ❌ 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1219) 240mins ✔️ (build failed due to cancellation signal 🤔 , but passed through connecting to the node)

### Issue Number

<!-- Reference the Jira/GitHub issue that this PR relates to, and which requirements it tackles.
  Note: Jira issues of the form ADP- will be auto-linked. -->


Co-authored-by: Piotr Stachyra <[email protected]>
iohk-bors bot added a commit that referenced this pull request Jan 18, 2022
3081: Increase timeouts for fetching node in benchmarks and e2e tests r=rvl a=piotr-iohk

- [x] I have increased timeouts in restoration benchmarks and e2e tests

### Comments

Follow up to #3079.
 - e2e tests 120 mins was not enough for [MacOS](https://github.com/input-output-hk/cardano-wallet/actions/runs/1689464914). 
   - 🧪 [Testing](https://github.com/input-output-hk/cardano-wallet/actions/runs/1691714591). 180mins ✔️ 
 - hopefully the same issue in [benchmarks](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1216#68ecaf3e-2649-4d9e-804e-bef3bad97fc). 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1218)  180mins ❌ 
   - 🧪 [Testing](https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1219) 240mins ✔️ (build failed due to cancellation signal 🤔 , but passed through connecting to the node)

### Issue Number

<!-- Reference the Jira/GitHub issue that this PR relates to, and which requirements it tackles.
  Note: Jira issues of the form ADP- will be auto-linked. -->


Co-authored-by: Piotr Stachyra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants