Skip to content

Update CI#926

Merged
eschnett merged 40 commits intomasterfrom
eschnett/CI
Dec 19, 2025
Merged

Update CI#926
eschnett merged 40 commits intomasterfrom
eschnett/CI

Conversation

@eschnett
Copy link
Contributor

No description provided.

@giordano
Copy link
Member

Any reason for dropping testing for Julia v1.6? I'm uncomfortable with not testing what's the minimum supported version of a package, in fact I think that in GitHub Actions we should always use the min version, to be consistent with the Project.toml. If you believe there's a strong reason for dropping support for Julia v1.90, we could do that, but pretending that a version is supported but not testing it is less good for me.

@eschnett
Copy link
Contributor Author

No reason, except that I'm adding the more important more modern versions. I'll add it back.

@eschnett
Copy link
Contributor Author

OpenMPI is not working well with Julia 1.6: https://github.com/JuliaParallel/MPI.jl/actions/runs/20177999182/job/57930921483?pr=926

@eschnett
Copy link
Contributor Author

@giordano OpenMPI_jll 5.0.9 is not working with Julia 1.6. I recall the reason is that there is no libgfortran4 version. How do we solve this?

@eschnett
Copy link
Contributor Author

Can I get a "thumbs up" for this PR? All comments/requests are addressed, and CI is green.

eschnett and others added 2 commits December 16, 2025 12:43
Co-authored-by: Mosè Giordano <765740+giordano@users.noreply.github.com>
@eschnett
Copy link
Contributor Author

test_io_shared is now failing on Ubuntu as well, after reliably failing on Windows and sometimes failing on macOS. We have plastered that test with MPI_File_sync and MPI_Barrier, and I can't see what else to do.

This failing tests uses MPICH. MPICH implements the shared file pointer by creating a hidden file and storing the shared file pointer there. It then uses (file?) locking to ensure exclusive access.

@giordano Is file locking (via fcntl) implemented reliably for the file system where we're running the tests?

MPICH has special code for NFS – not sure it's using this code.

@eschnett
Copy link
Contributor Author

I tightened the cases where we skip tests. This is now only for MPICH (for Apple and Windows). It seems OpenMPI is working fine for us all the time.

@eschnett
Copy link
Contributor Author

I think this is good now. Do you want to have another look?

@eschnett eschnett marked this pull request as draft December 19, 2025 17:59
@eschnett eschnett marked this pull request as ready for review December 19, 2025 17:59
@eschnett eschnett merged commit 4a0bcc0 into master Dec 19, 2025
52 checks passed
@eschnett eschnett deleted the eschnett/CI branch December 19, 2025 20:33
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.

2 participants