Bug 843621
Summary: | engine-setup does not complete if RAM = 2GB | ||
---|---|---|---|
Product: | [Retired] oVirt | Reporter: | Cybertimber2011 <cybertimber2000> |
Component: | ovirt-engine-installer | Assignee: | Ofer Schreiber <oschreib> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | low | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.1 RC | CC: | acathrow, alourie, asegundo, eedri, iheim, jkt |
Target Milestone: | --- | Keywords: | Patch |
Target Release: | 3.3 | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | integration | ||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-09-23 07:35:34 UTC | 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: |
Description
Cybertimber2011
2012-07-26 18:58:25 UTC
Same issue here. Patch sent: http://gerrit.ovirt.org/6992 Thanks for working on that. I think this should make it a lot easier to avoid that error now. Thank you for reporting. Merged into master upstream; http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=158b29b6d720417eb371a8ad9bfabf824c36d87c This doesn't seem to have made it into 3.2 (or if it did, it's not working). Any chance this can make it? the way i read the patch, it will allow installing on a machine with 0.95*2048MB of free memory, not available memory. i think the request here is to allow installing on a machine with 2048-128 (say, video card) MB of installed memory, not available memory. availableMemory variable has the installed memory value. http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=blob;f=packaging/fedora/setup/engine-setup.py#l2176 Take my system as example: >>> print output total used free shared buffers cached Mem: 3881 3666 214 0 0 1928 -/+ buffers/cache: 1738 2143 Swap: 3887 300 3587 >>> for line in output.split("\n"): ... result = re.match("Mem:\s+(\d+)\s+.+", line) ... if result: ... availableMemory = result.group(1) ... >>> print availableMemory 3881 Definitely included in the 3.2. If there is any issue with it, please let us know. My apologies, I thought I only had the beta repo enabled, so when I ran "yum install ovirt-engine", it installed 3.1 from the Fedora repo and I didn't catch it until I went to the mailing list. Now that I have ovirt-engine-3.2 installed, I'll do some further testing. So far looks good but free -g did return 2GB so moot point. I'll shortly change to ~2000MB and see what happens. when using a virtual machine to install ovirt, and specifying 4096MB as memory, setup will fail currently since the actual memory on host is less (~3800). so it should at least allow installing on a system with 3800. 3800 isn't enough based on the patch. 3800 = 92.7% of 4096. Amador made it 95%. What's taking up almost 300MB (almost 10%) of your allocated RAM? What does "free -m" return? here is the vm output: (defined in engine as 4096mb) free -m total used free shared buffers cached Mem: 3831 3329 502 0 294 1261 -/+ buffers/cache: 1773 2057 Swap: 15999 29 15970 i would assume the delta is taken by qemu? for virt purposes? but i can't tell for sure. iirc 4096 is the warning limit. fail limit is 2048. changed after patch merged: http://gerrit.ovirt.org/#/c/13201/ big jump btw. 5% was supposed to be enough to most of the cases. New 4096MB limit is raising tolerance up to ~204MB, a considerable value imo. It's not a strong opinion, but I think we shouldn't bite the limit more than we currently do. And manual basedefs change is always possible for those who aren't able to meet the requirements. as RC is built, moving to ON_QA (hopefully did not catch incorrect bugs when doing this) Some feedback on this, based on 3.3 RC1. Looks good! The minimum RAM changed to 4096 (as Amador said), but the installer prompts the user with the minimum and recommended values, and asks if you want to install with less. Nice touch! closing as this should be in 3.3 (doing so in bulk, so may be incorrect) |