Skip to content

Fix condor runner after stage budget removal - #136

Merged
twybo merged 1 commit into
mainfrom
fix/condor
Sep 22, 2026
Merged

twybo merged 1 commit into
mainfrom
fix/condor

Conversation

@twybo

@twybo twybo commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Fix condor_runner.py after the shared stage-out budget was removed. The runner still imported and called the deleted begin_stage_budget, which prevented it from starting.

This change:

  • Removes the stale import and invocation.
  • Preserves five independent stage-out attempts for each output.
  • Adds regression tests for runner imports, retry behavior, stage-out failures, and fatal LFN preflight failures.
  • Updates the validation guide to match the current GFAL and filesystem publication behavior.

Verification

  • tests/test_stage_out.py: 33 passed
  • tests/test_merge_planning.py: 27 passed
  • Full test suite: 60 passed
  • condor_runner import smoke test: passed

@twybo
twybo requested a review from jniedzie September 22, 2026 10:53
@twybo
twybo merged commit 7851af5 into main Sep 22, 2026
4 checks passed
@twybo
twybo deleted the fix/condor branch September 22, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants