Remove duplicate ReturnToService line in /etc/slurm/slurm.conf

[mikerenfro]

Fixes #1991

[github-actions]

Test Results

14 files   -  20  14 suites   - 20   6s ⏱️ - 3m 0s
41 tests  - 184  41 ✅  - 184  0 💤 ±0  0 ❌ ±0 
54 runs   - 184  54 ✅  - 184  0 💤 ±0  0 ❌ ±0 

Results for commit a467269. ± Comparison against base commit 969681a.

This pull request removes 184 tests.

instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C binary includes expected Score-P instrumenter symbols (/gnu13/mpich)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C binary includes expected Score-P instrumenter symbols (/gnu13/openmpi5)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C binary includes expected Score-P instrumenter symbols (/intel/mpich)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C binary includes expected Score-P instrumenter symbols (/intel/openmpi5)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C++ binary includes expected Score-P instrumenter symbols (/gnu13/mpich)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C++ binary includes expected Score-P instrumenter symbols (/gnu13/openmpi5)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C++ binary includes expected Score-P instrumenter symbols (/intel/mpich)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) C++ binary includes expected Score-P instrumenter symbols (/intel/openmpi5)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) Fortran binary includes expected Score-P instrumenter symbols (/gnu13/mpich)
instrumenter_test ‑ [perf-tools/Score-P] Hybrid (MPI+OpenMP) Fortran binary includes expected Score-P instrumenter symbols (/gnu13/openmpi5)
…

[adrianreber]

Unfortunately our CI is not clever enough to detect that your change means that a rebuild of slurm is required to test the PR. Test builds in CI are only triggered if the actual spec file is changed. The changed patch does not trigger a rebuild.

Trying it out locally your PR fails building. Following change on top helps:

-@@ -149,5 +149,22 @@
+@@ -149,5 +149,20 @@

Also, we require patches to be signed-off-by as described in https://github.com/openhpc/ohpc/blob/3.x/CONTRIBUTING.md

Fixing the patch with what I posted and adding the correct sign off should make this ready to be merged.

[github-actions]

A friendly reminder that this PR had no activity for 30 days.