-
Notifications
You must be signed in to change notification settings - Fork 17
Add ram settings validation and flags to disable it #31
Add ram settings validation and flags to disable it #31
Conversation
Change-type: minor
Change-type: patch
Change-type: patch
Change-type: patch
This comment from the related issue is also relevant here |
Change-type: patch
@balena-ci retest |
boinc-client/start.sh
Outdated
fi | ||
|
||
totalmem=$(awk '/^MemTotal:/{print $2}' /proc/meminfo) | ||
|
||
if [ "$totalmem" -lt "1500000" ]; then | ||
if [[ -z $SKIP_BOINC_CPU_SETTINGS_CHECK && "$totalmem" -lt "2500000" ]]; then | ||
echo "Less than 1.5GB RAM - running single concurrent task" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be 2.5GB?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True. fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks :)
Change-type: patch
@balena-ci retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested working on the pi4 1GB version. Thanks for your contribution :)
+1 thanks for the great contribution @ptrm |
Fixes #29
There's also check for
SKIP_BOINC_MEM_SETTINGS_CHECK
andSKIP_BOINC_CPU_SETTINGS_CHECK
environment variables to disable the auto-correction of config values.Adds ca. 5MB to boinc-client image because of adding xml-twig-tools to read config values.