Project

General

Profile

Feature #17046

Minimum RAM Requirements Warning Message

Added by Joe Schmuck over 3 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Important
Assignee:
Vaibhav Chauhan
Category:
OS
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

I am seeing and have been seeing a lot of people running with low RAM, 2GB for example. We addressed this issue once before and thought there would be a RAM check and then a message stating the computer does not have enough RAM. This message needs to be on the Console and GUI. Could we please get this feature added soon? I know this can't take too much coding to make it happen. I would suggest a minimum limit of 6GB RAM available since 6GB can run a minimally configured system. The warning message should be just that, a warning, not cause a stop of the machine.

install_minmem.diff (1.05 KB) install_minmem.diff Warren Block, 12/08/2016 12:19 PM

Associated revisions

Revision d34369fa (diff)
Added by Sean Fagan about 3 years ago

Check for memory less than 4G. Also allow TrueNAS to be installed under bhyve. Ticket: #17046

Revision a3617bc5 (diff)
Added by Sean Fagan about 3 years ago

Check for memory less than 4G. Also allow TrueNAS to be installed under bhyve. Ticket: #17046

Revision 7cab7967 (diff)
Added by Sean Fagan about 3 years ago

Allow sysctl kern.vm_guest to return bhyve when running under bhyve. Ticket: #17046

Revision 7cab7967 (diff)
Added by Sean Fagan about 3 years ago

Allow sysctl kern.vm_guest to return bhyve when running under bhyve. Ticket: #17046

Revision edf23ae0 (diff)
Added by Sean Fagan about 3 years ago

Check for memory. Ticket: #17046

Revision 694ea6f2 (diff)
Added by Sean Fagan about 3 years ago

Squashed commit of the following: commit edf23ae0210345b9e841a11befc7cd4a06e7e191 Author: Sean Eric Fagan <sef@ixsystems.com> Date: Mon Dec 5 10:29:01 2016 -0800 Check for memory. Ticket: #17046 commit d34369fa79b7f0170544185581dffaebf8ad4536 Author: Sean Eric Fagan <sef@ixsystems.com> Date: Mon Nov 28 13:21:19 2016 -0800 Check for memory less than 4G. Also allow TrueNAS to be installed under bhyve. Ticket: #17046

Revision c2c3c945 (diff)
Added by Sean Fagan almost 3 years ago

Allow sysctl kern.vm_guest to return bhyve when running under bhyve. Ticket: #17046

Revision c2c3c945 (diff)
Added by Sean Fagan almost 3 years ago

Allow sysctl kern.vm_guest to return bhyve when running under bhyve. Ticket: #17046

History

#1 Avatar?id=14398&size=24x24 Updated by Kris Moore over 3 years ago

  • Target version set to 9.10.1-U1

I think this is a reasonable thing. Basically something that just says "FreeNAS Recommends at least <X>Gb of memory. Less than this is unsupported."

Something we can easily do?

#2 Updated by Josh Paetzel over 3 years ago

  • Status changed from Unscreened to Screened

Yes.

#3 Updated by Dru Lavigne over 3 years ago

Should this be passed to William to add to the GUI?

#4 Updated by Sean Fagan over 3 years ago

I'm in the process of rewriting install.sh for both 9.10 and 10 so please don't do it just yet ;).

There's a check for pre_install_check that currently does nothing, and I assume that's where we'd do RAM checks.)

#5 Avatar?id=14398&size=24x24 Updated by Kris Moore over 3 years ago

  • Target version changed from 9.10.1-U1 to 9.10.2

#6 Updated by Josh Paetzel about 3 years ago

  • Assignee changed from Josh Paetzel to Sean Fagan

#7 Avatar?id=14398&size=24x24 Updated by Kris Moore about 3 years ago

Sef, will this be part of your re-write of install.sh? How's that shaping up, will it be in time for 9.10.2 (by end of week?)

#8 Updated by Sean Fagan about 3 years ago

By the end of the week? I could, I suppose, but note that we're talking adding some UI to it, which is so not my forte.

The function is pre_install_check. Checking for hw.physmem being less than, say, 4G is quite possible. It's using dialog that will take me the most time.

#9 Updated by Sean Fagan about 3 years ago

  • Status changed from Screened to 19

a3617bc579cf41c55a610a13ea8e371964b8db2f and branch FIX-17046 in freenas.

#10 Updated by Sean Fagan about 3 years ago

  • Status changed from 19 to Investigation
  • Assignee changed from Sean Fagan to Jakub Klama

Apparently the string is wrong for bhyve. And the syctl variable. Jakub, please adjust it accordingly. (Or, I guess, make the kernel set kern.vm_guest to bhye, as that'd do the same thing.) Same change needs to go into fn9 and fn10.

If all you wan to do is change the line to use "$(sysctl -n hw.hv_vendor)" != "bhyve bhyve", that'd work, but make sure there aren't any annoying trailing spaces or anything. :)

#11 Updated by Sean Fagan about 3 years ago

I've submitted a pull request for os that changes kern.vm_guest to recognize bhyve. Such a change should go into FN10, FN9, and be submitted back to fbsd.

#12 Updated by Sean Fagan about 3 years ago

  • Status changed from Investigation to 19

Oops, incomplete fix in fn9. 694ea6f2b5f628ce587c1c2b1132a075dc549e5f

#13 Avatar?id=14398&size=24x24 Updated by Kris Moore about 3 years ago

  • Status changed from 19 to Reviewed
  • Assignee changed from Jakub Klama to Vaibhav Chauhan

#14 Updated by Warren Block about 3 years ago

This patch parameterizes the minimum value of memory in gigabytes so it can be used in the dialog without repeating literal values. It rewords the message to use the standard GB abbreviation used in the docs and to use shorter, more precise sentences. ("You" might have a lot more than 4 GB of RAM, but this computer doesn't have it.) Also, it breaks up the separate parts of the message onto separate lines. The default option was set to Cancel. Not really sure if the work to return a 1 or 0 is necessary, rather than the plain dialog return codes, so didn't touch that.

Note that we say 8 GB is the recommended minimum value in the user guide.

Attempted to test on 9.10.2, the installer did not call pre_install_check(), possibly due previous setup being needed.

#15 Updated by Sean Fagan about 3 years ago

I've no objection to this change. It may need to go in a separate ticket, but feel free to check it in. Note that it needs to go into both FN9 and FN10 (which uses a different repository for the file).

#16 Updated by Joe Schmuck about 3 years ago

What is the minimum RAM spec you are going with for the warning message? It should not be 8GB but rather just below 8GB in order to allow for any shared RAM the system may have such built in video using shared RAM.

#17 Updated by Sean Fagan about 3 years ago

Currently, as done in this ticket, it warns about 4G, but allows you to continue installation. Warren's patch is just a slight cleanup to it.

#18 Updated by Vaibhav Chauhan about 3 years ago

  • Status changed from Reviewed to Ready For Release

#19 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF