Skip to content

Commit

Permalink
Fix bug in coordination dir override (#5153)
Browse files Browse the repository at this point in the history
The override of the coordination directory that a user could make was not applied correctly. Instead of setting `self.coordination_dir` the variable `self.workDir` would be modified. This has been fixed.

Co-authored-by: stxue1 <[email protected]>
  • Loading branch information
gmloose and stxue1 authored Dec 2, 2024
1 parent 89d8e53 commit 588492a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/toil/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -424,9 +424,9 @@ def set_option(option_name: str, old_names: Optional[list[str]] = None) -> None:
# Override workDir with value of TOIL_WORKDIR_OVERRIDE if it exists
if os.getenv("TOIL_WORKDIR_OVERRIDE") is not None:
self.workDir = os.getenv("TOIL_WORKDIR_OVERRIDE")
# Override workDir with value of TOIL_WORKDIR_OVERRIDE if it exists
# Override coordination_dir with value of TOIL_COORDINATION_DIR_OVERRIDE if it exists
if os.getenv("TOIL_COORDINATION_DIR_OVERRIDE") is not None:
self.workDir = os.getenv("TOIL_COORDINATION_DIR_OVERRIDE")
self.coordination_dir = os.getenv("TOIL_COORDINATION_DIR_OVERRIDE")

self.check_configuration_consistency()

Expand Down

0 comments on commit 588492a

Please sign in to comment.