Skip to content

Conversation

@scrasmussen
Copy link
Member

@scrasmussen scrasmussen commented Jun 26, 2024

Fixing Github runner nvidia MPI installation and caching. List of changes

  • installing OpenMPI v4.1.6 manually, making MPI compilers default
  • installing curl and zlibs with apt-get instead of manually
  • bumping actions' versions
  • quiet wget and tarball expansion
  • small changes to caching and PATH and LD_LIBRARY_PATH variable setting. Separated from build steps

NOTE: nvidia-with-GPU build still fails

- installing OpenMPI v4.1.6 manually, making MPI compiler default
- installing curl, zlibs with apt-get
- bumping actions' versions
- quite wget and tarball expansion
- caching and PATH, LD_LIBRARY_PATH setting changes

fix

fix
Copy link
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me @scrasmussen. It looks like it's failing at the point where #486 should fix it. @dustinswales Would you review this since you were working on this?

Copy link
Member

@dustinswales dustinswales left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@scrasmussen
This looks good to me. Thanks for fixing this.
(Hopefully this is the last of the non-containerized CI test we fix!)

@grantfirl grantfirl merged commit afb3827 into NCAR:main Jul 1, 2024
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.

3 participants