Skip to content

Commit db2fe27

Browse files
committed
Revert "[Shutdown] Kill slurmd and slurmstepd before shutting down to prevent shutdown hanging on these running processes, which is an unexpected behaviour observed in Ubuntu24.04."
This reverts commit 50a682f.
1 parent 50a682f commit db2fe27

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ This file is used to list changes made in each version of the aws-parallelcluste
77
------
88

99
**CHANGES**
10-
- Kill slurmd and slurmstepd before shutdown to prevent the shutdown hanging on those processing be running.
10+
- There were no changes for this version.
1111

1212
3.12.0
1313
------

src/slurm_plugin/computemgtd.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,10 +131,6 @@ def _self_terminate():
131131
# Sleep for 10 seconds so termination log entries are uploaded to CW logs
132132
log.info("Preparing to self terminate the instance in 10 seconds!")
133133
time.sleep(10)
134-
log.info("Killing slurm processes")
135-
# TOFIX WORKAROUND: We kill Slurm processes because we observed in 3.13.0 on Ubuntu24.04
136-
# that the shutdown hangs waiting for these processes to terminate.
137-
run_command("sudo killall -9 --quiet slurmd slurmstepd")
138134
log.info("Self terminating instance now!")
139135
run_command("sudo shutdown -h now")
140136

0 commit comments

Comments
 (0)