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

BUG: Revert to valid manylinux2014-x64 image tag #240

Merged
merged 1 commit into from Dec 7, 2022
Merged

Conversation

tbirdso
Copy link
Contributor

@tbirdso tbirdso commented Dec 7, 2022

While manylinux_2_28 and manylinux2014 images are typically tagged together, a recent manylinux_2_28 release addressing a _2_28 toolset compatibility issue was not paired with a manylinux2014 release. This commit updates Linux build scripts to decouple the tag used for each image.

See also: https://hub.docker.com/u/dockcross

While `manylinux_2_28` and `manylinux2014` images are typically tagged
together, a recent `manylinux_2_28` release addressing a _2_28 toolset
compatibility issue was not paired with a `manylinux2014` release. This
commit updates Linux build scripts to decouple the tag used for each
image.

See also: https://hub.docker.com/u/dockcross
@tbirdso
Copy link
Contributor Author

tbirdso commented Dec 7, 2022

This issue was not caught because docker failed silently in ITKSplitComponents in the most recent tag bump, will open an issue in ITKRemoteModuleBuildTestPackageAction to ensure that a docker failure results in a failed job.

https://github.com/InsightSoftwareConsortium/ITKSplitComponents/actions/runs/3632771768/jobs/6129000489

~/work/ITKSplitComponents/ITKSplitComponents
Unable to find image 'dockcross/manylinux2014-x64:20221205-459c9f0' locally
docker: Error response from daemon: manifest for dockcross/manylinux2014-x64:20221205-459c9f0 not found: manifest unknown: manifest unknown.
See 'docker run --help'.

@tbirdso
Copy link
Contributor Author

tbirdso commented Dec 7, 2022

Seeing successful ITKSplitComponents 2014 builds with changes applied: https://github.com/InsightSoftwareConsortium/ITKSplitComponents/actions/runs/3642021820/jobs/6148645321

@tbirdso
Copy link
Contributor Author

tbirdso commented Dec 7, 2022

@tbirdso tbirdso merged commit 943b4b7 into master Dec 7, 2022
@hjmjohnson hjmjohnson deleted the bump-manylinux branch April 4, 2023 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants