Repository navigation
Add SLURM executor - #11
Merged
Merged
Conversation
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…d of sacct --array Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…, array headers Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…t codes Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… SLURM Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… for non-terminal states Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…y state and trace polls Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… pending ranges Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…de poll() Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
SlurmExecutor(cluster_api/executors/slurm.py) behind the existingExecutorinterface:sbatch --parsablesubmission,sacctstatus polling,scancelcancellation,squeue+sacctreconnect.docs/superpowers/specs/2026-10-18-slurm-executor-design.mdand the design review alongside it.--no-requeueon every job so preempted/node-failed jobs are terminal;--dependency=afterok --kill-on-invalid-dep=yesfordepends_on;--export=NONE+ in-script exports forinherit_env=False;bash -lshebang forlogin_shell.123_4to the base class's123[4]form sopoll()aggregation is unchanged; compressed pending ranges are expanded withSLURM_BITSTR_LEN=0.exit_code = -signal, matching the local executor.slurm-integrationrunstests/test_slurm_integration.pyon a real single-node SLURM with slurmdbd (koesterlab/setup-slurm-action);CLUSTER_API_REQUIRE_SLURM=1turns the module's skip into a failure so the job cannot pass green by skipping.Test plan
pixi run check(267 passed)slurm-integrationCI job green🤖 Generated with Claude Code