Skip to content

Commit b9e9836

Browse files
committed
Fix docker_monitor memory usage report
Correct memory calculation. max_mem_percent needs to be divided by 100 to get true percentage.
1 parent 8d85120 commit b9e9836

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

cwltool/job.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -736,7 +736,7 @@ def docker_monitor(self, cidfile, tmpdir_prefix, cleanup_cidfile, process):
736736
except ValueError:
737737
break
738738
_logger.info(u"[job %s] Max memory used: %iMiB", self.name,
739-
int((max_mem_percent * max_mem) / (2 ** 20)))
739+
int((max_mem_percent / 100 * max_mem) / (2 ** 20)))
740740
if cleanup_cidfile:
741741
os.remove(cidfile)
742742

0 commit comments

Comments
 (0)