-
Notifications
You must be signed in to change notification settings - Fork 78
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
Image upload to hactrn.kostersitz.com is corrupted #2299
Comments
Checked the server side logs, show Client disconnected, nothing more. |
They fixed an error in the quota settings file (should be unlimited and was set to 250MB and 4 concurrent connections) of the home path of the account we use to upload. Now it should be unlimited around. @larsbrinkhoff can you kick off a build and see if that resolved the issue? |
Thanks. I started rebuild of the latest commit on master: https://github.com/PDP-10/its/actions/runs/8651430739 |
The restarted build failed for three out of six targets, due to the FTP check. It's a good thing you added that verification! Still, the upload goes wrong sometimes, apparently. (Since we need "make download" to work, I reran the builds until they passed, so now everything is green again.) |
@larsbrinkhoff I am seeing the build right now not deploying at all to the FTP server, as well as the Ubuntu build failing before it gets to the deploy stage. |
Only commits on master will run the deploy script. I have merged the pull request now; here's the build to watch: https://github.com/PDP-10/its/actions/runs/8905672868 |
Here's a curious case: your retry mechanism kicked in, but two targets reached the limit. The other four worked on the first try. |
And then the next build went perfectly fine. No retries needed. |
Added to the retry logic to recreate the tarball upon retry to ensure it is actually correct. |
Originally posted by @larsbrinkhoff in #2294 (comment)
The text was updated successfully, but these errors were encountered: