-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Adds a check that errors on store start if MariaDB is detected #3725
Conversation
@mgdelacroix this feels like we're telling customers MariaDB is officially dropped as an unofficially supported database. In other words, we're saying if you're on MariaDB you can't use Mattermost going forward. Is that really what we want? |
We already say that here - https://docs.mattermost.com/install/software-hardware-requirements.html#database-software
|
We could tone down the error to say something less "radical" like The effect will still be that Boards is not going to work as of now, but they'll have a pointer to follow besides the error message. |
@mgdelacroix I guess adding the link sbishel posted (https://docs.mattermost.com/install/software-hardware-requirements.html#database-software) would give the user some info. |
Error message updated on 59d6739 |
/update-branch |
/cherry-pick release-7.2 |
Cherry pick is scheduled. |
…rmost-community#3725) * Adds a check that errors on store start if MariaDB is detected * Fix linter * Update error message Co-authored-by: Mattermod <mattermod@users.noreply.github.com> (cherry picked from commit e1a39e5)
Summary
These changes causes the app to error on start if the database is detected to be MariaDB