Update IP allowlist information for Dedicated with 'lessons learned'
What does this MR do and why?
This MR updates the IP allowlist section of the Configure GitLab Dedicated docs to include some information we learned (see this Confidential
work item - gitlab-com/gl-infra/gitlab-dedicated/team#3039). Specifically:
-
Clarify what happens when an IP not on the allowlist attempts to access the instance - We used to say "the connection is refused". With this MR, that is clarified to "
HTTP 403
".
- We used to say "the connection is refused". With this MR, that is clarified to "
-
Clarify a bit that the IP allowlist is not enabled by default -
Clarify that we add IPs to the allowlist when you send them in via a ticket
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Merge request reports
Activity
assigned to @bcarranza
added Support Team Contributions label
added 6520 commits
-
6cbfd608...0ebe55d7 - 6519 commits from branch
master
- d584df41 - Merge branch 'master' into 'docs-bcarranza-dedicated-ip-allowlist'
-
6cbfd608...0ebe55d7 - 6519 commits from branch
- Resolved by 🤖 GitLab Bot 🤖
Proper labels assigned to this merge request. Please ignore me.
@bcarranza - please see the following guidance and update this merge request.1 Error Please add typebug typefeature, or typemaintenance label to this merge request. Edited by 🤖 GitLab Bot 🤖
added docs-only label
- A deleted user
added documentation label
7 Warnings This MR has a Changelog commit for EE, but no code changes in ee/
. Consider removing theEE: true
trailer from your commits.edcd4259: 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. a0a28c17: 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. 6d8986c6: The commit body should not contain more than 72 characters per line. For more information, take a look at our Commit message guidelines. ca1360f1: 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. b1c426a3: 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. This merge request has more than 20 commits which may cause issues in some of the jobs. If you see errors like missing commits, please consider squashing some commits so it is within 20 commits. 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/dedicated/index.md
(Link to current live version)
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
DangerEdited by Ghost User-
added maintenancerefactor label
added typemaintenance label
added groupdedicated label
added devopsplatforms sectionplatforms labels
changed milestone to %16.7
marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed
added 2 commits