Merge pull request #60 from mlucool/patch-1

jtpio · web-flow · commit de920c0aec27 · 2020-07-24T10:58:51.000+02:00
Do not warn if mem_warning_threshold is 0
diff --git a/nbresuse/api.py b/nbresuse/api.py
@@ -36,7 +36,7 @@ async def get(self):
             mem_limit = config.mem_limit
 
         limits = {"memory": {"rss": mem_limit}}
-        if config.mem_limit:
+        if config.mem_limit and config.mem_warning_threshold != 0:
             limits["memory"]["warn"] = (mem_limit - rss) < (
                 mem_limit * config.mem_warning_threshold
             )

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ async def get(self):`
`36`	`36`	`mem_limit = config.mem_limit`
`37`	`37`
`38`	`38`	`limits = {"memory": {"rss": mem_limit}}`
`39`		`- if config.mem_limit:`
	`39`	`+ if config.mem_limit and config.mem_warning_threshold != 0:`
`40`	`40`	`limits["memory"]["warn"] = (mem_limit - rss) < (`
`41`	`41`	`mem_limit * config.mem_warning_threshold`
`42`	`42`	`)`