-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
roll-out of alma8-based infrastructure (aka centos 8) #1941
Comments
What I understood was:
And maybe I am missing something else, but we can probably merge the discussion with the CentOS 8 thread mentioned in my comment above. |
Someone, I think @isuruf, mentioned listing versions for other things. libm maybe? |
Manylinux uses glibc major.minor for its versioning, and I think it's good to match that (I think in the core call the mood was that That said, not least after the ill-fated Debian-based
That's because manylinux cannot bring its own updated compiler stack along and so is dependent on having the devtoolset backports. Perhaps keeping RHEL X as a base (through one of its ABI-compatible derivatives like Alma,Rocky,UBI) solves the other versioning questions, even if we call it |
Yes. We plan to keep alma8 as the base. |
One plus about just using |
We'll need to tuck the cdt name in the CDTs somewhere maybe? Idk if the CDT package names will conflict or not. |
Can you send an example recipe where people are dealing directly with cdt_name? I thought the jinja2 function took care of that. |
On the other hand, that line in |
Hmmmm. Being able to at least match cdts to the os for our docker containers is inherently useful possibly? This would argue for using conda_2_28 in both places. |
That's not really needed. We use |
I'm not saying we alway have to have them matched. I'm saying that using the same notation in both places is helpful. |
Would it help to start a PR for Docker images? Or are we not ready for that yet? |
Go for it but I don't expect it to be merged anytime soon. |
Gotcha, what do we see as the required steps before they are merged? Asking since they wouldn't be integrated anywhere by just publishing the image. Or is there something else I'm missing? |
I am not sure what goes in them. If we need the sysroots to put in them, then we'd need that. If they don't have anything special, then we can just build it. |
Gotcha Don't think the sysroot is needed The images cache the compiler packages as a convenience, but that can be disabled temporarily or it can use older compilers for now. Not too worried about this Can't think of anything else of concern If something comes up when we start working on them, we can always discuss |
Started adding an image in PR ( conda-forge/docker-images#235 ) |
From Matt, we need to update |
What's the intention here? Being able to switch the image? (I'm asking because I'd be willing to give it a shot...) FWIW, the current 2.17 setup doesn't need any smithy interaction, it's enough to just add Is there a reason we couldn't switch the images to alma8, but keep the sysroot at cos6 (with opt-in upgrade to cos7 & alma8)? |
There is no reason we couldn't bump images. This may break builds using yum requirements if the packages have changed names or conventions upstream. |
According to this search, there's around 170 recipes that use |
Fair point. I'm happy with simply bumping the default image. |
There seems to be something going awry with the new repodata hack. I'm getting Alma 8 kernel headers together with the COS 7 sysroot on aarch:
Interestingly, this is not happening on PPC, where I get:
|
What makes you think it is the repodata hack? |
It must be related to the sysroot, where the kernel-headers are built, and I couldn't see a difference between aarch/ppc in conda-forge/linux-sysroot-feedstock#46. Both variants are pulling in crdh 4, but looking a bit closer, that divergence between aarch & ppc goes back much further. Seems we've been using newer kernel headers on aarch since conda-forge/linux-sysroot-feedstock#15 (corresponding to linux version in RHEL 8, but apparently still being downloaded through CentOS 7 repos1). Seems it's not critical. Footnotes
|
This is worth a read |
Yes but see the list above. We are trying to get rid of as many CDTs as we can. |
We have merged conda-forge/docker-images#242 recently. What are some next steps here? Are we ready to tackle the first CDTs (modulo conda-forge/cdt-builds#66)? |
We agreed to get rid of as many CDTs as we could. So that's the next step. To go through them and figure out what we can build ourselves. |
Maybe an earlier step is just getting a list of CDTs we use so we can search conda-forge for fuzzy matches |
This issue tracks centos 8 implementation PRs
track_features
.track_features
(ENH glibc 2.28 from alma8 linux-sysroot-feedstock#47)linux-anvil-cuda
on x64 with RHEL 8, consolidate image names, distinguish distro baseline in tag docker-images#291make sure cdt_name is conda_2_28justconda
nowThe alma 8 repo is https://repo.almalinux.org/almalinux/8.7/BaseOS/x86_64/os/Packages/ etc.
The text was updated successfully, but these errors were encountered: