-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Update yarn network-timeout: 30s(default) -> 600s #6531
Update yarn network-timeout: 30s(default) -> 600s #6531
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6531 +/- ##
=======================================
Coverage 61.53% 61.53%
=======================================
Files 158 158
Lines 12898 12898
Branches 1756 1756
=======================================
Hits 7937 7937
Misses 4704 4704
Partials 257 257 |
Ahhh, cool. I was thinking we might need to back out the arm64 PR, but if this works instead then it's the better way to go. |
Co-authored-by: Guido Petri <[email protected]>
We'd better back out this change, and the arm64 one too, so that That being said, if there's some way to get the arm64 builds working reliably later on then that'd be welcome. 😄 |
@justinclift Sorry for breaking the test.. Thanks for undoing the changes. |
It's no worries. It was worth trying out the PR just in case it all worked. 😄 Don't be afraid to experiment and try things out. 😁 |
This PR reverts PRs getredash#6525 and getredash#6531, due to problems building the arm64 images with GitHub Actions
What type of PR is this?
Description
In the docker build of GitHub Actions, yarn failed with ESOCKETTIMEDOUT, so change the network-timeout.
cf. https://github.com/getredash/redash/actions/runs/6533929053/job/17752711820#step:9:2944
How is this tested?
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)
N/A