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.

Sign up for GitHub

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

Jump to bottom

Catch network error #74

Merged
merged 3 commits into from
Dec 15, 2022
Merged

Catch network error #74

merged 3 commits into from
Dec 15, 2022

Conversation

tuliren
Copy link
Member

@tuliren tuliren commented Dec 15, 2022 •
edited
Loading

  • I saw some S3 uploading failures, probably due to large file size.
  • This is a stopgap solution. The plan is to retry and log more information about the error and the offending files first.
  • Will probably need to implement multipart uploading in the future. Issue: Implement multipart S3 uploading #75

@stoat-app
Copy link

stoat-app bot commented Dec 15, 2022 •
edited
Loading

Easy and customizable dashboards for your build system. Learn more about Stoat ↗︎

Static Hosting

Name Link Commit Status
Action Test Coverage Visit fe013f7 ✅
Action Test Coverage (single file) Visit fe013f7 ✅

Job Runtime

debug

@tuliren tuliren merged commit 0c2379a into main Dec 15, 2022
@tuliren tuliren deleted the liren/catch-network-error branch December 15, 2022 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jrhizor jrhizor

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tuliren @jrhizor