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

CI no longer working? #820

Closed
edwardhartnett opened this issue Apr 10, 2024 · 8 comments
Closed

CI no longer working? #820

edwardhartnett opened this issue Apr 10, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@edwardhartnett
Copy link
Contributor

The CI is failing like this:

/home/runner/work/fv3atm/fv3atm/fv3atm/ccpp/framework/src/ccpp_types.F90:22:9:

   22 |     use mpi_f08, only: MPI_Comm
      |         1
Fatal Error: Cannot open module file ‘mpi_f08.mod’ for reading at (1): No such file or directory
compilation terminated.

@AlexanderRichert-NOAA any idea why this is happening?

@edwardhartnett edwardhartnett added the bug Something isn't working label Apr 10, 2024
@AlexanderRichert-NOAA
Copy link
Contributor

Hm not offhand. It's using a cached copy of mpich (which is built through Spack), so it's possible but unlikely that mpi_f08.mod would be randomly missing... I'm not sure why this would be happening "suddenly," but it might be worth explicitly setting "CMAKE_C_COMPILER=mpicc" and "CMAKE_Fortran_COMPILER=mpifort". If that doesn't work, you could look at the output of find $(spack location --install-dir mpich) right before the cmake call just to confirm that mpi_f08.mod is present, plus make VERBOSE=1 to confirm that there's an appropriate '-I' entry for that module.

Also clearing the cache is my go-to "turn it off and back on" solution...

@AlexanderRichert-NOAA
Copy link
Contributor

Addendum: I just tried installing [email protected] with Spack on my personal machine and there's also no mpi_f08.mod, so I have to imagine that's related. I'll look into why it's missing (or maybe it was never there?).

@AlexanderRichert-NOAA
Copy link
Contributor

Likely culprit: https://github.com/NCAR/ccpp-framework/blob/011db4f80a02cba6d65958ace56e8efb197be62b/src/ccpp_types.F90 (the ccpp framework submodule was updated in fv3atm a couple weeks ago)
I'm betting #define MPI wasn't set before so it wasn't using the use mpi_f08.mod previously...

@DusanJovic-NOAA
Copy link
Collaborator

It will be fixed in this PR #805

@edwardhartnett
Copy link
Contributor Author

@junwang-noaa and @DusanJovic-NOAA the settings on this repo should be set so that no PR which breaks CI can be merged.

image

@DusanJovic-NOAA
Copy link
Collaborator

Fixed in #811

@edwardhartnett
Copy link
Contributor Author

@DusanJovic-NOAA have repo settings been changed to prevent recurrence?

@DusanJovic-NOAA
Copy link
Collaborator

I didn't change anything but I see those two settings are already selected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants