Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to change memory limits on yacy-search snap #269

Closed
lucyllewy opened this issue Jan 8, 2019 · 1 comment
Closed

Unable to change memory limits on yacy-search snap #269

lucyllewy opened this issue Jan 8, 2019 · 1 comment

Comments

@lucyllewy
Copy link

I've installed and tested yacy-search as a snap but am hitting memory limit issues. I tried raising the limit but it seems that the startup script is ignoring the settings in favour of the default values (600MB).

Looking at the startup script yacy/startYACY.sh it appears that it tries to load the $CONFIGFILE from a relative path DATA/SETTINGS/yacy.conf, the parent of which is set by cd "`dirname $0`". As $0 references the startup script and this is inside the snap the relative path does not exist. The snap-specific preload script yacy/addon/snap/snapStartYACY.sh appears to believe it can set the data path but this overridden path is not used when setting the memory limits in the startup script.


Like @popey who reported #254 I work with snaps, though as a community contributor not sponsored/employed by Canonical where Popey is.

@luccioman
Copy link
Member

Thank you @diddledan for reporting this.

This was indeed the startYACY.sh script that did not read the configuration file at the proper path when explicitly passing a custom data path with -s or -g option, as done in the snap package.

The fix is published on the yacy-search snap, so you can refresh and check it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants