summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2020-02-28 12:52:06 +0000
committerGitLab Bot <gitlab-bot@gitlab.com>2020-02-28 12:52:06 +0000
commita9d7456c9ce00dccb775201b0649e384a475a055 (patch)
tree48be96b61b10e1ef9c8d520cf6882f809dd458ca
parent40a4f1e07e1575f073bde7b19f960e9cb56fdc6e (diff)
downloadgitlab-ce-a9d7456c9ce00dccb775201b0649e384a475a055.tar.gz
Add latest changes from gitlab-org/gitlab@12-8-stable-ee
-rw-r--r--.gitlab/issue_templates/Security developer workflow.md19
-rw-r--r--.gitlab/merge_request_templates/Security Release.md4
2 files changed, 11 insertions, 12 deletions
diff --git a/.gitlab/issue_templates/Security developer workflow.md b/.gitlab/issue_templates/Security developer workflow.md
index 1b6a1f87216..56be0453b33 100644
--- a/.gitlab/issue_templates/Security developer workflow.md
+++ b/.gitlab/issue_templates/Security developer workflow.md
@@ -9,10 +9,11 @@ Set the title to: `Description of the original issue`
## Prior to starting the security release work
- [ ] Read the [security process for developers] if you are not familiar with it.
-- [ ] Link this issue in the Security Release issue on GitLab.com. You can find this issue in the topic of the `#releases` channel.
-- [ ] Add a link to the confidential `gitlab-org/gitlab` issue describing the vulnerability next to **Original issue** in the [links table](#links).
-- [ ] Add a link to the confidential `gitlab-org/gitlab` Security release issue next to **Security release issue** in the [links table](#links).
+- [ ] Mark this [issue as related] to the Security Release tracking issue. You can find it on the topic of the `#releases` Slack channel.
- [ ] Run `scripts/security-harness` in your local repository to prevent accidentally pushing to any remote besides `gitlab.com/gitlab-org/security`.
+- Fill out the [Links section](#links):
+ - [ ] Next to **Issue on GitLab**, add a link to the `gitlab-org/gitlab` issue that describes the security vulnerability.
+ - [ ] Next to **Security Release tracking issue**, add a link to the security release issue that will include this security issue.
## Development
@@ -29,7 +30,8 @@ After your merge request has being approved according to our [approval guideline
* You can use the script `bin/secpick` instead of the following steps, to help you cherry-picking. See the [secpick documentation]
- [ ] Create each MR targeting the stable branch `X-Y-stable`, using the [Security Release merge request template].
* Every merge request will have its own set of TODOs, so make sure to complete those.
-- [ ] Make sure all MRs are linked in the [Links section](#links)
+- [ ] On the "Related merge requests" section, ensure all MRs are linked to this issue.
+ * This section should only list the merge requests created for this issue: One targeting `master` and the 3 backports.
## Documentation and final details
@@ -46,12 +48,8 @@ After your merge request has being approved according to our [approval guideline
| Description | Link |
| -------- | -------- |
-| Original issue | #TODO |
-| Security release issue | #TODO |
-| `master` MR | !TODO |
-| `Backport X.Y` MR | !TODO |
-| `Backport X.Y` MR | !TODO |
-| `Backport X.Y` MR | !TODO |
+| Issue on [GitLab](https://gitlab.com/gitlab-org/gitlab/issues) | #TODO |
+| Security Release tracking issue | #TODO |
### Details
@@ -68,5 +66,6 @@ After your merge request has being approved according to our [approval guideline
[security Release merge request template]: https://gitlab.com/gitlab-org/security/gitlab/blob/master/.gitlab/merge_request_templates/Security%20Release.md
[code review process]: https://docs.gitlab.com/ee/development/code_review.html
[approval guidelines]: https://docs.gitlab.com/ee/development/code_review.html#approval-guidelines
+[issue as related]: https://docs.gitlab.com/ee/user/project/issues/related_issues.html#adding-a-related-issue
/label ~security
diff --git a/.gitlab/merge_request_templates/Security Release.md b/.gitlab/merge_request_templates/Security Release.md
index cccfafe397e..02cb4c59fd1 100644
--- a/.gitlab/merge_request_templates/Security Release.md
+++ b/.gitlab/merge_request_templates/Security Release.md
@@ -8,11 +8,11 @@ See [the general developer security release guidelines](https://gitlab.com/gitla
## Related issues
-<!-- Mention the issue(s) this MR is related to -->
+<!-- Mention the GitLab Security issue this MR is related to -->
## Developer checklist
-- [ ] Link this MR in the `links` section of the related issue on [GitLab Security].
+- [ ] **Make sure this merge request mentions the [GitLab Security] issue it belongs to (i.e. `Related to <issue_id>`).**
- [ ] Merge request targets `master`, or `X-Y-stable` for backports.
- [ ] Milestone is set for the version this merge request applies to. A closed milestone can be assigned via [quick actions].
- [ ] Title of this merge request is the same as for all backports.