-
Notifications
You must be signed in to change notification settings - Fork 598
Deployment Section Restructure #2 #7852
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
Conversation
Newest code from mattermost has been published to preview environment for Git SHA 63fc34b |
Newest code from mattermost has been published to preview environment for Git SHA 1daeb3d |
Newest code from mattermost has been published to preview environment for Git SHA fa9f367 |
Newest code from mattermost has been published to preview environment for Git SHA f082be4 |
@esethna Good notes - I'll update the pages later today and re-request review. Thanks! |
@@ -27,6 +27,9 @@ | |||
|
|||
</div> | |||
|
|||
.. warning:: |
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.
In addition to this docs warning, a Forum announcement post is also required.
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 Did you discuss this one with PM/CS already? Would you be open to prepare the forum post?
Newest code from mattermost has been published to preview environment for Git SHA 608f7bf |
Newest code from mattermost has been published to preview environment for Git SHA 716ed9b |
Updated the PRs, outstanding organizational questions re: deprecation notice and preview orders, but otherwise good to go for second review. Spiros will prepare a second PR for other changes. |
@DSchalla - the following build warnings are present via this docs PR. In all 3 cases, the source file isn't defined as a nav pane item. Can I get your help to confirm whether this is intentional or not, please?
|
Newest code from mattermost has been published to preview environment for Git SHA 0c9db0c |
@DSchalla added one comment above, re Azure vs Docker being the default (and if the docker image can be E0), we don't need to block the PR since @spirosoik is out for the week, we can follow-up when he's back |
Newest code from mattermost has been published to preview environment for Git SHA 654e350 |
@DSchalla - There's 2 questions outstanding ahead of merging this PR:
|
@cwarnermm Azure VM (which has yet to be documented here) is simply a bundled version of omnibus. That means it would follow the same recommendations (or lack thereof) as regular Ubuntu omnibus - ie, not recommended for mission critical production deployments. I'm actively working on documentation for the Azure VM offer, and will have a PR open in the coming days. |
Thanks, @nickmisasi! Currently the Quick Start Evaluation page notes that both deploy options use SQLite as the database and are configured for demonstration purposes only, but the Azure Marketplace option notes that the Azure Marketplace image comes with PostgreSQL and is more suitable for testing production-like scenarios. Curious to know if there's a contradiction there we need to fix? |
Newest code from mattermost has been published to preview environment for Git SHA 40961dc |
Newest code from mattermost has been published to preview environment for Git SHA 692face |
Newest code from mattermost has been published to preview environment for Git SHA 241618f |
Newest code from mattermost has been published to preview environment for Git SHA c81a18c |
* Add documentation for Azure AIO VM offer * Update documentation for Azure AIO offer * Update * Style fixes * More style fixes * More style fixes
Newest code from mattermost has been published to preview environment for Git SHA 171163a |
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.
LGTM!
@esethna Can you re-review that merging gets unblocked? Thanks. |
I created a separate branch to improve the reference architecture/scale pages and in parallel working to update deployment for AWS/Azure. This PR is in WIP #7893 I will finalise this tomorrow. |
Newest code from mattermost has been published to preview environment for Git SHA 6572e92 |
Summary
Follow up to #7816, which introduces multiple bug fixes and enhancements.