-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Mandate minimum requirements for ram/cpu #3275
Mandate minimum requirements for ram/cpu #3275
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #3275 +/- ##
==========================================
+ Coverage 98.02% 99.01% +0.98%
==========================================
Files 3 3
Lines 203 203
==========================================
+ Hits 199 201 +2
+ Misses 4 2 -2 ☔ View full report in Codecov by Sentry. |
Would be great to record the reasoning for this change somewhere |
I'm using a server (Hetzner) with 16GB of RAM, but a few MB are missing. The install script fails with:
Would be great to decrease the limit a little bit. |
@joshuarli is it possible to reduce the limit? Just like @lukasleitsch, my instance hosted in AWS it's supposed to be 16GB of RAM, but it's a little below of the imposed limit. Well, I changed the file manually inside the EC2 to make it work. |
Yes, sorry. Thanks for the feedback! #3290 |
get rid of soft requirements for ram/cpu, enforce this as a hard requirement instead.
16 GB of ram, 4 CPU cores