diff --git a/src/client/cmdVM/vmEdit.go b/src/client/cmdVM/vmEdit.go index b57a51dc7311f4f2dbc753ee529be4303a1dc69b..4710b2ceacaf93b6c248b2cf018cbabe071a6c20 100644 --- a/src/client/cmdVM/vmEdit.go +++ b/src/client/cmdVM/vmEdit.go @@ -99,7 +99,7 @@ func (cmd *Edit)Run(args []string) int { } func (cmd *Edit)parseArgs(args []string) (*params.VMEdit, []string, error) { - vmParams := ¶ms.VMEdit {} + vmParams := ¶ms.VMEdit { Cpus: 0, Ram: 0 } var patterns []string atLeastOneArg := false @@ -121,10 +121,9 @@ func (cmd *Edit)parseArgs(args []string) (*params.VMEdit, []string, error) { s = getStringVal(arg, "cpus=") if s != "" { cpus, err := strconv.Atoi(s) - if err != nil { - return nil, nil, errors.New("Invalid number of CPU(s)") + if err != nil || cpus < 1 { + return nil, nil, errors.New("Error: invalid number of CPU(s)!") } - vmParams.CpusUpdated = true vmParams.Cpus = cpus atLeastOneArg = true continue @@ -132,10 +131,9 @@ func (cmd *Edit)parseArgs(args []string) (*params.VMEdit, []string, error) { s = getStringVal(arg, "ram=") if s != "" { ram, err := strconv.Atoi(s) - if err != nil { - return nil, nil, errors.New("Invalid amount of RAM") + if err != nil || ram < 1 { + return nil, nil, errors.New("Error: invalid amount of RAM!") } - vmParams.RamUpdated = true vmParams.Ram = ram atLeastOneArg = true continue diff --git a/src/client/version/version.go b/src/client/version/version.go index 9ca6a9135482bcca2e52e0a72595da6ae3e767a9..4877ef2a89f3b9d1ed956c110b72049168074c64 100644 --- a/src/client/version/version.go +++ b/src/client/version/version.go @@ -7,8 +7,8 @@ import ( const ( major = 1 - minor = 8 - bugfix = 10 + minor = 9 + bugfix = 0 ) type Version struct { diff --git a/src/common/params/vms.go b/src/common/params/vms.go index bc5cabf87009e9ca27fbadbf2a1edde8a0e81dc3..5af3ed73dc109cc9dafd86a9a74865b8d7a4a09a 100644 --- a/src/common/params/vms.go +++ b/src/common/params/vms.go @@ -26,8 +26,6 @@ type VMEdit struct { Ram int `json:"ram" validate:"required,gte=256,lte=32768"` Nic vm.NicType `json:"nic" validate:"required` UsbDevs []string `json:"usbDevs" validate:"required` - CpusUpdated bool - RamUpdated bool } type VMAddAccess struct { diff --git a/src/server/version/version.go b/src/server/version/version.go index 66b06f850313ce3b086147ec28466df22ff39838..a9c0dced03066790a6b623e10b0f4df336fb6a09 100644 --- a/src/server/version/version.go +++ b/src/server/version/version.go @@ -6,8 +6,8 @@ import ( const ( major = 1 - minor = 8 - bugfix = 8 + minor = 9 + bugfix = 0 ) type Version struct { diff --git a/src/server/vms/vms.go b/src/server/vms/vms.go index 8ddf6c8750eea3d27755d751e1d6f900490498a1..dbba5a53231c54fe0ed4e3dd45333cab2c18cdb0 100644 --- a/src/server/vms/vms.go +++ b/src/server/vms/vms.go @@ -555,10 +555,10 @@ func (vms *VMs)EditVM(vmID uuid.UUID, p *params.VMEdit) error { if p.Name != "" { vm.v.Name = p.Name } - if p.CpusUpdated { + if p.Cpus > 0 { vm.v.Cpus = p.Cpus } - if p.RamUpdated { + if p.Ram > 0 { vm.v.Ram = p.Ram } if p.Nic != "" {