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.

Sign up for GitHub

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

Jump to bottom

Enable SignalChildrenProcesses by default #37

Open
jamesbeedy opened this issue Nov 9, 2024 · 2 comments
Open

Enable SignalChildrenProcesses by default #37

jamesbeedy opened this issue Nov 9, 2024 · 2 comments
Labels
C-slurm Component: Slurm P-medium Priority: Issue needs addressed within a month

Comments

@jamesbeedy
Copy link
Contributor

Bug Description

Slurm process tracking is currently not configured to kill child processes of a job. Set SignalChildrenProcesses=yes in cgroup.conf to enable this functionality.

To Reproduce

NA

Environment

NA

Relevant log output

NA

Additional context

No response
@NucciTheBoss
Copy link
Member

Surprising that Slurm doesn't have this enabled by default. This will be helpful for when we eventually enable Open OnDemand in Charmed HPC since VSCode Server is notorious for leaving behind child processes that bog down cluster performance.

@jamesbeedy jamesbeedy mentioned this issue Nov 24, 2024
Closed
@NucciTheBoss NucciTheBoss added C-slurm Component: Slurm P-medium Priority: Issue needs addressed within a month and removed enhancement labels Feb 4, 2025
@NucciTheBoss
Copy link
Member

Now that we have the apt charm library working on Noble again, we can pick back up on this enhancement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-slurm Component: Slurm P-medium Priority: Issue needs addressed within a month
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

enhancement(slurmctld): Signal children processes jamesbeedy/slurm-charms
2 participants
@jamesbeedy @NucciTheBoss