From 5a4461b07fc2095228b71922606e4a1875852995 Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Sat, 18 May 2019 08:41:05 +0200 Subject: pkg/instance: fix types in OverrideVMCount vmConfig value is interface{}, not string. So compare with nil and assign int. --- pkg/instance/instance.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'pkg/instance/instance.go') diff --git a/pkg/instance/instance.go b/pkg/instance/instance.go index c2556bcd9..960272245 100644 --- a/pkg/instance/instance.go +++ b/pkg/instance/instance.go @@ -136,10 +136,10 @@ func OverrideVMCount(cfg *mgrconfig.Config, n int) error { if err := json.Unmarshal(cfg.VM, &vmConfig); err != nil { return fmt.Errorf("failed to parse VM config: %v", err) } - if vmConfig["count"] == "" || !vm.AllowsOvercommit(cfg.Type) { + if vmConfig["count"] == nil || !vm.AllowsOvercommit(cfg.Type) { return nil } - vmConfig["count"] = fmt.Sprint(n) + vmConfig["count"] = n vmCfg, err := json.Marshal(vmConfig) if err != nil { return fmt.Errorf("failed to serialize VM config: %v", err) -- cgit mrf-deployment