summaryrefslogtreecommitdiffstats
path: root/docs/bug-mgmt/policies/triage-bugzilla.rst
blob: 276979eaf15667e0a6bc0d8ba9662288bb57bc2b (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
Triage for Bugzilla
===================

Expectations
------------

All teams working on Firefox using either or both Mozilla-central and
Bugzilla are expected to follow the following process.

What is a Triaged Bug
---------------------

The new definition of Triaged will be Firefox-related bugs of type
``defect`` where the component is not
``UNTRIAGED``, and a severity value not equal to ``--`` or ``N/A``.

Bugs of type Task or Enhancement may have a severity of ``N/A``,
but defects must have a severity that is neither ``--`` or
``N/A``.

Why Triage
----------

We want to make sure that we looked at every defect and a severity has
been defined. This way, we make sure that we did not miss any critical
issues during the development and stabilization cycles.

Staying on top of the bugs in your component means:

-  You get ahead of critical regressions and crashes which could trigger
   a point release if uncaught

   -  And you don’t want to spend your holiday with the Release
      Management team (not that they don’t like you)

-  Your bug queue is not overwhelming

   -  Members of your team do not see the bug queue and get the
      ‘wiggins’

Who Triages
-----------

Engineering managers and directors are responsible for naming the
individuals responsible for triaging :ref:`all types of bugs <Bug Types>` in a component.

We use Bugzilla to manage this. See the `list of triage
owners <https://bugzilla.mozilla.org/page.cgi?id=triage_owners.html>`__.

If you need to change who is responsible for triaging a bug in a
component, please `file a bug against bugzilla.mozilla.org in the
Administration
component <https://bugzilla.mozilla.org/enter_bug.cgi?product=bugzilla.mozilla.org&component=Administration>`__.
When a new component is created, a triage owner **must** be named.

Rotating triage
~~~~~~~~~~~~~~~

Some components are monitored by a rotation of triagers. In those cases,
the triage owner on Bugzilla will be automatically updated to reflect the
person on the rotation. The rotations are managed as calendars.

If you wish to set up a rotation for triaging one or more components,
add a link to your rotation calendar in the `triage rotations spreadsheet <https://docs.google.com/spreadsheets/d/1EK6iCtdD8KP4UflIHscuZo6W5er2vy_TX7vsmaaBVd4>`__.

Firefox::General and Toolkit::General
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Bugs in Firefox::General are fitted with Bug Bug’s model to see if
there’s another component with a high liklihood of fit, and if a
threshold confidence is achieved, the bug is moved to that component.

Members of the community also review bugs in this component and try to
move them.

What Do You Triage
------------------

As a triage owner the queries you should be following for your component
are:

-  All open bugs, in your components without a pending ``needinfo`` flag
   which do not have a valid value of severity set
-  All bugs with active review requests in your component which have not
   been modified in five days
-  All bugs with reviewed, but unlanded patches in your components
-  All bugs with a needinfo request unanswered for more than 10 days

There’s a tool with these queries to help you find bugs
https://mozilla.github.io/triage-center/ and the source is at
https://github.com/mozilla/triage-center/.

If a bug is an enhancement it needs a priority set and a target release
or program milestone. These bugs are normally reviewed by product
managers. Enhancements can lead to release notes and QA needed that we
also need to know about

If a bug is a task resulting in a changeset, release managers will need
to known when this work will be done. A task such as refactoring fragile
code can be risky.

Weekly or More Frequently (depending on the component) find un-triaged
bugs in the components you triage.

Decide the :ref:`Severity <Defect Severity>`  for each untriaged bug
(you can override what’s already been set.)

These bugs are reviewed in the weekly Regression Triage meeting

- Bugs of type ``defect`` with the ``regression`` keyword without
  ``status-firefoxNN`` decisions
- Bugs of type ``defect`` with the ``regression`` keyword without
  a regression range

Automatic Bug Updates
~~~~~~~~~~~~~~~~~~~~~

When a bug is tracked for a release, i.e. the ``tracking_firefoxNN``
flag is set to ``+`` or ``blocking`` triage decisions will be overridden,
or made as follows:

-  If a bug is tracked for or blocking beta, release or ESR, its
   priority will be set to ``P1``
-  If a bug is tracked for or blocking nightly, its priority will be set
   to ``P2``

Because bugs can be bumped in priority it’s essential that triage owners
review their
`P1 <https://bugzilla.mozilla.org/buglist.cgi?priority=P1&f1=triage_owner&o1=equals&resolution=---&v1=%25user%25>`__
and
`P2 <https://bugzilla.mozilla.org/buglist.cgi?priority=P2&f1=triage_owner&o1=equals&resolution=---&v1=%25user%25>`__
bugs frequently.

Assumptions
~~~~~~~~~~~

If a bug's release status in Firefox version N was ``affected`` or ``wontfix``,
its Severity is ``S3`` or ``S4`` and its Priority is ``P3`` or lower (backlog,)
then its release status in Firefox version N+1, if the bug is still open,
is considered to be ``wontfix``.

Questions and Edge Cases
------------------------

This bug is a feature request
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Set the bug’s type to ``enhancement``, add the ``feature`` keyword if
relevant, and state to ``NEW``. Set the bug's Severity to ``N/A``. This
bug will be excluded from future triage queries.

This bug is a task, not a defect
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Set the bug’s type to ``task``, and state to ``NEW``. Set the bug's
Severity to ``N/A``. This bug will be excluded from future triage queries.


If you are not sure of a bug’s type, check :ref:`our rules for bug
types <Bug Types>`.

This bug’s state is ``UNCONFIRMED``
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Are there steps to reproduce? If not, needinfo the person who filed the
bug, requesting steps to reproduce. You are not obligated to wait
forever for a response, and bugs for which open requests for information
go unanswered can be ``RESOLVED`` as ``INCOMPLETE``.

I need help reproducing the bug
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Set a needinfo for the QA managers, Softvision project managers, or the
QA owner of the component of the bug.

I don’t have enough information to make a decision
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

If you don’t have a reproduction or confirmation, or have questions
about how to proceed, ``needinfo`` the person who filed the bug, or
someone who can answer.

The ``stalled`` keyword
~~~~~~~~~~~~~~~~~~~~~~~

The extreme case of not-enough-information is one which cannot be
answered with a ``needinfo`` request. The reporter has shared all they
know about the bug, we are out of strategies to take to resolve it, but
the bug should be kept open.

Mark the bug as stalled by adding the ``stalled`` keyword to it. The
keyword will remove it from the list of bugs to be triaged.

If a patch lands on a ``stalled`` bug, automation will remove the
keyword. Otherwise, when the ``keyword`` is removed, the bug will have
its priority reset to ``--`` and the components triage owner notified by
automation.

Bugs which remain ``stalled`` for long periods of time should be
reviewed, and closed if necessary.

Bug is in the wrong Component
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

If the bug has a Severity of ``S3``, ``S4``, or ``N/A`` move the what
you think is the correct component, or needinfo the person
responsible for the component to ask them.

If the bug has a Severity of ``S1`` or ``S2`` then notify Release Management
and contact the triage owner of the component for which you think it belongs to.
We cannot lose track of a high severity bug because it is in the wrong component.

My project is on GitHub
~~~~~~~~~~~~~~~~~~~~~~~

We have :ref:`a guide for GitHub projects to follow <GitHub Metadata Recommendations>` when
triaging. (Note: this guide needs updating.)

Summary
-------

Multiple times weekly
~~~~~~~~~~~~~~~~~~~~~

Use queries for the components you are responsible for in
https://mozilla.github.io/triage-center/ to find bugs in
need of triage.

For each untriaged bug:

-  Assign a Severity
-  **Do not** assign a ``defect`` a Severity of
   ``N/A``

You can, but are not required to set the bug's :ref:`Priority <Priority Definitions>`.

Watch open needinfo flags
~~~~~~~~~~~~~~~~~~~~~~~~~

Don’t let open needinfo flags linger for more than two weeks.

Close minor bugs with unresponded needinfo flags.

Follow up on needinfo flag requests.

The `Triage Center tool <https://mozilla.github.io/triage-center/>`__ will help you find these.

End of Iteration/Release Cycle
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Any open ``S1`` or ``S2`` bugs at the end of the release cycle
will require review by engineering and release management. A
policy on this is forthcoming.

Optional
^^^^^^^^

(The guidelines on bug priority are under review.)

Are there open P1s? Revisit their priority,
and move to them to the backlog (``P3``) or ``P2``.

Are there ``P2`` bugs that should move to ``P1``
for the next cycle?

Are there ``P2`` you now know are lower priority,
move to ``P3``.

Are there ``P3`` bugs you now know you won’t get to?
Either demote to ``P5`` (will accept patch) or
resolve as ``WONTFIX``.

Getting help
------------

-  Ask in #bug-handling on chat.mozilla.org