issue_workflow.md 21.8 KB
Newer Older
1
# Workflow labels
2 3

To allow for asynchronous issue handling, we use [milestones][milestones-page]
4
and [labels](https://gitlab.com/gitlab-org/gitlab-ce/-/labels). Leads and product managers handle most of the
5 6 7 8
scheduling into milestones. Labelling is a task for everyone.

Most issues will have labels for at least one of the following:

9 10 11 12 13 14 15
- Type: ~feature, ~bug, ~backstage, etc.
- Subject: ~wiki, ~"Container Registry", ~ldap, ~api, etc.
- Team: ~Documentation, ~Delivery, etc.
- Stage: ~"devops::plan", ~"devops::create", etc.
- Group: ~"group::source code" ~"group::knowledge" ~"group::editor", etc.
- Department: ~UX, ~Quality
- Specialization: ~frontend, ~backend
16 17 18 19 20
- Release Scoping: ~Deliverable, ~Stretch, ~"Next Patch Release"
- Priority: ~P1, ~P2, ~P3, ~P4
- Severity: ~S1, ~S2, ~S3, ~S4

All labels, their meaning and priority are defined on the
21
[labels page](https://gitlab.com/gitlab-org/gitlab-ce/-/labels).
22 23 24 25

If you come across an issue that has none of these, and you're allowed to set
labels, you can _always_ add the team and type, and often also the subject.

26
[milestones-page]: https://gitlab.com/groups/gitlab-org/-/milestones
27

28
## Type labels
29 30 31 32

Type labels are very important. They define what kind of issue this is. Every
issue should have one or more.

33
Examples of type labels are ~feature, ~bug, ~backstage and ~security
34 35 36 37 38 39 40

A number of type labels have a priority assigned to them, which automatically
makes them float to the top, depending on their importance.

Type labels are always lowercase, and can have any color, besides blue (which is
already reserved for subject labels).

41
The descriptions on the [labels page](https://gitlab.com/gitlab-org/gitlab-ce/-/labels) explain what falls under each type label.
42

43
## Subject labels
44 45 46 47

Subject labels are labels that define what area or feature of GitLab this issue
hits. They are not always necessary, but very convenient.

48 49 50 51
Subject labels are now used to infer and apply relevant group and devops stage
labels. Please apply them whenever possible to facilitate accurate matching.
Please refer to [this merge request][inferred-labels] for more information.

52
Examples of subject labels are ~wiki, ~ldap, ~api,
53
~issues, ~"merge requests", ~labels, and ~"Container Registry".
54 55 56 57 58 59 60

If you are an expert in a particular area, it makes it easier to find issues to
work on. You can also subscribe to those labels to receive an email each time an
issue is labeled with a subject label corresponding to your expertise.

Subject labels are always all-lowercase.

61
## Team labels
62

63
**Important**: Most of the team labels will be soon deprecated in favor of [Group labels](#group-labels).
64 65 66 67 68

Team labels specify what team is responsible for this issue.
Assigning a team label makes sure issues get the attention of the appropriate
people.

69
The team labels planned for deprecation are:
70

71
- ~Configure
72
- ~Create
73
- ~Defend
74
- ~Distribution
75
- ~Ecosystem
76 77
- ~Geo
- ~Gitaly
78
- ~Growth
79
- ~Manage
80
- ~Memory
81
- ~Monitor
82 83 84
- ~Plan
- ~Release
- ~Secure
85
- ~Verify
86

87 88 89 90 91
The following team labels are **true** teams per our [organization structure](https://about.gitlab.com/company/team/structure/#organizational-structure) which will remain post deprecation.

- ~Delivery
- ~Documentation

92
The descriptions on the [labels page](https://gitlab.com/gitlab-org/gitlab-ce/-/labels) explain what falls under the
93 94 95 96 97
responsibility of each team.

Team labels are always capitalized so that they show up as the first label for
any issue.

98 99 100 101
## Stage labels

Stage labels specify which [DevOps stage][devops-stages] the issue belongs to.

102
The current stage labels can be found by [searching the labels list for `devops::`](https://gitlab.com/groups/gitlab-org/-/labels?search=devops%3A%3A).
R
Rémy Coutable 已提交
103 104 105

These labels are [scoped labels](../../user/project/labels.md#scoped-labels-premium)
and thus are mutually exclusive.
106 107 108 109 110 111

The Stage labels are used to generate the [direction pages][direction-pages] automatically.

[devops-stages]: https://about.gitlab.com/direction/#devops-stages
[direction-pages]: https://about.gitlab.com/direction/

112 113 114 115
## Group labels

Group labels specify which [groups][structure-groups] the issue belongs to.

116 117
The current group labels can be found by [searching the labels list for `group::`](https://gitlab.com/groups/gitlab-org/-/labels?search=group%3A%3A).

R
Rémy Coutable 已提交
118 119
These labels are [scoped labels](../../user/project/labels.md#scoped-labels-premium)
and thus are mutually exclusive.
120

121 122 123 124 125 126 127 128 129 130
You can find the groups listed in the [Product Stages, Groups, and Categories][product-categories] page.

We use the term group to map down product requirements from our product stages. 
As a team needs some way to collect the work their members are planning to be assigned to, we use the `~group::` labels to do so.

Normally there is a 1:1 relationship between Stage labels and Group labels. In the spirit of "Everyone can contribute",
any issue can be picked up by any group, depending on current priorities. For example, an issue labeled ~"devops::create" may be picked up by the ~"group::access" group.

We also use stage and group labels to help quantify our [throughput](https://about.gitlab.com/handbook/engineering/management/throughput). 
Please read [Stage and Group labels in Throughtput](https://about.gitlab.com/handbook/engineering/management/throughput/#stage-and-group-labels-in-throughput) for more information on how the labels are used in this context.
131 132 133 134

[structure-groups]: https://about.gitlab.com/company/team/structure/#groups
[product-categories]: https://about.gitlab.com/handbook/product/categories/

135 136 137 138
## Department labels

The current department labels are:

139 140
- ~UX
- ~Quality
141 142 143 144 145

## Specialization labels

These labels narrow the [specialization](https://about.gitlab.com/company/team/structure/#specialist) on a unit of work.

146 147
- ~frontend
- ~backend
148

149
## Release Scoping labels
150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170

Release Scoping labels help us clearly communicate expectations of the work for the
release. There are three levels of Release Scoping labels:

- ~Deliverable: Issues that are expected to be delivered in the current
  milestone.
- ~Stretch: Issues that are a stretch goal for delivering in the current
  milestone. If these issues are not done in the current release, they will
  strongly be considered for the next release.
- ~"Next Patch Release": Issues to put in the next patch release. Work on these
  first, and add the "Pick Into X" label to the merge request, along with the
  appropriate milestone.

Each issue scheduled for the current milestone should be labeled ~Deliverable
or ~"Stretch". Any open issue for a previous milestone should be labeled
~"Next Patch Release", or otherwise rescheduled to a different milestone.

### Priority labels

Priority labels help us define the time a ~bug fix should be completed. Priority determines how quickly the defect turnaround time must be.
If there are multiple defects, the priority decides which defect has to be fixed immediately versus later.
171
This label documents the planned timeline & urgency which is used to measure against our target SLO on delivering ~bug fixes.
172

173
| Label | Meaning         | Target SLO (applies only to ~bug and ~security defects)                                                    |
M
Mek Stittri 已提交
174 175 176 177 178
|-------|-----------------|----------------------------------------------------------------------------|
| ~P1   | Urgent Priority | The current release + potentially immediate hotfix to GitLab.com (30 days) |
| ~P2   | High Priority   | The next release (60 days)                                                 |
| ~P3   | Medium Priority | Within the next 3 releases (approx one quarter or 90 days)                 |
| ~P4   | Low Priority    | Anything outside the next 3 releases (more than one quarter or 120 days)   |
179

180
## Severity labels
181 182

Severity labels help us clearly communicate the impact of a ~bug on users.
183 184 185 186 187 188 189 190 191 192 193
There can be multiple facets of the impact. The below is a guideline.

| Label | Meaning           | Functionality                                         | Affected Users                   | GitLab.com Availability                            | Performance Degradation      |
|-------|-------------------|-------------------------------------------------------|----------------------------------|----------------------------------------------------|------------------------------|
| ~S1   | Blocker           | Unusable feature with no workaround, user is blocked  | Impacts 50% or more of users     | Outage, Significant impact on all of GitLab.com    |                                                       |
| ~S2   | Critical Severity | Broken Feature, workaround too complex & unacceptable | Impacts between 25%-50% of users | Significant impact on large portions of GitLab.com | Degradation is guaranteed to occur in the near future |
| ~S3   | Major Severity    | Broken feature with an acceptable workaround          | Impacts up to 25% of users       | Limited impact on important portions of GitLab.com | Degradation is likely to occur in the near future     |
| ~S4   | Low Severity      | Functionality inconvenience or cosmetic issue         | Impacts less than 5% of users    | Minor impact on GitLab.com                         | Degradation _may_ occur but it's not likely           |

If a bug seems to fall between two severity labels, assign it to the higher-severity label.

194
- Example(s) of ~S1
195
  - Data corruption/loss.
196
  - Security breach.
197
  - Unable to create an issue or merge request.
198
  - Unable to add a comment or thread to the issue or merge request.
199 200 201 202 203 204 205 206 207
- Example(s) of ~S2
  - Cannot submit changes through the web IDE but the commandline works.
  - A status widget on the merge request page is not working but information can be seen in the test pipeline page.
- Example(s) of ~S3
  - Can create merge requests only from the Merge Requests list view, not from an Issue page.
  - Status is not updated in real time and needs a page refresh.
- Example(s) of ~S4
  - Label colors are incorrect.
  - UI elements are not fully aligned.
208

209
## Label for community contributors
210 211 212

Issues that are beneficial to our users, 'nice to haves', that we currently do
not have the capacity for or want to give the priority to, are labeled as
213
~"Accepting merge requests", so the community can make a contribution.
214 215

Community contributors can submit merge requests for any issue they want, but
216
the ~"Accepting merge requests" label has a special meaning. It points to
217 218 219 220 221 222 223
changes that:

1. We already agreed on,
1. Are well-defined,
1. Are likely to get accepted by a maintainer.

We want to avoid a situation when a contributor picks an
224
~"Accepting merge requests" issue and then their merge request gets closed,
225 226 227
because we realize that it does not fit our vision, or we want to solve it in a
different way.

228
We add the ~"Accepting merge requests" label to:
229 230

- Low priority ~bug issues (i.e. we do not add it to the bugs that we want to
231
  solve in the ~"Next Patch Release")
232
- Small ~feature
233 234
- Small ~"technical debt" issues

235
After adding the ~"Accepting merge requests" label, we try to estimate the
236 237 238
[weight](#issue-weight) of the issue. We use issue weight to let contributors
know how difficult the issue is. Additionally:

239
- We advertise [`Accepting merge requests` issues with weight < 5][up-for-grabs]
240 241 242
  as suitable for people that have never contributed to GitLab before on the
  [Up For Grabs campaign](http://up-for-grabs.net)
- We encourage people that have never contributed to any open source project to
243
  look for [`Accepting merge requests` issues with a weight of 1][firt-timers]
244 245 246 247 248

If you've decided that you would like to work on an issue, please @-mention
the [appropriate product manager](https://about.gitlab.com/handbook/product/#who-to-talk-to-for-what)
as soon as possible. The product manager will then pull in appropriate GitLab team
members to further discuss scope, design, and technical considerations. This will
249
ensure that your contribution is aligned with the GitLab product and minimize
250 251
any rework and delay in getting it merged into master.

252
GitLab team members who apply the ~"Accepting merge requests" label to an issue
253 254 255
should update the issue description with a responsible product manager, inviting
any potential community contributor to @-mention per above.

256 257
[up-for-grabs]: https://gitlab.com/groups/gitlab-org/-/issues?state=opened&label_name[]=Accepting+merge+requests&assignee_id=0&sort=weight
[firt-timers]: https://gitlab.com/groups/gitlab-org/-/issues?state=opened&label_name[]=Accepting+merge+requests&assignee_id=0&sort=weight&weight=1
258

259
## Issue triaging
260 261 262 263 264 265 266 267 268 269 270 271

Our issue triage policies are [described in our handbook]. You are very welcome
to help the GitLab team triage issues. We also organize [issue bash events] once
every quarter.

The most important thing is making sure valid issues receive feedback from the
development team. Therefore the priority is mentioning developers that can help
on those issues. Please select someone with relevant experience from the
[GitLab team][team]. If there is nobody mentioned with that expertise look in
the commit history for the affected files to find someone.

We also use [GitLab Triage] to automate some triaging policies. This is
272
currently set up as a [scheduled pipeline] running on [quality/triage-ops]
273 274 275 276 277 278 279
project.

[described in our handbook]: https://about.gitlab.com/handbook/engineering/issue-triage/
[issue bash events]: https://gitlab.com/gitlab-org/gitlab-ce/issues/17815
[GitLab Triage]: https://gitlab.com/gitlab-org/gitlab-triage
[scheduled pipeline]: https://gitlab.com/gitlab-org/quality/triage-ops/pipeline_schedules/10512/edit
[quality/triage-ops]: https://gitlab.com/gitlab-org/quality/triage-ops
280
[team]: https://about.gitlab.com/team/
281

282
## Feature proposals
283 284 285 286 287 288 289 290

To create a feature proposal for CE, open an issue on the
[issue tracker of CE][ce-tracker].

For feature proposals for EE, open an issue on the
[issue tracker of EE][ee-tracker].

In order to help track the feature proposals, we have created a
291
[`feature`][fl] label. For the time being, users that are not members
292
of the project cannot add labels. You can instead ask one of the [core team](https://about.gitlab.com/community/core-team/)
293 294
members to add the label ~feature to the issue or add the following
code snippet right after your description in a new line: `~feature`.
295 296 297 298

Please keep feature proposals as small and simple as possible, complex ones
might be edited to make them small and simple.

299
Please submit Feature Proposals using the ['Feature Proposal' issue template](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.gitlab/issue_templates/Feature%20proposal.md) provided on the issue tracker.
300 301 302

For changes in the interface, it is helpful to include a mockup. Issues that add to, or change, the interface should
be given the ~"UX" label. This will allow the UX team to provide input and guidance. You may
303
need to ask one of the [core team](https://about.gitlab.com/community/core-team/) members to add the label, if you do not have permissions to do it by yourself.
304 305 306 307

If you want to create something yourself, consider opening an issue first to
discuss whether it is interesting to include this in GitLab.

308
[fl]: https://gitlab.com/gitlab-org/gitlab-ce/issues?label_name=feature
309

310
## Issue tracker guidelines
311 312 313 314 315 316 317 318 319 320 321

**[Search the issue tracker][ce-tracker]** for similar entries before
submitting your own, there's a good chance somebody else had the same issue or
feature proposal. Show your support with an award emoji and/or join the
discussion.

Please submit bugs using the ['Bug' issue template](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.gitlab/issue_templates/Bug.md) provided on the issue tracker.
The text in the parenthesis is there to help you with what to include. Omit it
when submitting the actual issue. You can copy-paste it and then edit as you
see fit.

322
## Issue weight
323 324 325 326 327 328 329 330 331

Issue weight allows us to get an idea of the amount of work required to solve
one or multiple issues. This makes it possible to schedule work more accurately.

You are encouraged to set the weight of any issue. Following the guidelines
below will make it easy to manage this, without unnecessary overhead.

1. Set weight for any issue at the earliest possible convenience
1. If you don't agree with a set weight, discuss with other developers until
332
   consensus is reached about the weight
333
1. Issue weights are an abstract measurement of complexity of the issue. Do not
334 335
   relate issue weight directly to time. This is called [anchoring](https://en.wikipedia.org/wiki/Anchoring)
   and something you want to avoid.
336
1. Something that has a weight of 1 (or no weight) is really small and simple.
337 338 339
   Something that is 9 is rewriting a large fundamental part of GitLab,
   which might lead to many hard problems to solve. Changing some text in GitLab
   is probably 1, adding a new Git Hook maybe 4 or 5, big features 7-9.
340
1. If something is very large, it should probably be split up in multiple
341 342
   issues or chunks. You can simply not set the weight of a parent issue and set
   weights to children issues.
343

344
## Regression issues
345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363

Every monthly release has a corresponding issue on the CE issue tracker to keep
track of functionality broken by that release and any fixes that need to be
included in a patch release (see [8.3 Regressions] as an example).

As outlined in the issue description, the intended workflow is to post one note
with a reference to an issue describing the regression, and then to update that
note with a reference to the merge request that fixes it as it becomes available.

If you're a contributor who doesn't have the required permissions to update
other users' notes, please post a new note with a reference to both the issue
and the merge request.

The release manager will [update the notes] in the regression issue as fixes are
addressed.

[8.3 Regressions]: https://gitlab.com/gitlab-org/gitlab-ce/issues/4127
[update the notes]: https://gitlab.com/gitlab-org/release-tools/blob/master/doc/pro-tips.md#update-the-regression-issue

364
## Technical and UX debt
365 366 367

In order to track things that can be improved in GitLab's codebase,
we use the ~"technical debt" label in [GitLab's issue tracker][ce-tracker].
P
Pedro Moreira da Silva 已提交
368
For missed user experience requirements, we use the ~"UX debt" label.
369 370 371 372 373

These labels should be added to issues that describe things that can be improved,
shortcuts that have been taken, features that need additional attention, and all
other things that have been left behind due to high velocity of development.
For example, code that needs refactoring should use the ~"technical debt" label,
P
Pedro Moreira da Silva 已提交
374 375
something that didn't ship according to our Design System guidelines should
use the ~"UX debt" label.
376 377 378 379 380 381 382 383 384 385 386 387 388

Everyone can create an issue, though you may need to ask for adding a specific
label, if you do not have permissions to do it by yourself. Additional labels
can be combined with these labels, to make it easier to schedule
the improvements for a release.

Issues tagged with these labels have the same priority like issues
that describe a new feature to be introduced in GitLab, and should be scheduled
for a release by the appropriate person.

Make sure to mention the merge request that the ~"technical debt" issue or
~"UX debt" issue is associated with in the description of the issue.

389 390 391 392
## Technical debt in follow-up issues

It's common to discover technical debt during development of a new feature. In
the spirit of "minimum viable change", resolution is often deferred to a
393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424
follow-up issue. However, this cannot be used as an excuse to merge poor-quality
code that would otherwise not pass review, or to overlook trivial matters that
don't deserve the be scheduled independently, and would be best resolved in the
original merge request - or not tracked at all!

The overheads of scheduling, and rate of change in the GitLab codebase, mean
that the cost of a trivial technical debt issue can quickly exceed the value of
tracking it. This generally means we should resolve these in the original merge
request - or simply not create a follow-up issue at all.

For example, a typo in a comment that is being copied between files is worth
fixing in the same MR, but not worth creating a follow-up issue for. Renaming a
method that is used in many places to make its intent slightly clearer may be
worth fixing, but it should not happen in the same MR, and is generally not
worth the overhead of having an issue of its own. These issues would invariably
be labelled `~P4 ~S4` if we were to create them.

More severe technical debt can have implications for development velocity. If
it isn't addressed in a timely manner, the codebase becomes needlessly difficult
to change, new features become difficult to add, and regressions abound.

Discoveries of this kind of technical debt should be treated seriously, and
while resolution in a follow-up issue may be appropriate, maintainers should
generally obtain a scheduling commitment from the author of the original MR, or
the engineering or product manager for the relevant area. This may take the form
of appropriate Priority / Severity labels on the issue, or an explicit milestone
and assignee.

The maintainer must always agree before an outstanding discussion is resolved in
this manner, and will be the one to create the issue. The title and description
should be of the same quality as those created
[in the usual manner](#technical-and-ux-debt) - in particular, the issue title
425 426
**must not** begin with `Follow-up`! The creating maintainer should also expect
to be involved in some capacity when work begins on the follow-up issue.
427

428
## Stewardship
429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445

For issues related to the open source stewardship of GitLab,
there is the ~"stewardship" label.

This label is to be used for issues in which the stewardship of GitLab
is a topic of discussion. For instance if GitLab Inc. is planning to add
features from GitLab EE to GitLab CE, related issues would be labelled with
~"stewardship".

A recent example of this was the issue for
[bringing the time tracking API to GitLab CE][time-tracking-issue].

[time-tracking-issue]: https://gitlab.com/gitlab-org/gitlab-ce/issues/25517#note_20019084

---

[Return to Contributing documentation](index.md)
446 447 448

[ce-tracker]: https://gitlab.com/gitlab-org/gitlab-ce/issues
[ee-tracker]: https://gitlab.com/gitlab-org/gitlab-ee/issues
449
[inferred-labels]: https://gitlab.com/gitlab-org/quality/triage-ops/merge_requests/155