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

Image upload to hactrn.kostersitz.com is corrupted #2299

Open
oilcan-productions opened this issue Apr 15, 2024 · 9 comments
Open

Image upload to hactrn.kostersitz.com is corrupted #2299

oilcan-productions opened this issue Apr 15, 2024 · 9 comments
Labels

Comments

@oilcan-productions
Copy link
Contributor

          @oilcan-productions, I often get this now.  If I rerun the build 1-2 times it will usually work.  Any idea why?

Screenshot from 2024-04-12 07-02-06

Originally posted by @larsbrinkhoff in #2294 (comment)

@oilcan-productions oilcan-productions changed the title Image upload to hactren.kostersitz.com is corrupted Image upload to hactrn.kostersitz.com is corrupted Apr 15, 2024
@oilcan-productions
Copy link
Contributor Author

Checked the server side logs, show Client disconnected, nothing more.
I opened a support ticket to see if there is throttling happening in the background somewhere

@oilcan-productions
Copy link
Contributor Author

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?

@larsbrinkhoff
Copy link
Member

Thanks. I started rebuild of the latest commit on master: https://github.com/PDP-10/its/actions/runs/8651430739

@larsbrinkhoff
Copy link
Member

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.)

@oilcan-productions
Copy link
Contributor Author

@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.

@larsbrinkhoff
Copy link
Member

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

@larsbrinkhoff
Copy link
Member

larsbrinkhoff commented May 4, 2024

Here's a curious case: your retry mechanism kicked in, but two targets reached the limit.
https://github.com/PDP-10/its/actions/runs/8948158873

The other four worked on the first try.

@larsbrinkhoff
Copy link
Member

And then the next build went perfectly fine. No retries needed.
https://github.com/PDP-10/its/actions/runs/8953189776

@oilcan-productions
Copy link
Contributor Author

Added to the retry logic to recreate the tarball upon retry to ensure it is actually correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants