-
Notifications
You must be signed in to change notification settings - Fork 7
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
Cannot upload artifacts #160
Comments
That is probably a temporary connection error on GitHub's side. Can you please try restarting the workflow? Just in case, remove any asset from https://github.com/JoyMoe/OpenData/releases/tag/nightly starting with |
@eine after remove the release, it works well now. I'll keep watching for some days. |
we have the same problem here https://github.com/msys2/msys2-installer/runs/3980189142?check_suite_focus=true this started ~18 day ago |
It works well for 8 days now. Seems no other action need to take. |
Same issue over here: https://github.com/stnolting/neorv32/runs/3995838174?check_suite_focus=true The problem started yesterday; four days ago everything was still working fine. |
@stnolting, I cleaned the assets of the nightly release in NEORV32 and I restarted the Documentation workflow. It's green now. |
@umarcor |
yeah, deleting assets before the upload seems to work around the issue |
The main problem here is that GitHub's internal connections seem not to be very stable, so rather frequently uploading artifacts as assets does produce failures. Until recently, tip could not handle the broken state, and that's why the recommended workaround was to remove the assets manually and restart the workflow. Most of the time, that works. However, it is undesirable, because assets are not available for the duration of the workflow (which can take up to 6h). So, users willing to retrieve tip/nightly assets do get errors during that period. In order to maximise the availability of assets, tip first uploads new versions with a different name, then removes the old one and last it renames the new: Lines 160 to 166 in fb0c52c
Until some days ago, when the first step failed ("uploading tmp."), all subsequent executions of the Action would fail, because there was no procedure for cleaning it. So, a possible less intrusive solution was to manually remove the assets shown as Lines 139 to 152 in fb0c52c
Therefore, most of the problems you had should not happen again, as long as they were related to this "uploading of tmp files failing". However, I still found some failing cases that required me to manually remove all assets (not the release, just the assets), and not only the On the other hand, users with option |
Tried again, still fails :( https://github.com/msys2/msys2-installer/runs/4202718077?check_suite_focus=true Likely the problem is with github, still annoying :( |
Could this be an alpine related issue? searching for the SSL error gives me various alpine user reports. Maybe we could try the default docker image instead ( |
@lazka, absolutely! If that is the case, that'd be an awesome catch! Let me run some local tests and I'll update the dockerfile. |
So, on the one hand I changed the base image from @lazka, @stnolting, @kinosang, if you want to try the composite variant, just changing the name should be enough: |
Just wanted to point out that despite it being larger it is still faster to run since there are Python wheels for Debian while it compiles packages on alpine. |
Still the same issue: https://github.com/msys2/msys2-installer/runs/4229934050?check_suite_focus=true Sorry for the wrong lead :/ |
So, another lead: this started to happen exactly when Python 3.10 was released (5 Oct on docker hub). And the docker image is always using the newest one. |
Related to eine#160. The errors started arround the time when Python 3.10 was released, so go back to 3.9.x
Related to eine#160. The errors started around the time when Python 3.10 was released, so go back to 3.9.x to see if that is related.
Related to #160. The errors started around the time when Python 3.10 was released, so go back to 3.9.x to see if that is related.
Now it's just another error: https://github.com/msys2/msys2-installer/runs/4273363943?check_suite_focus=true But at least it only fails every second time instead of every time :) oh well... |
Got error of
ssl.SSLEOFError: EOF occurred in violation of protocol (_ssl.c:2384)
full logs see https://github.com/JoyMoe/OpenData/runs/3917346331?check_suite_focus=true
The text was updated successfully, but these errors were encountered: