Univention Bugzilla – Full Text Bug Listing |
Summary: | On the first page system setup should warn if RAM is insufficient | ||
---|---|---|---|
Product: | UCS | Reporter: | Sönke Schwardt-Krummrich <schwardt> |
Component: | UMC - Setup wizard | Assignee: | Julia Bremer <bremer> |
Status: | CLOSED FIXED | QA Contact: | Florian Best <best> |
Severity: | normal | ||
Priority: | P5 | CC: | best, botner, bremer, damrose, hahn, steuwer |
Version: | UCS 4.4 | Flags: | best:
Patch_Available+
|
Target Milestone: | UCS 4.4-2-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
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: | |||
Bug Depends on: | 36460 | ||
Bug Blocks: |
Description
Sönke Schwardt-Krummrich
2019-11-05 13:45:53 CET
If easier to implement, it would also be OK to show a static page instead of the configuration wizard with a warning. 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. (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? The updated EC2 image description recommends 2GB of RAM. (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. 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 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 (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 > 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
(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) {.... (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! 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 OK: adjusted warning message OK: adjusted minimum RAM requirement OK~: YAML |