From 15bedfb6889a769e237920c3a1efec0cada233e8 Mon Sep 17 00:00:00 2001 From: Vishal Date: Tue, 20 Mar 2018 14:11:30 -0700 Subject: [PATCH] Warning when updating function from poolmgr type to newdeploy executor type When changing the function executor type from pool manager to new deployment - warning should not be given about ignore environment resources --- fission/function.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fission/function.go b/fission/function.go index 0e1c6090..beb4afca 100644 --- a/fission/function.go +++ b/fission/function.go @@ -510,7 +510,7 @@ func fnUpdate(c *cli.Context) error { default: fatal("Executor type must be one of 'poolmgr' or 'newdeploy', defaults to 'poolmgr'") } - if c.IsSet("mincpu") || c.IsSet("maxcpu") || c.IsSet("minmemory") || c.IsSet("maxmemory") && + if (c.IsSet("mincpu") || c.IsSet("maxcpu") || c.IsSet("minmemory") || c.IsSet("maxmemory")) && fnExecutor == fission.ExecutorTypePoolmgr { warn("CPU/Memory specified for function with pool manager executor will be ignored in favor of resources specified at environment") }