-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🐛 Bug Report: migration fails with Allowed memory size exhausted php memory_limit hardcoded to 512 #5528
Comments
@bfritscher thanks for raising this issue! 🙏🏼 Interesting challenge... 🧐 For now, I would suggest either modifying this: Line 15 in 181741d
before running the migrate command. |
@bfritscher, did the suggestion work for you? |
@stnguyen90 Thanks for following up. Yes, it is the workaround I described (and used) in my initial report, as well as the possible solution which would be to replace the hardcoded value with an environment variable. Another would be to identify which part of the migration process uses too much memory. |
Hi, @stnguyen90 the fix is only the workaround? Or do you plan to implement any of the above suggestions or any other fix? |
👟 Reproduction steps
Run a migration fails upgrading audit data, with out of memory
👍 Expected behavior
Expect the migration to work.
Or at least be able to specify a higher php memory_limit via environnment without the need to edit the app/init.php hardcoded value:
ini_set('memory_limit', '512M');
👎 Actual Behavior
fails with php out of memory error
🎲 Appwrite version
Version 1.3.x
💻 Operating system
Linux
🧱 Your Environment
docker self hosted
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: