Skip to content

add support for prepare_cmds in the EESSI environments - #354

Merged
laraPPr merged 6 commits into
EESSI:mainfrom
smoors:eessi-modules
Sep 11, 2026
Merged

laraPPr merged 6 commits into
EESSI:mainfrom
smoors:eessi-modules

Conversation

@smoors

@smoors smoors commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

in the config file, EESSI prepare_cmds really belong in the environments instead of the partitions, because otherwise it would mess up the local environment.

to help with this, i added the eessi_prepare_cmds argument to set_required_config. now i can change in the config file this:

set_common_required_config(site_configuration)

for environ in site_configuration['environments']:
    if environ['name'].startswith('EESSI'):
        environ['prepare_cmds'] = ['export MODULEPATH=/cvmfs/software.eessi.io/init/modules']

to this:

eessi_prepare_cmds = ['export MODULEPATH=/cvmfs/software.eessi.io/init/modules']
set_common_required_config(site_configuration, set_memory=False, eessi_prepare_cmds=eessi_prepare_cmds)                                                                                                                                                                               

see also the companion docs PR EESSI/docs#862

Comment on lines -16 to -18
pull_request:
branches:
- main

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

i removed this because the scorecards checks are not designed for running on every PR. it will still run every week on the main branch.

@satishskamath satishskamath self-assigned this Sep 10, 2026
@laraPPr

laraPPr commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

@smoors I noticed that what I did is indeed not working. But this would also not work at UGent. Because it still requires the module --force purge

Comment thread config/settings_example.py Outdated
Comment on lines +128 to +129
'source /cvmfs/software.eessi.io/2025.06/init/lmod/bash',
'module unload EESSI',

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this not better?

Suggested change
'source /cvmfs/software.eessi.io/2025.06/init/lmod/bash',
'module unload EESSI',
'module use /cvmfs/software.eessi.io/init/modules',

@laraPPr laraPPr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@laraPPr
laraPPr merged commit ea8327e into EESSI:main Sep 11, 2026
14 checks passed
laraPPr added a commit to laraPPr/EESSI-test-suite that referenced this pull request Sep 11, 2026
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