-
Notifications
You must be signed in to change notification settings - Fork 5
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
Deliver/verify that Lmod works on a Bright cluster, in default rollout (at rpm/system level) #5
Comments
I'm interested in this too, i.e. in a script that tests a bunch of |
Testing Lmod is done with Hermes (https://github.com/rtmclay/Hermes). You just need to run Not sure if this works on an installed Lmod though (EasyBuild ships it's unit tests in the installation, I don't think Lmod does). |
The github repo does include the tests. The 6.1 tar ball doesn't but all new releases will. R. |
as of 20160711, it all looks good on this end:
@rtmclay: is the test-suite included in the default lmod rollout? |
@fgeorgatos sort of, it needs to be run from the Lmod repo (not from an installation), cfr. TACC/Lmod#130 |
As @boegel said, the test suite runs against the source code in the repo not an installation. You can look at the .travis.yml file to see what it does. Mainly you will download hermes, put the bin directory in your path and run tm. |
I must be doing something wrong; lua_cmd does not exist on my machines (is this an alias?) and if I hack it I get:
|
OK, I run it twice, all tests passed, however watch out what happens at the end - bug #45:
|
This is because of the startup behavior in /etc/profile.d/z01-default_modules.sh:
The problem is that LD_LIBRARY_PATH is empty so when you create a subshell you end up doing:
I added the check for |
Do I correctly realize that a system with empty default |
It seems to me that you should just remove the test for an empty |
btw. I confirm that the following makes the issue go away: Here's the longer explanation on the why of this: Interestingly, although Lmod's approach is a kludge, it is a better kludge than most other things I see people proposing. It only does not behave when you do mean to have empty LD_LIBRARY_PATH. (perhaps, this is our answer: point to an empty directory, to be sure) |
OK, I start to understand a little bit better what is going on here, and I'd propose the following:
Instead of relying on the value of If not, I come to realize that there is no perfect solution here because there are assumptions to be made. |
fixed via Lmod_test_suite and proven generally OK via report in #85 |
issue closed |
having #4 resolved makes the user experience better
The text was updated successfully, but these errors were encountered: