Add Cloud Native Hybrid instructions on 3k users RA page
What does this MR do?
Changes:
- Adds 3k hybrid environment guidance based on the performance testing in gitlab-org/quality/reference-architectures#21 (comment 624291967) and gitlab-org/charts/gitlab!2084 (merged).
- Revamps 3k architecture diagram:
- Removes Redis Persistent/Cache components from larger HA environment components
- Collocates Consul and Redis per 3k docs
Related issues
Author's checklist
-
Follow the Documentation Guidelines and Style Guide. -
Ensure that the product tier badge is added to doc's h1
. -
Request a review based on the documentation page's metadata and associated Technical Writer.
To avoid having this MR be added to code verification QA issues, don't add these labels: feature, frontend, backend, ~"bug", or database
Review checklist
Documentation-related MRs should be reviewed by a Technical Writer for a non-blocking review, based on Documentation Guidelines and the Style Guide.
-
If the content requires it, ensure the information is reviewed by a subject matter expert. - Technical writer review items:
-
Ensure docs metadata is present and up-to-date. -
Ensure the appropriate labels are added to this MR. - If relevant to this MR, ensure content topic type principles are in use, including:
-
The headings should be something you'd do a Google search for. Instead of Default behavior
, say something likeDefault behavior when you close an issue
. -
The headings (other than the page title) should be active. Instead of Configuring GDK
, say something likeConfigure GDK
. -
Any task steps should be written as a numbered list. - If the content still needs to be edited for topic types, you can create a follow-up issue with the docs-technical-debt label.
-
-
-
Review by assigned maintainer, who can always request/require the above reviews. Maintainer's review can occur before or after a technical writer review. -
Ensure a release milestone is set.
Merge request reports
Activity
changed milestone to %14.2
assigned to @niskhakova
2 Warnings This merge request is quite big (593 lines changed), please consider splitting it into multiple merge requests. 781b7f6a: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines. 1 Message This merge request adds or changes documentation files. A review from the Technical Writing team before you merge is recommended. Reviews can happen after you merge. Documentation review
The following files require a review from a technical writer:
doc/administration/reference_architectures/10k_users.md
doc/administration/reference_architectures/25k_users.md
doc/administration/reference_architectures/2k_users.md
doc/administration/reference_architectures/3k_users.md
doc/administration/reference_architectures/50k_users.md
doc/administration/reference_architectures/5k_users.md
doc/administration/reference_architectures/index.md
The review does not need to block merging this merge request. See the:
-
Metadata for the
*.md
files that you've changed. The first few lines of each*.md
file identify the stage and group most closely associated with your docs change. - The Technical Writer assigned for that stage and group.
- Documentation workflows for information on when to assign a merge request for review.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
Dangermentioned in issue gitlab-com/www-gitlab-com#11593 (closed)
added 440 commits
-
3224c981...8de2fc1e - 439 commits from branch
master
- 95bdaca0 - Merge branch 'master' into docs-add-3k-hybrid-env
-
3224c981...8de2fc1e - 439 commits from branch
added 1 commit
- efe35d5d - Remove link between Rails and Sidekiq in diagrams
added 122 commits
-
efe35d5d...51062952 - 121 commits from branch
master
- 5caced0d - Merge branch 'master' into docs-add-3k-hybrid-env
-
efe35d5d...51062952 - 121 commits from branch
added Technical Writing label
- Resolved by Kati Paizee
@axil @grantyoung @WarheadsSE could you please review when you have a moment?
requested review from @axil, @grantyoung, and @WarheadsSE
- Resolved by Nailia Iskhakova
- Resolved by Kati Paizee
@WarheadsSE, thanks for approving this merge request.Please consider starting a new pipeline if:
- This is the first time the merge request is approved, or
- The merge request is ready to be merged, and there has not been a merge request pipeline in the last 2 hours.
For more info, refer to the guideline.
added 394 commits
-
5caced0d...b504b297 - 392 commits from branch
master
- 12ce1354 - Merge branch 'master' into docs-add-3k-hybrid-env
- 446947fb - Use collections for Redis and Sentinel per review
-
5caced0d...b504b297 - 392 commits from branch
requested review from @kpaizee
- Resolved by Kati Paizee
@grantyoung, thanks for approving this merge request.Please consider starting a new pipeline if:
- This is the first time the merge request is approved, or
- The merge request is ready to be merged, and there has not been a merge request pipeline in the last 2 hours.
For more info, refer to the guideline.
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
- Resolved by Kati Paizee
added docsimprovement twdoing labels
- Resolved by Kati Paizee
@kpaizee, thanks for approving this merge request.Please consider starting a new pipeline if:
- This is the first time the merge request is approved, or
- The merge request is ready to be merged, and there has not been a merge request pipeline in the last 2 hours.
For more info, refer to the guideline.
mentioned in commit 96773b51
mentioned in issue gitlab-org/quality/reference-architectures#21 (closed)
added workflowstaging label
added workflowcanary label and removed workflowstaging label
added workflowproduction label and removed workflowcanary label
added releasedcandidate label
added releasedpublished label and removed releasedcandidate label
added typemaintenance label