Skip to content

Commit d220820

Browse files
committed
refactor: changed to only signalling, no distinction between scopes
1 parent 8f85dcd commit d220820

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

snakemake_executor_plugin_slurm/validation.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -223,21 +223,6 @@ def validate_executor_settings(settings, logger=None):
223223
# signal
224224
parse_slurm_signal_settings(settings.signal)
225225

226-
# Validate that if any signal uses reservation scope (R:), a reservation is
227-
# configured
228-
if settings.signal:
229-
parsed_signals = parse_slurm_signal_settings(settings.signal)
230-
for rule_name, signal_spec in parsed_signals.items():
231-
# signal_spec format: [B:|R:]<sig_num>@<time>
232-
if signal_spec.startswith("R:"):
233-
if not settings.reservation:
234-
raise WorkflowError(
235-
f"Signal for rule '{rule_name}' targets reservation (R:) "
236-
"but no --slurm-reservation is configured. "
237-
"Either specify a reservation or change the signal to use "
238-
"batch (B:) scope."
239-
)
240-
241226
# status_command warnings (optional logger)
242227
if logger:
243228
validate_status_command_settings(settings, logger)

0 commit comments

Comments
 (0)