Skip to content

Conversation

priteau
Copy link
Member

@priteau priteau commented Jun 23, 2025

This includes the /usr/bin/mail command, which is required for Slurm to send email notifications [1].

[1] https://slurm.schedmd.com/slurm.conf.html#OPT_MailProg

@priteau priteau self-assigned this Jun 23, 2025
@priteau priteau requested a review from a team as a code owner June 23, 2025 16:03
@priteau priteau marked this pull request as draft June 26, 2025 19:45
@priteau priteau marked this pull request as ready for review June 26, 2025 19:48
This includes the /usr/bin/mail command, which is required for Slurm to
send email notifications [1].

[1] https://slurm.schedmd.com/slurm.conf.html#OPT_MailProg
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

Is the package alone enough to be useful? I was expecting this was e.g. to send emails on job status and therefore to see e.g. Mail* parameters configured in slurm.conf configured?

@priteau
Copy link
Member Author

priteau commented Jul 8, 2025

Is the package alone enough to be useful? I was expecting this was e.g. to send emails on job status and therefore to see e.g. Mail* parameters configured in slurm.conf configured?

There is actually no other configuration to perform in Slurm itself. Of course, you need a functional Postfix SMTP relay, but its configuration is mostly site-specific. I was planning to push some documentation separately, as I still want to check if we should import an existing Ansible role for Postfix configuration.

@priteau
Copy link
Member Author

priteau commented Jul 8, 2025

To be more precise, this is because the default value of MailProg is already /bin/mail (or /usr/bin/mail if /bin/mail does not exist but /usr/bin/mail does exist), as documented in https://slurm.schedmd.com/slurm.conf.html.

@priteau priteau requested a review from sjpb July 9, 2025 08:10
@priteau
Copy link
Member Author

priteau commented Jul 9, 2025

Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

OK - as it is just a package it doens't need docs for slurm-controlled-rebuild or isolated slurm.

@sjpb sjpb merged commit 61692eb into main Jul 9, 2025
8 of 9 checks passed
@sjpb sjpb deleted the slurm-mail branch July 9, 2025 10:59
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.

2 participants