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

Error message reported about an indirect dependency issue whereas module is correctly loaded #509

Closed
xdelaruelle opened this issue Sep 8, 2023 · 2 comments
Labels
Milestone

Comments

@xdelaruelle
Copy link
Member

xdelaruelle commented Sep 8, 2023

A requirement error message appears when loading a module whereas an alternative dependency has been successfully loaded, thus loaded environment satisfies all dependency constraints.

$ cat modulefiles/cuda/11.6
#%Module
conflict cuda 
$ cat modulefiles/cuda/12.2
#%Module
conflict cuda 
$ cat modulefiles/hwloc/2.2.0
#%Module
conflict hwloc
$ cat modulefiles/hwloc/2.9.2
#%Module
prereq cuda/12
conflict hwloc
$ cat modulefiles/hwloc/.modulerc
#%Module
module-version hwloc/2.2.0 default
$ cat modulefiles/openmpi/4.1.5
#%Module
prereq hwloc/2.9.2 hwloc

$ ml cuda/11
$ ml openmpi
Loading openmpi/4.1.5
  ERROR: Load of requirement cuda/12 failed
  Loading requirement: hwloc/2.2.0
$ ml
Currently Loaded Modulefiles:
 1) cuda/11.6   2) hwloc/2.2.0   3) openmpi/4.1.5  

In the above example, an attempt to load hwloc/2.9.2 is done which fails as its dependency cuda/12 cannot be loaded (cuda/11 is already loaded). Main module openmpi/4.1.5 successfully loads as hwloc/2.2.0 is loaded instead of version 2.9.2.

The error message on openmpi loading block message about cuda/12 is disturbing:

  • it does not relate to a direct dependency of openmpi/4.1.5, thus it is hard to understand what module requires it
  • this error is not one for openmpi module load as an alternative has been successfully loaded.

Thanks to @adrien-cotte and his team for the bug report.

@xdelaruelle xdelaruelle added the bug label Sep 8, 2023
@xdelaruelle xdelaruelle added this to the 5.4.0 milestone Sep 8, 2023
xdelaruelle added a commit to xdelaruelle/modules that referenced this issue Sep 10, 2023
xdelaruelle added a commit to xdelaruelle/modules that referenced this issue Sep 10, 2023
xdelaruelle added a commit to xdelaruelle/modules that referenced this issue Sep 10, 2023
@xdelaruelle
Copy link
Member Author

@adrien-cotte This fix will be shipped in v5.4. If you want to quickly deploy it on your v5.2+ setup, I am confident that 39c7cdc could apply on a v5.2+ version.

@adrien-cotte
Copy link
Contributor

Thank you for anticipating my question 🙂👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants