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

Set snellius zen4 override #187

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

casparvl
Copy link
Collaborator

No description provided.

@@ -63,7 +63,10 @@
{
'name': 'genoa',
'scheduler': 'slurm',
'prepare_cmds': [common_eessi_init()],
'prepare_cmds': [
'export EESSI_SOFTWARE_SUBDIR_OVERRIDE=x86_64/amd/zen4',
Copy link
Collaborator

Choose a reason for hiding this comment

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

what does this do exactly?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Make sure that on our Genoa nodes, the zen4 stack is used. The EESSI init script currently falls back on zen3, since the zen4 stack is not 'complete' yet (it doesn't have all the modules that are there for zen3, though it's getting pretty close). I don't want that fallback, I really want to test the zen4 stack :)

You'll find the same in our azure_mc.py config, that's the build cluster for our zen4 stuff.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note that it also doesn't hurt to hard code those overrides if you know what partition you are on. It simply means you're not relying on the autodetection in the init script (but I know the hardware in this partition, so that's ok :P)

Copy link
Collaborator

@satishskamath satishskamath left a comment

Choose a reason for hiding this comment

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

Since this is in the ReFrame config file this should not hurt. lgtm!

@satishskamath satishskamath merged commit b97f9e9 into EESSI:main Sep 26, 2024
10 checks passed
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