Bug 50448 - On the first page system setup should warn if RAM is insufficient
On the first page system setup should warn if RAM is insufficient
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Setup wizard
UCS 4.4
Other Linux
: P5 normal (vote)
: UCS 4.4-2-errata
Assigned To: Julia Bremer
Florian Best
:
Depends on: 36460
Blocks:
  Show dependency treegraph
 
Reported: 2019-11-05 13:45 CET by Sönke Schwardt-Krummrich
Modified: 2019-11-27 14:20 CET (History)
6 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 6: Setup Problem: Issue for the setup process
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 4: A User would return the product
User Pain: 0.137
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:
best: Patch_Available+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2019-11-05 13:45:53 CET
Since UCS 4.0 system setup shows a popup if the memory is insufficient. 
BUT the popup is shown, when entering the summary page, which is too late, because in the meantime system setup fails due to OutOfMemory. It should be shown on the first page (selection of location/language).

Screenshot at Bug 36460: 
https://forge.univention.org/bugzilla/attachment.cgi?id=6771

Additionally the text should be adapted, too.
Comment 1 Ingo Steuwer univentionstaff 2019-11-05 17:33:16 CET
If easier to implement, it would also be OK to show a static page instead of the configuration wizard with a warning.
Comment 2 Florian Best univentionstaff 2019-11-14 14:53:16 CET
A fix has been implemented in git:fbest/50448-system-setup-ram-warning, which:
1. moves the pop up to a page
* It's displayed on page 3, after the language and location selection (so that the message is translated)
2. Changes the "Next" button to a "Ignore" button
3. The product owner decided to also show a link to an SDB article which describes more information. The draft for this has been added in 
https://help.univention.com/t/minimum-ram-requirements-for-the-ucs-installation/13595.
4. The minimum values has already been raised from 512MB to 1GB in the past. Should it be raised even more?
5. The pop up dialog when entering the success page has been removed.
Comment 3 Ingo Steuwer univentionstaff 2019-11-14 15:50:21 CET
(In reply to Florian Best from comment #2)
> 4. The minimum values has already been raised from 512MB to 1GB in the past.
> Should it be raised even more?

@Philipp: I think we need to increase it to ~1,3 GB based on the analysis in EC2?
Comment 4 Erik Damrose univentionstaff 2019-11-14 16:01:28 CET
The updated EC2 image description recommends 2GB of RAM.
Comment 5 Philipp Hahn univentionstaff 2019-11-14 16:11:44 CET
(In reply to Ingo Steuwer from comment #3)
> (In reply to Florian Best from comment #2)
> > 4. The minimum values has already been raised from 512MB to 1GB in the past.
> > Should it be raised even more?
> 
> @Philipp: I think we need to increase it to ~1,3 GB based on the analysis in
> EC2?

1 GiB is bare minimum for just installation, if you really want to do productive work start at 2 GiB.
Comment 6 Julia Bremer univentionstaff 2019-11-19 15:33:48 CET
Successful build
Package: univention-system-setup
Version: 12.0.2-17A~4.4.0.201911191343
Branch: ucs_4.4-0
Scope: errata4.4-2
User: jbremer

Merged branch and built univention-system-setup
Comment 7 Felix Botner univentionstaff 2019-11-19 15:35:05 CET
Check works and the required amount of RAM can be configured with system/setup/boot/minimal_memory.

FAIL: i would change to warning message to something like

_('At least {memory_min} of RAM is required for the installation of {product_name}.') + ' ' +
_('This system only has {memory_total} of RAM.') + ' ' + _('Continuing the installation might lead to a non-functional system.'
_('For further information please look at https://help.univention.com/t/13595.') # unchanged

TODO: our default minimum is 1024M, i think we need to adjust this, 
      @Ingo please decide

TODO: update default minimum in univention-system-setup/www/ApplianceWizard.js
      update default minimum in https://help.univention.com/t/minimum-ram-requirements-for-the-ucs-installation/13595
Comment 8 Ingo Steuwer univentionstaff 2019-11-20 08:48:28 CET
(In reply to Felix Botner from comment #7)
> Check works and the required amount of RAM can be configured with
> system/setup/boot/minimal_memory.
> 
> FAIL: i would change to warning message to something like
> 
> _('At least {memory_min} of RAM is required for the installation of
> {product_name}.') + ' ' +
> _('This system only has {memory_total} of RAM.') + ' ' + _('Continuing the
> installation might lead to a non-functional system.'
> _('For further information please look at
> https://help.univention.com/t/13595.') # unchanged
> 
> TODO: our default minimum is 1024M, i think we need to adjust this, 
>       @Ingo please decide

we discussed this already, UCS currently needs ~1.4G (if there is no SWAP like in EC2 instances). I recommend to define a limit of 1.8G. (if we define 2G we might see the warning in IaaS instances with 2GB as there are a few KB reserved)
 
> TODO: update default minimum in
> univention-system-setup/www/ApplianceWizard.js
>       update default minimum in
> https://help.univention.com/t/minimum-ram-requirements-for-the-ucs-
> installation/13595
Comment 9 Erik Damrose univentionstaff 2019-11-20 09:43:00 CET
> we discussed this already, UCS currently needs ~1.4G (if there is no SWAP
> like in EC2 instances). I recommend to define a limit of 1.8G. (if we define
> 2G we might see the warning in IaaS instances with 2GB as there are a few KB
> reserved)

If i recall correctly there was some logic which reduces the value given in UCR by a factor
Comment 10 Julia Bremer univentionstaff 2019-11-20 10:54:24 CET
(In reply to Erik Damrose from comment #9)
> > we discussed this already, UCS currently needs ~1.4G (if there is no SWAP
> > like in EC2 instances). I recommend to define a limit of 1.8G. (if we define
> > 2G we might see the warning in IaaS instances with 2GB as there are a few KB
> > reserved)
> 
> If i recall correctly there was some logic which reduces the value given in
> UCR by a factor

Correct. 
So I guess it would be fine to set the memory minimum to 2048?

var memory_min = Math.max(parseInt(this.ucr['system/setup/boot/minimal_memory'], 10) || 0, 2048);

// The reported total memory is always a bit lower, than the actually installed amount:
var memory_total_plus_margin = this.values.memory_total * 1.1;

if (memory_min > memory_total_plus_margin) {....
Comment 11 Ingo Steuwer univentionstaff 2019-11-20 11:32:26 CET
(In reply to Julia Bremer from comment #10)
> (In reply to Erik Damrose from comment #9)
> > > we discussed this already, UCS currently needs ~1.4G (if there is no SWAP
> > > like in EC2 instances). I recommend to define a limit of 1.8G. (if we define
> > > 2G we might see the warning in IaaS instances with 2GB as there are a few KB
> > > reserved)
> > 
> > If i recall correctly there was some logic which reduces the value given in
> > UCR by a factor
> 
> Correct. 
> So I guess it would be fine to set the memory minimum to 2048?

OK for me!
Comment 12 Julia Bremer univentionstaff 2019-11-20 11:35:23 CET
2158b18184 Bug #50448: yaml
0059a92982 Bug #50448: Adjust minum RAM to 2GiB and adjust warning message


Package: univention-system-setup
Version: 12.0.2-18A~4.4.0.201911201132
Branch: ucs_4.4-0
Scope: errata4.4-2


Adjusted the warning message and set the minimal memory to 2GiB
Comment 13 Florian Best univentionstaff 2019-11-20 12:26:02 CET
OK: adjusted warning message
OK: adjusted minimum RAM requirement
OK~: YAML
Comment 14 Arvid Requate univentionstaff 2019-11-27 14:20:07 CET
<http://errata.software-univention.de/ucs/4.4/381.html>