-
Notifications
You must be signed in to change notification settings - Fork 585
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
Deployment documentation updates #7731
Merged
+415
−659
Merged
Changes from 12 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
b720fd1
Base updates for docs
nickmisasi 1134c98
Adjustments to rhel
nickmisasi 6dd1d0b
Fix warning
nickmisasi 7e0a17e
fix conflicts
nickmisasi 39bb3de
Merge branch 'master' into install-docs-updates
cwarnermm a77022b
Fixed build warnings & errors
cwarnermm b60f894
Add support email snippet, and new work around install-kubernetes docs
nickmisasi 1681a69
Merge branch 'install-docs-updates' of github.com:mattermost/docs int…
nickmisasi e669e4e
fix conflicts
nickmisasi e6b0c1c
Fix a few things
nickmisasi 9d7c374
More fixes
nickmisasi d3f229e
Merge branch 'master' into install-docs-updates
cwarnermm 9bf8a5a
Added redirects, editorial pass
cwarnermm 40823d2
Merge branch 'master' into install-docs-updates
cwarnermm ddff5a0
Merge branch 'master' into install-docs-updates
cwarnermm 31e3535
Merge branch 'master' into install-docs-updates
cwarnermm b7a8369
Remove docker production deployment docs
nickmisasi 240257a
Fix conflicts
nickmisasi 98198ef
Merge branch 'master' into install-docs-updates
cwarnermm 8a5cb6d
Merge branch 'master' into install-docs-updates
cwarnermm 0e493ef
Editorial updates
cwarnermm 9752302
Merge branch 'master' into install-docs-updates
cwarnermm e6a787d
Editorial updates
cwarnermm File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
:orphan: | ||
:nosearch: | ||
|
||
.. note:: | ||
We recommend configuring the `Support Email <https://docs.mattermost.com/administration/config-settings.html#support-email>`_ under ``SupportSettings``, set ``"SupportEmail"``. This is the email address your users will contact when they need help. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
:orphan: | ||
:nosearch: | ||
|
||
Configure the following properties in this file: | ||
|
||
* Under ``SqlSettings``, set ``DriverName`` to ``"postgres"``. This is the default and recommended database for all Mattermost installations. | ||
* Under ``SqlSettings``, set ``DataSource`` to ``"postgres://mmuser:<mmuser-password>@<host-name-or-IP>:5432/mattermost?sslmode=disable&connect_timeout=10"`` replacing ``mmuser``, ``<mmuser-password>``, ``<host-name-or-IP>`` and ``mattermost`` with your database name. | ||
* Under ``ServiceSettings``, set ``"SiteURL"``: The domain name for the Mattermost application (e.g. ``https://mattermost.example.com``). |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
:orphan: | ||
:nosearch: | ||
|
||
.. important:: | ||
|
||
The GPG public key has changed. You can `import the new public key <https://deb.packages.mattermost.com/pubkey.gpg>`_ or run the automatic Mattermost PPA repository setup script provided below. Depending on your setup, additional steps may also be required, particularly for installations that didn't rely on the repository setup script. We recommend deleting the old key from ``/etc/apt/trusted.gpg.d`` before adding the apt repository. | ||
|
||
- For Ubuntu Focal - 20.04 LTS: | ||
|
||
``sudo apt-key del A1B31D46F0F3A10B02CF2D44F8F2C31744774B28`` | ||
|
||
``curl -sL -o- https://deb.packages.mattermost.com/pubkey.gpg | gpg --dearmor | sudo apt-key add`` | ||
|
||
- For Ubuntu Jammy - 22.04 LTS and Ubuntu Noble - 24.04 LTS: | ||
|
||
``sudo rm /usr/share/keyrings/mattermost-archive-keyring.gpg`` | ||
|
||
``curl -sL -o- https://deb.packages.mattermost.com/pubkey.gpg | gpg --dearmor | sudo tee /usr/share/keyrings/mattermost-archive-keyring.gpg > /dev/null`` |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
:orphan: | ||
:nosearch: | ||
|
||
.. tip:: | ||
|
||
If you are running the Mattermost Server and database on a single system, we recommend the :doc:`Mattermost Omnibus install method </install/installing-mattermost-omnibus>` as this greatly reduces setup and ongoing maintenance. | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
:orphan: | ||
:nosearch: | ||
|
||
.. important:: | ||
|
||
You need a PostgreSQL database. See the :doc:`database preparation </install/prepare-mattermost-database>` documentation for details on this prerequisite. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I do not think we should even consider this as an option for Production use .
It's best to redirect them to omnibus documentation and clearly specify that this is a deprecated way to deploy Mattermost and unmaintained .
Below we have a note stating that the repo is deprecated , we should not provide this as a viable production solution.
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.
I've had some discussions around this as well, and I tend to agree with you. That being said, I'm wondering if we should keep it (but perhaps add a "this is no longer recommended" note) as there are likely a bunch of customers using this method right now?
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.
@toninis that's not the up-to-date docker repo, that's the old one which caused tons of issues, hence the deprecation and complete rewrite. the very much not deprecated docker repo is at https://github.com/mattermost/docker
imo we should get rid of that note to avoid confusion, like here, and possibly even just get rid of the repo altogether. at this point it's been years since it was archived/deprecated.
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.
Docker can be for production only in cases like ECS (AWS) and Azure Container Apps. Do we have any idea if there's any deployment like this?
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.
@spirosoik @toninis @svelle -- what's the right way forward here? We need a decision to unblock this section of the docs.
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.
@spirosoik - How do we want to proceed based on this input? CC @nickmisasi
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 @coltoneshaw @victorehernandez. Just to summarise, so we are all in agreement. Docker can be in production but with not a very straightforward easy path and that's the last option we offer and mostly for "trying" purposes. Based on this, docker should not be used for production use but for trying/early phases/development. Do I get this correctly?
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.
Fairly easy path with the work CRE has put into the docker containers
Is a valid choice for testing/early phases or 200 seat deployments.
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.
@coltoneshaw Let me rephrase it. it's straighforward to run, though based on the followings:
It's not for production use and mission-critical work. So we can make clear in the docs that Docker is for early stage POCs/testing with a sub-note, you can use it in prod with your own risk. @cwarnermm can frame it much better than me of course.
thoughts?
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.
@coltoneshaw I think the key here is that while the Docker deployment can easily support 200 seat and smaller deployments, it doesn't support our goals of mission-critical communication, so we don't think it should.
In addition, I think it sets up the customer for a more difficult move down the road, should they decide to go for a proper highly available deployment (now you have to migrate your database on top of the application nodes, etc). Anecdotally I have heard of a few customers who were in the position of having followed this doc, ending up with a database in Docker, and then lacking the resources to move that database to RDS after the fact.
We are looking to become more prescriptive in the way we recommend deployment methods, aiming to guide people to more managed or semi-managed gold standard deployments as these can drastically improve the user experience within Mattermost itself, as well as reduce the time to get up and running.
There's no imminent plan to abandon support for Docker based deployments. But, we want to encourage users to choose other things. For example, if you plan to have under 200 seats, and are OK with your database being on the same instance as the application itself, you should probably just use Omnibus, as you'll be up and running quicker.