Differences between revisions 2 and 32 (spanning 30 versions)
Revision 2 as of 2008-10-19 15:13:32
Size: 1700
Comment: add unloved query
Revision 32 as of 2015-03-11 18:18:47
Size: 6267
Editor: mpm
Comment:
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
== Bug statuses and what they mean: == #pragma section-numbers 2
<<Include(A:dev)>>
Line 3: Line 4:
 * Unread - no one's ever responded to this bug (bad)
 * Deferred - we've decided not to deal with this problem now
 * Chatting - we're discussing a solution
 * Need-eg - we need more information to progress
 * In-progress - a partial fix exists
 * Testing - a patch is merged, but poster has not confirmed it works
 * Done-cbb - (could be better or couldn't be bothered) not perfect, but we've done as much as we're going to do
 * Resolved - problem is fixed
= Managing Bugs =
{{https://docs.google.com/spreadsheets/d/1lEvdGoN-jgLMZ_1eu8ceXDzEdWGWuDTzPSDVCz4jtxs/embed/oimg?id=1lEvdGoN-jgLMZ_1eu8ceXDzEdWGWuDTzPSDVCz4jtxs&oid=1929125339&zx=w96cwe5uoapn}}
Line 12: Line 7:
== Priorities and what they mean: == How to do bug triage on the BugTracker.
Line 14: Line 9:
 * critical - data loss or security issue
 * urgent - bug that is blocking development
 * bug - bug that's not blocking development
 * feature - it's not a bug, it's a feature
 * wish - would be nice
<<TableOfContents>>
Line 20: Line 11:
== Triage == == How to do triage ==
It's important to approach bug reports as a triage problem. Some reports are five-alarm fires, some are papercuts. We need to make sure that we don't miss the former while we're addressing the latter. And we need to be honest that we're probably not going to have time to address the latter.
Line 22: Line 14:
 * Sort open issues by priority and activity - here is a [http://www.selenic.com/mercurial/bts/issue?@columns=title,id,activity,status,assignedto&@sort=status,activity&@group=priority&@filter=status,priority,status,priority&@pagesize=100&@startwith=0&status=-1,1,3,5&priority=1,2,3&@dispname=triage triage] and a [http://www.selenic.com/mercurial/bts/issue?@columns=title,id,activity,priority,status&@sort=activity&@filter=status&@pagesize=100&@startwith=0&status=-1,1,2,3,4,5,6,7&@dispname=unloved unloved] query.
 * Diagnose issues and add appropriate people to the nosy list
 * Merge duplicates by setting superseder in the more recent bug, copy its nosy list to the older bug, and add a pointer to the new issue in the old issue if it contains new info
 * Improve bug titles to be more informative
 * Add appropriate categories
 * Set appropriate priorities (eg degrade urgent issues with workarounds to bug)
 
The most crucial step in this process is 'intake': dealing with newly-reported issues. We need to quickly and aggressively categorize these and then move them out of the intake stage. Bug reporters are likely to vanish within a couple days so we have only a narrow window to get any information we need from them. Every new bug you look at should be moved to one of these states:

 * security or data loss: CONFIRMED / CRITICAL / BUG
 * regression: CONFIRMED / URGENT / BUG
 * normal bug: CONFIRMED / NORMAL / BUG
 * feature request: CONFIRMED / FEATURE / WISH
 * duplicate: RESOLVED / DUPLICATE
 * not a bug: RESOLVED / INVALID or WONTFIX
 * not enough information: NEED_EXAMPLE

First responders should:

 * try to identify regressions
 * cc: relevant developers
 * ask for essential missing information
 * update the state flags appropriately

First responders should not:

 * suggest work-arounds before root causes are identified
 * leave the bug UNCONFIRMED

== Definitions ==
=== Status levels: ===
 * '''UNCONFIRMED''' - no one's ever responded to this bug (bad)
 * '''CONFIRMED''' - we're discussing a solution
 * '''NEED_EXAMPLE''' - we need more information from the reporter
 * '''IN_PROGRESS''' - a partial fix exists or fix exists but isn't yet integrated
 * '''TESTING''' - a patch is merged, but poster has not confirmed it works, or it didn't reach the main repo yet
 * '''RESOLVED''' - problem is fixed or was not a bug

=== Priorities ===
 * '''critical''' - data loss or security issue
 * '''urgent''' - bug that's blocking development or is a regression
 * '''normal''' - bug that's not blocking development
 * '''wish''' - would be nice

=== Regressions, and why they're a priority ===
We pay special attention to '''regressions''', which are defined as "a bug that breaks something that worked in earlier releases". These issues should immediately be set to urgent. We should attempt to fix these bugs before the next release.

Because regressions may break workflows or tools that users have come to depend on, they are more important than other bugs that simply impede new ways of trying to use Mercurial. Large numbers of unaddressed regressions will also make upgrading risky and unattractive for users, which is unhealthy for a project.

Note that "performance regressions" generally do not meet the above definition unless they're intolerably large, and thus should not necessarily be promoted to urgent.

/!\ Because regressions are higher priority than other bugs, bug fixes in one area that introduce a regression elsewhere are considered worse than no action. This will prevent certain classes of bugs from being fixed.

== HG Bot and the BTS Reaper ==
There is a hook called !HgBot to update the BTS according to changelog commit messages. Summaries containing '(issueNNNN)' will automatically move issues to the testing state. References to `issueNNNN` in the body of the message will create references in that issue to the changeset. You can also explicitly write `closes issueNNN` or `fixes issueNNN` to move the bug into testing, or `see issueNNN` or `addressesNNN` to create a reference in the issue.

There is also a nightly process called the BTS Reaper that automates some bug gardening:

 * close NEED_EXAMPLE reports with no response as INVALID
 * close TESTING reports as RESOLVED/FIXED
 * archive very old CONFIRMED bugs
 * move IN_PROGRESS bugs back to CONFIRMED
 * kick UNCONFIRMED reports (this should never happen)

== Manual bug gardening hints ==
In addition to the above, we should try to keep the BTS data high-quality:

 * adjust bug summary lines to be precise and specific
 * identify components correctly
 * actively discourage discussion of multiple topics in a single report
 * add keywords like EASY
 * try to link related bugs and mark duplicates
 * close bugs that are no longer relevant

=== Marking Duplicates ===
 * choose whichever bug is resolved (or has a clear description) as the master
 * Set superseder on the duplicate to point to the master, add a note, and set the bug as resolved
 * If the master is not resolved, copy the nosy list from the duplicate and add a message pointing back to 'issueXXX'

=== Dealing with third-party issues ===
If the third-party project is large enough to have its own BTS:

 * assign to project maintainer or add URL for proper BTS
 * mark the issue RESOLVED/INVALID

For smaller projects, we'll allow using our BTS to track issues if the maintainer is a known BTS user:

 * assign to project maintainer
 * reduce priority to bug or lower
 * mark '''RESOLVED/INVALID''' if bug remains inactive

For any other projects:

 * link to project homepage or extension page on wiki
 * mark the issue '''RESOLVED/INVALID'''

=== Dealing with patches on the BTS ===
When someone adds a patch to the BTS (which already contains a link to ContributingChanges next to the attach button!), do the following immediately so the patch doesn't sit in limbo for years:

 * immediately assign the bug to the submitter
 * add the keyword "orphan"
 * add a comment that says "Read http://mercurial.selenic.com/wiki/ContributingChanges"

== Attacking the backlog ==
We will likely always have a backlog of issues that exceeds our development capacity, so it's tempting for developers to ignore the BTS and just work on features.

 * please address URGENT bugs as soon as possible
 * please use the freeze window of the release cycle to lower the bug count

== See also ==
 * WritingTests

----
CategoryBugs CategoryDeveloper CategoryProcess

Note:

This page is primarily intended for developers of Mercurial.

Managing Bugs

https://docs.google.com/spreadsheets/d/1lEvdGoN-jgLMZ_1eu8ceXDzEdWGWuDTzPSDVCz4jtxs/embed/oimg?id=1lEvdGoN-jgLMZ_1eu8ceXDzEdWGWuDTzPSDVCz4jtxs&oid=1929125339&zx=w96cwe5uoapn

How to do bug triage on the BugTracker.

1. How to do triage

It's important to approach bug reports as a triage problem. Some reports are five-alarm fires, some are papercuts. We need to make sure that we don't miss the former while we're addressing the latter. And we need to be honest that we're probably not going to have time to address the latter.

The most crucial step in this process is 'intake': dealing with newly-reported issues. We need to quickly and aggressively categorize these and then move them out of the intake stage. Bug reporters are likely to vanish within a couple days so we have only a narrow window to get any information we need from them. Every new bug you look at should be moved to one of these states:

  • security or data loss: CONFIRMED / CRITICAL / BUG
  • regression: CONFIRMED / URGENT / BUG
  • normal bug: CONFIRMED / NORMAL / BUG
  • feature request: CONFIRMED / FEATURE / WISH
  • duplicate: RESOLVED / DUPLICATE
  • not a bug: RESOLVED / INVALID or WONTFIX
  • not enough information: NEED_EXAMPLE

First responders should:

  • try to identify regressions
  • cc: relevant developers
  • ask for essential missing information
  • update the state flags appropriately

First responders should not:

  • suggest work-arounds before root causes are identified
  • leave the bug UNCONFIRMED

2. Definitions

2.1. Status levels:

  • UNCONFIRMED - no one's ever responded to this bug (bad)

  • CONFIRMED - we're discussing a solution

  • NEED_EXAMPLE - we need more information from the reporter

  • IN_PROGRESS - a partial fix exists or fix exists but isn't yet integrated

  • TESTING - a patch is merged, but poster has not confirmed it works, or it didn't reach the main repo yet

  • RESOLVED - problem is fixed or was not a bug

2.2. Priorities

  • critical - data loss or security issue

  • urgent - bug that's blocking development or is a regression

  • normal - bug that's not blocking development

  • wish - would be nice

2.3. Regressions, and why they're a priority

We pay special attention to regressions, which are defined as "a bug that breaks something that worked in earlier releases". These issues should immediately be set to urgent. We should attempt to fix these bugs before the next release.

Because regressions may break workflows or tools that users have come to depend on, they are more important than other bugs that simply impede new ways of trying to use Mercurial. Large numbers of unaddressed regressions will also make upgrading risky and unattractive for users, which is unhealthy for a project.

Note that "performance regressions" generally do not meet the above definition unless they're intolerably large, and thus should not necessarily be promoted to urgent.

/!\ Because regressions are higher priority than other bugs, bug fixes in one area that introduce a regression elsewhere are considered worse than no action. This will prevent certain classes of bugs from being fixed.

3. HG Bot and the BTS Reaper

There is a hook called HgBot to update the BTS according to changelog commit messages. Summaries containing '(issueNNNN)' will automatically move issues to the testing state. References to issueNNNN in the body of the message will create references in that issue to the changeset. You can also explicitly write closes issueNNN or fixes issueNNN to move the bug into testing, or see issueNNN or addressesNNN to create a reference in the issue.

There is also a nightly process called the BTS Reaper that automates some bug gardening:

  • close NEED_EXAMPLE reports with no response as INVALID
  • close TESTING reports as RESOLVED/FIXED
  • archive very old CONFIRMED bugs
  • move IN_PROGRESS bugs back to CONFIRMED
  • kick UNCONFIRMED reports (this should never happen)

4. Manual bug gardening hints

In addition to the above, we should try to keep the BTS data high-quality:

  • adjust bug summary lines to be precise and specific
  • identify components correctly
  • actively discourage discussion of multiple topics in a single report
  • add keywords like EASY
  • try to link related bugs and mark duplicates
  • close bugs that are no longer relevant

4.1. Marking Duplicates

  • choose whichever bug is resolved (or has a clear description) as the master
  • Set superseder on the duplicate to point to the master, add a note, and set the bug as resolved
  • If the master is not resolved, copy the nosy list from the duplicate and add a message pointing back to 'issueXXX'

4.2. Dealing with third-party issues

If the third-party project is large enough to have its own BTS:

  • assign to project maintainer or add URL for proper BTS
  • mark the issue RESOLVED/INVALID

For smaller projects, we'll allow using our BTS to track issues if the maintainer is a known BTS user:

  • assign to project maintainer
  • reduce priority to bug or lower
  • mark RESOLVED/INVALID if bug remains inactive

For any other projects:

  • link to project homepage or extension page on wiki
  • mark the issue RESOLVED/INVALID

4.3. Dealing with patches on the BTS

When someone adds a patch to the BTS (which already contains a link to ContributingChanges next to the attach button!), do the following immediately so the patch doesn't sit in limbo for years:

5. Attacking the backlog

We will likely always have a backlog of issues that exceeds our development capacity, so it's tempting for developers to ignore the BTS and just work on features.

  • please address URGENT bugs as soon as possible
  • please use the freeze window of the release cycle to lower the bug count

6. See also


CategoryBugs CategoryDeveloper CategoryProcess

ManagingBugs (last edited 2015-03-11 18:18:47 by mpm)