-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Show a setup warning in case S3 object storage is used as primary storage #14603
Conversation
…rage * checks for at least 50 GB of free space Signed-off-by: Morris Jobke <hey@morrisjobke.de>
/backport to stable15 |
/backport to stable14 |
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16884: failureDB=mysql, ENABLE_REDIS=false, PHP=7.3Show full log
DB=mysqlmb4, ENABLE_REDIS=false, PHP=7.3Show full log
TESTS=integration-federation_features
Show full log
TESTS=integration-sharing-v1
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files-sharing-link
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=login
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=users
Show full log
|
The backport to stable15 failed. Please do this backport manually. |
The backport to stable14 failed. Please do this backport manually. |
'core', | ||
'This instance uses an S3 based object store as primary storage. The uploaded files are stored temporarily on the server and thus it is recommended to have 50 GB of free space available in the temp directory of PHP. Check the logs for full details about the path and the available space. To improve this please change the temporary directory in the php.ini or make more space available in that path.', | ||
{ | ||
docLink: oc_defaults.docPlaceholderUrl.replace('PLACEHOLDER', 'admin-mysql-utf8mb4'), |
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.
@MorrisJobke Is the docLink a copy&paste mistake? I don't see a placeholder in the message and linking to 'admin-mysql-utf8mb4' is obviously wrong in this case.
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 for noticing - yes this is an c&p mistake.
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.
Here is the fix: #14620
/backport to stable15 |
backport to stable15 in #14865 |
As discussed @rullzer