-
Notifications
You must be signed in to change notification settings - Fork 566
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
Python 3.12.4: Extract downloaded archive; gzip: stdin: not in gzip format #886
Comments
Same thing with 3.13.0b2:
https://github.com/pypa/pip/actions/runs/9443536250/job/26007467646 Both 3.12.4 and 3.13.0-beta.2 were released on https://github.com/actions/python-versions a couple of hours ago: |
Seems like the header is incorrect?
|
the same with macos and win archives :
|
The file is completely garbled. It mostly consists of |
I have noticed this is working in |
The hosted runners have an cached toolchain with multiple versions of python already installed. The action is using that instead downloading an version. GitHub hosted is running on 50Gb+ VMs in Azure to minimize download and install cycles. Edit: |
macOS runners can't extract the 3.12.4 tarball properly it seems. actions/setup-python#886
Ah, nice spotting. However it's weird that the deflate header is kept as-is while the |
The That is the symbol you usually see when UTF-8 is being displayed as ASCII. It looks the files have been treated as text causing all the non-ASCII (i.e. binary) data being treated as unprintable characters thus corrupting it in the process. |
This might be related to the This commit is dated on May 22nd whereas the last previous release is from May 9th |
This removes Node.js 16 deprecation warnings
Hello @frenck |
The hashes do not match at all.
The files are also double in size than the previous versions because of the corruption. The source archives are fine.
|
Download artifact will corrupt files if there are any matching file names when merging multiple, this looks suspiciously like that. actions/download-artifact#298 |
The manifest that holds broken releases has been reverted by @HarithaVattikuti : actions/python-versions#282 This removes actions/python-versions#285 should resolve the issue. |
* Update xcode to 15.4 * update runners * remove `arch -x86_64` for python * use python 3.12.3 as a workaround for actions/setup-python#886 * do not specify arch
Hello Everyone, we merged the fix for this issue. Please validate from your end and confirm. |
Thanks @aparnajyothi-y, I can confirm the latest release / manifest fixes the issue in #863 (some GH-hosted workers) and #884 (all GH-hosted workers). |
All looks good for us at flake8-pyi and typeshed now -- thanks @aparnajyothi-y! |
CI now passes at |
Some good confirmations above. Can confirm it works on our end too. Closing the issue for that reason. Thanks for the fix! ❤️ ../Frenck |
* Update xcode to 15.4 * update runners * remove `arch -x86_64` for python * use python 3.12.3 as a workaround for actions/setup-python#886 * do not specify arch
macOS runners can't extract the 3.12.4 tarball properly it seems. actions/setup-python#886
macOS runners can't extract the 3.12.4 tarball properly it seems. actions/setup-python#886
macOS runners can't extract the 3.12.4 tarball properly it seems. actions/setup-python#886
Description:
A clear and concise description of what the bug is.
https://github.com/home-assistant/core/actions/runs/9442844577/job/26006478894?pr=119255
Action version:
Specify the action version
actions/[email protected]
Platform:
Runner type:
Tools version:
3.12 (resulting in 3.12.4 right now)
Repro steps:
A description with steps to reproduce the issue. If your have a public example or repo to share, please provide the link.
https://github.com/home-assistant/core/blob/159503b8d37fbc46a353287416a640ad07c2cced/.github/workflows/ci.yaml#L230-L235
Expected behavior:
A description of what you expected to happen.
Actual behavior:
A description of what is actually happening.
The text was updated successfully, but these errors were encountered: