-
Notifications
You must be signed in to change notification settings - Fork 202
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
add support for easyconfig parameter module_only
#4537
Conversation
module_only
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes in easybuild/*
look OK to me, but we need a test for this...
test added in 64cde9e |
# check use of module_only parameter | ||
test_ec_txt += "\nmodule_only = True\n" | ||
write_file(test_ec, test_ec_txt) | ||
self.eb_main(args, do_build=True, raise_error=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@smoors You should tweak args
here too, so it doesn't include --skip
anymore (or --force
, I guess), and also check that no software was installed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done in 430189c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
the motivation is to support installing only the module for certain deps of a software when installing with
eb --robot
the
module_only
parameter can then be set in a hook depending on a certain conditionuse case: installing only the module for CUDA software on non-GPU nodes (e.g. the login nodes), thus making the CUDA modules visible (but not actually usable)
@boegel this helps getting the VSC-bot working in hydra by replacing the 'dummy modules' feature in our submit_build script with a hook