Univention Bugzilla – Full Text Bug Listing |
Summary: | MinPhysicalRAM should be tested only when installing a app, not during an update | ||
---|---|---|---|
Product: | UCS | Reporter: | Felix Botner <botner> |
Component: | UMC - App-Center | Assignee: | Dirk Wiesenthal <wiesenthal> |
Status: | CLOSED FIXED | QA Contact: | Alexander Kläser <klaeser> |
Severity: | normal | ||
Priority: | P3 | CC: | gohmann, gulden, jmm |
Version: | UCS 3.1 | ||
Target Milestone: | UCS 3.1-1-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
What kind of report is it?: | --- | What type of bug is this?: | --- |
Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
User Pain: | 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: | |||
Bug Blocks: | 31856 |
Description
Felix Botner
2013-05-14 13:33:32 CEST
I think we should change it more generally. Every ISV misunderstood the value. Currently we check the real free memory, I think we should rather check the available memory in the system. And it should be possible to continue even if the system has less memory. (In reply to Stefan Gohmann from comment #1) > I think we should change it more generally. Every ISV misunderstood the > value. > > Currently we check the real free memory, I think we should rather check the > available memory in the system. And it should be possible to continue even > if the system has less memory. If we only check for the available memory we do not prevent installing every single app in the app center with only 1024 MB RAM. Or did I misunderstand the sentence? If we would do that, the whole check should be removed completely as it does not make much sense any more. We _could_ somehow check for the difference of MinPhysicalRAM during an update if necessary. But if I remember the code correctly this might get hacky or dev-time consuming... I like the idea with a simple warning not preventing the installation/update. This could be shown even during the update of OX when the old OX consumed all the RAM. Fixed in univention-management-console-module-appcenter 2.0.139-14.143.201306281536 and univention-management-console-module-appcenter 3.0.1-1.144.201306281606 YAML: 2013-06-28-univention-management-console-module-appcenter.yaml Changelog updated For testing I reserved 1.5GB of memory via python: import array a = array.array('d', xrange(int(1.5 * 1024 * 1024 * 1024 / 8))) Erratum changes → OK YAML file → OK 3.2 changes → OK 3.2 changelog → OK |