Commit 9235f1d4 authored by Achilleas Pipinellis's avatar Achilleas Pipinellis

Merge branch 'link-to-downtime-requires-from-migration-style-guide' into 'master'

Link to the "What requires downtime?" page from the Migration Style Guide

## What does this MR do?

Adds a link from migration style guide -> "What requires downtime?"

## Are there points in the code the reviewer needs to double check?

## Why was this MR needed?

The two pages really need to be read together, so having the link is helpful.

## Screenshots (if relevant)

## Does this MR meet the acceptance criteria?

- [ ] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- [ ] API support added
- Tests
  - [ ] Added for this feature/bug
  - [ ] All builds are passing
- [ ] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html)
- [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [ ] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)

## What are the relevant issue numbers?

See merge request !6636
parents 77f832d8 267a96f3
...@@ -9,10 +9,10 @@ a big burden for most organizations. For this reason it is important that your ...@@ -9,10 +9,10 @@ a big burden for most organizations. For this reason it is important that your
migrations are written carefully, can be applied online and adhere to the style guide below. migrations are written carefully, can be applied online and adhere to the style guide below.
Migrations should not require GitLab installations to be taken offline unless Migrations should not require GitLab installations to be taken offline unless
_absolutely_ necessary. If a migration requires downtime this should be _absolutely_ necessary - see the ["What Requires Downtime?"](what_requires_downtime.md)
clearly mentioned during the review process as well as being documented in the page. If a migration requires downtime, this should be clearly mentioned during
monthly release post. For more information see the "Downtime Tagging" section the review process, as well as being documented in the monthly release post. For
below. more information, see the "Downtime Tagging" section below.
When writing your migrations, also consider that databases might have stale data When writing your migrations, also consider that databases might have stale data
or inconsistencies and guard for that. Try to make as little assumptions as possible or inconsistencies and guard for that. Try to make as little assumptions as possible
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment