Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/issues/5134-ftp-get-size' into i…
Browse files Browse the repository at this point in the history
…ssues/5134-ftp-get-size
  • Loading branch information
stxue1 committed Nov 14, 2024
2 parents a2105af + d2da46c commit 66d3587
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion requirements-cwl.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
cwltool==3.1.20241024121129
cwltool==3.1.20241112140730
schema-salad>=8.4.20230128170514,<9
galaxy-tool-util<25
galaxy-util<25
Expand Down
2 changes: 1 addition & 1 deletion requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ types-aws-xray-sdk
types-boto<2.49.18.20241020
types-Flask-Cors
types-requests
types-psutil
types-psutil>=6.1.0.20241102
types-python-dateutil
types-setuptools
types-xmltodict
Expand Down
7 changes: 6 additions & 1 deletion src/toil/test/cwl/cwlTest.py
Original file line number Diff line number Diff line change
Expand Up @@ -526,7 +526,12 @@ def test_slurm_node_memory(self) -> None:
log.debug("Workflow output: %s", out)
memory_string = out["memory"]
log.debug("Observed memory: %s", memory_string)
result = int(memory_string)
# If there's no memory limit enforced, Slurm will return "unlimited".
# Set result to something sensible.
if memory_string.strip() == "unlimited":
result = 4 * 1024 * 1024
else:
result = int(memory_string)
# We should see more than the CWL default or the Toil default, assuming Slurm nodes of reasonable size (3 GiB).
self.assertGreater(result, 3 * 1024 * 1024)

Expand Down

0 comments on commit 66d3587

Please sign in to comment.