Commit 4a0b0e55 authored by Marcel Amirault's avatar Marcel Amirault

Merge branch 'furkanayhan-master-patch-58052' into 'master'

Fix misinformation in FF documentation

See merge request gitlab-org/gitlab!28908
parents d8c31d19 e4942375
...@@ -98,6 +98,7 @@ Slack: ...@@ -98,6 +98,7 @@ Slack:
This will enable the feature for GitLab.com, with `new_navigation_bar` being the This will enable the feature for GitLab.com, with `new_navigation_bar` being the
name of the feature. name of the feature.
This command does *not* enable the feature for 25% of the total users. Instead, when the feature is checked with `enabled?`, it will return `true` 25% of the time.
If you are not certain what percentages to use, simply use the following steps: If you are not certain what percentages to use, simply use the following steps:
...@@ -136,9 +137,8 @@ you run these 2 commands: ...@@ -136,9 +137,8 @@ you run these 2 commands:
/chatops run feature set some_feature 25 /chatops run feature set some_feature 25
``` ```
Then `some_feature` will be enabled for 25% of the time the users are interacting with Then `some_feature` will be enabled for both 25% of users and all users interacting with
`gitlab-org/gitlab`. Note that the the feature is not enabled to 25% `gitlab-org/gitlab`.
of the users, rather a simple randomization is made each time the `enabled?` is checked.
NOTE: **Note:** NOTE: **Note:**
**Percentage of time** rollout is not a good idea if what you want is to make sure a feature **Percentage of time** rollout is not a good idea if what you want is to make sure a feature
......
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