Bug 1014952

Summary: [admin portal] cannot add Windows VM with 512MB memory, or more than 64GB
Product: Red Hat Enterprise Virtualization Manager Reporter: Jiri Belka <jbelka>
Component: ovirt-engine-webadmin-portalAssignee: Roy Golan <rgolan>
Status: CLOSED CURRENTRELEASE QA Contact: Ilanit Stein <istein>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 3.3.0CC: acathrow, ecohen, iheim, istein, mavital, michal.skrivanek, Rhev-m-bugs, s.kieske, srevivo, yeylon
Target Milestone: ---Keywords: Regression
Target Release: 3.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: virt
Fixed In Version: is24 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1026487, 1038284    

Description Jiri Belka 2013-10-03 07:57:33 UTC
Description of problem:

In 3.2 I can create Windows 7 64bit with 512Mb of RAM. But in 3.3 you force me to use 1024!

-%-
Error while executing action:

test-tz:

    Cannot edit VM. Illegal memory size is provided, size needs to be between 1024 MB and 64000 MB.
-%-

Regression for Windows VM in comparison with 3.2.

<fyi>
Anyway, the admin is boss here and he should have possibility to define what he wants in supported range of qemu. This is not any freedom to force the user to use minimal 1024 Mb for Windows (or another value for different OS)...

Ridiculous, it allows me to set 256 Mb for RHEL6 64bit, but the installer fails because of not much memory. So I the ideas was to meet vendors' RAM recommendations, then there's inconsistency :-)

If you have decided to have some minimal memory for an OS type, 'New VM' dialog should be populated with "your" recommendation of memory but the admin should still have an opportunity to go below (if it is supported by qemu).

Comparison with VMWare product: I could create W2k12 VM, it created for me a VM with 2048 Mb of RAM. But after that I could put 128 Mb! Commercial software!
</fyi>

Version-Release number of selected component (if applicable):
is17

How reproducible:
100%

Steps to Reproduce:
1. Create a W7 64bit VM in 3.2 with 512 Mb
2. Export the VM
3. Create a W7 64bit VM in 3.3 with 512 Mb
4. Import the VM into 3.3
5. Edit this new imported VM (or just click OK)

Actual results:
impossible

Expected results:
should be possible, otherwise I suppose customer after upgrade would not be able to even change anything in VM properties.

Additional info:
There's bug that import mangles OS type of the imported VM. Thus do something like this to change the Os type to Windows:

update vm_static set os = 12 where vm_name = 'test-tz';

Comment 2 Michal Skrivanek 2013-10-04 17:13:02 UTC
we should only warn, or maybe even just a tooltip, but not block on other memory sizes...

[fixing flags]

Comment 3 Pavel Novotny 2013-10-29 16:19:16 UTC
FailedQA in rhevm-3.3.0-0.28.beta1.el6ev.noarch (is20).

Verification steps were done according to reproducer in comment#0. 
Optional approach: In 3.3 try to create a Windows 7 x64 VM with RAM less than 1024 MB or greater than 64000 MB. 

Result is the same - error dialog: Cannot add VM. Illegal memory size is provided, size needs to be between 1024 MB and 64000 MB.

Comment 4 Shai Revivo 2013-11-03 09:10:30 UTC
in build 21

Comment 5 Ilanit Stein 2013-11-05 10:19:33 UTC
Verified on is21.

Try to create a Windows 7 x64 VM with RAM less than 1024 MB or greater than 64000 MB - and see it succeeds.

Comment 6 Ilanit Stein 2013-11-05 11:58:42 UTC
correction to comment #5: I checked less than 512M, not 1024M

Comment 7 Roy Golan 2013-11-12 11:30:18 UTC
Ilanit I'm re-opening this. I'm fixing this to write an Event log and not a the file log so the automation could be fixed to check the event log

Comment 8 Ilanit Stein 2013-11-12 15:24:20 UTC
Please consider returning memory min\max validation. 
It is not always clear from logs, why it fails to run. 
For example, Boot from pxe of a VM with 1M memory: VM turn into pause mode, and there was no clear error on vdsm log, pointing that the root cause was the small size of VM memory.
It seem more reasonable to me, to stop user at the point of creating the VM, than later on let him handle the consequences of the wrong memory size. 
Also warning in the audit log, might be not noticed.

Ilanit.

Comment 9 Jiri Belka 2013-11-12 15:34:32 UTC
I don't agree. It's PEBKAC problem. Software should not always try to detect if a user put stupid settings. But if a memory value is accepted by UI and not by qemu, then this is different case. UI should not allow to accept lower mem value than qemu could accept.

Comment 10 Ilanit Stein 2013-11-13 11:24:38 UTC
We can also add comment near the VM memory size in UI like: 
"Recommended values 256mb - 64,000mb for Other OS"

Comment 11 Ilanit Stein 2013-11-21 13:48:08 UTC
Verified on is24.1

Comment 12 Michal Skrivanek 2013-11-21 16:38:05 UTC
(In reply to Ilanit Stein from comment #10)
> We can also add comment near the VM memory size in UI like: 
> "Recommended values 256mb - 64,000mb for Other OS"

yeah, that would be nice a nice improvement

Comment 13 Itamar Heim 2014-01-21 22:24:59 UTC
Closing - RHEV 3.3 Released

Comment 14 Itamar Heim 2014-01-21 22:25:41 UTC
Closing - RHEV 3.3 Released

Comment 15 Itamar Heim 2014-01-21 22:29:14 UTC
Closing - RHEV 3.3 Released