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

Increase default memory limit for cli scripts to prevent runtime memory issues #3462

Closed
rprofijt opened this issue Apr 15, 2020 · 2 comments
Closed
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone

Comments

@rprofijt
Copy link

The default cli memory limit of php in Debian/Ubuntu is -1, in include/cli_check.php however, the limit is overwritten (to 500MB) which causes our cli scripts (poller_reindex) to fail.

@rprofijt rprofijt added bug Undesired behaviour unverified Some days we don't have a clue labels Apr 15, 2020
@TheWitness TheWitness removed the unverified Some days we don't have a clue label Apr 15, 2020
@TheWitness TheWitness added this to the 1.2.12 milestone Apr 15, 2020
@TheWitness
Copy link
Member

Unlimited makes sense to me. The reindex can take a lot of memory.

TheWitness added a commit that referenced this issue Apr 16, 2020
Incorrect default cli php memory_limit
@TheWitness TheWitness added the resolved A fixed issue label Apr 16, 2020
@TheWitness
Copy link
Member

@rprofijt, thanks for reporting. I've opened the flood gates. I think it'll be fine though. Not many systems have less than a dozen or so GB these days.

@netniV netniV added the confirmed Bug is confirm by dev team label Apr 26, 2020
@netniV netniV changed the title Incorrect default cli php memory_limit Increase default memory limit for cli scripts to prevent runtime memory issues May 3, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

3 participants