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

ci: increase timeout for flaky e2e test steps #18189

Merged
merged 11 commits into from
Apr 14, 2023

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Mar 16, 2023

Explanation

See:

Increases the timeouts for intermittently failing e2e steps (theory: prob heavy throttling in circleci workers)

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

@github-actions
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch from 6be289a to d306a41 Compare March 16, 2023 12:28
@legobeat legobeat marked this pull request as ready for review March 16, 2023 12:34
@legobeat legobeat requested a review from a team as a code owner March 16, 2023 12:34
@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch from d306a41 to ba1bb89 Compare March 16, 2023 12:45
@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch from ba1bb89 to c1c9538 Compare March 17, 2023 22:56
@codecov
Copy link

codecov bot commented Mar 17, 2023

Codecov Report

Merging #18189 (8583555) into develop (f92e463) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 8583555 differs from pull request most recent head b9ec0ae. Consider uploading reports for the commit b9ec0ae to get more accurate results

@@             Coverage Diff             @@
##           develop   #18189      +/-   ##
===========================================
- Coverage    65.15%   65.15%   -0.00%     
===========================================
  Files          936      936              
  Lines        35965    35963       -2     
  Branches      9231     9231              
===========================================
- Hits         23432    23429       -3     
- Misses       12533    12534       +1     

see 6 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Gudahtt
Gudahtt previously approved these changes Mar 21, 2023
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@legobeat legobeat requested review from jpuri and kumavis March 21, 2023 22:48
@legobeat legobeat added the DO-NOT-MERGE Pull requests that should not be merged label Mar 22, 2023
@legobeat
Copy link
Contributor Author

timeouts not actually bumped this way - fix incoming

@kumavis
Copy link
Member

kumavis commented Mar 22, 2023

we identified it should not be merged bc the api usage is not correct

@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch 2 times, most recently from 89aa869 to 5be212f Compare March 27, 2023 06:44
@legobeat legobeat mentioned this pull request Mar 27, 2023
@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch from 5be212f to 049e075 Compare March 27, 2023 06:47
@legobeat legobeat force-pushed the test-e2e-flaky-increase-timeout branch from 049e075 to 95b3867 Compare March 27, 2023 07:51
@legobeat
Copy link
Contributor Author

legobeat commented Mar 27, 2023

@legobeat legobeat removed the DO-NOT-MERGE Pull requests that should not be merged label Mar 27, 2023
@legobeat legobeat merged commit 17147b3 into MetaMask:develop Apr 14, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Apr 14, 2023
@legobeat legobeat deleted the test-e2e-flaky-increase-timeout branch April 14, 2023 08:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants