Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 # Chrome Network Bug Triage | |
| 2 | |
| 1 The Chrome network team uses a two day bug triage rotation. The main goals are | 3 The Chrome network team uses a two day bug triage rotation. The main goals are |
| 2 to identify and label new network bugs, and investigate network bugs when no | 4 to identify and label new network bugs, and investigate network bugs when no |
| 3 label seems suitable. | 5 label seems suitable. |
| 4 | 6 |
| 5 Responsibilities | 7 ## Responsibilities |
| 6 | 8 |
| 7 Required: | 9 ### Required: |
| 8 * Identify new crashers | 10 * Identify new crashers |
| 9 * Identify new network issues. | 11 * Identify new network issues. |
| 10 * Request data about recent Cr-Internals-Network issue. | 12 * Request data about recent Cr-Internals-Network issue. |
| 11 * Investigate each recent Cr-Internals-Network issue. | 13 * Investigate each recent Cr-Internals-Network issue. |
| 12 * Monitor UMA histograms and gasper alerts. | 14 * Monitor UMA histograms and gasper alerts. |
| 13 | 15 |
| 14 Best effort: | 16 ### Best effort: |
| 15 * Investigate unowned and owned-but-forgotten net/ crashers | 17 * Investigate unowned and owned-but-forgotten net/ crashers |
| 16 * Investigate old bugs | 18 * Investigate old bugs |
| 17 * Close obsolete bugs. | 19 * Close obsolete bugs. |
| 18 | 20 |
| 19 All of the above is to be done on each rotation. These | 21 All of the above is to be done on each rotation. These responsibilities should |
| 20 responsibilities should be tracked, and anything left undone at the | 22 be tracked, and anything left undone at the end of a rotation should be handed |
| 21 end of a rotation should be handed off to the next triager. The | 23 off to the next triager. The downside to passing along bug investigations like |
| 22 downside to passing along bug investigations like this is each new | 24 this is each new triager has to get back up to speed on bugs the previous |
| 23 triager has to get back up to speed on bugs the previous triager was | 25 triager was investigating. The upside is that triagers don't get stuck |
| 24 investigating. The upside is that triagers don't get stuck | 26 investigating issues after their time after their rotation, and it results in a |
| 25 investigating issues after their time after their rotation, and it | 27 uniform, predictable two day commitment for all triagers. |
| 26 results in a uniform, predictable two day commitment for all triagers. | |
| 27 | 28 |
| 28 More detail: | 29 ## Details |
| 29 | 30 |
| 30 Required activities: | 31 ### Required: |
| 32 | |
| 31 * Identify new crashers that are potentially network related. You should check | 33 * Identify new crashers that are potentially network related. You should check |
| 32 the most recent canary, the previous canary (if the most recent less than a | 34 the most recent canary, the previous canary (if the most recent less than a |
| 33 day old), and any of dev/beta/stable that were released in the last couple | 35 day old), and any of dev/beta/stable that were released in the last couple of |
| 34 of days, for each platform. File Cr-Internals-Network bugs on the tracker | 36 days, for each platform. File Cr-Internals-Network bugs on the tracker when |
| 35 when new crashers are found. | 37 new crashers are found. |
| 38 | |
| 36 * Identify new network bugs, both on the bug tracker and on the crash server. | 39 * Identify new network bugs, both on the bug tracker and on the crash server. |
| 37 All Unconfirmed issues filed during your triage rotation should be scanned, | 40 All Unconfirmed issues filed during your triage rotation should be scanned, |
| 38 and, for suspected network bugs, a network label assigned. A triager is | 41 and, for suspected network bugs, a network label assigned. A triager is |
| 39 responsible for looking at bugs reported from noon PST / 3:00 pm EST of the | 42 responsible for looking at bugs reported from noon PST / 3:00 pm EST of the |
| 40 last day of the previous triager's rotation until the same time on the last | 43 last day of the previous triager's rotation until the same time on the last |
| 41 day of their rotation. | 44 day of their rotation. |
| 45 | |
| 42 * Investigate each recent (New comment within the past week or so) | 46 * Investigate each recent (New comment within the past week or so) |
| 43 Cr-Internals-Network issue, driving getting information from reporters as | 47 Cr-Internals-Network issue, driving getting information from reporters as |
| 44 needed, until you can do one of the following: | 48 needed, until you can do one of the following: |
|
Randy Smith (Not in Mondays)
2015/03/18 16:25:59
The HTML output following this is still not indent
asanka
2015/03/18 20:47:31
Done.
| |
| 45 * Mark it as WontFix (working as intended, obsolete issue) or a duplicate. | 49 |
| 50 * Mark it as *WontFix* (working as intended, obsolete issue) or a duplicate. | |
| 51 | |
| 46 * Mark it as a feature request. | 52 * Mark it as a feature request. |
| 53 | |
| 47 * Remove the Cr-Internals-Network label, replacing it with at least one more | 54 * Remove the Cr-Internals-Network label, replacing it with at least one more |
| 48 specific network label or non-network label. Promptly adding non-network | 55 specific network label or non-network label. Promptly adding non-network |
| 49 labels when appropriate is important to get new bugs in front of someone | 56 labels when appropriate is important to get new bugs in front of someone |
| 50 familiar with the relevant code, and to remove them from the next triager's | 57 familiar with the relevant code, and to remove them from the next triager's |
| 51 radar. Because of the way the bug report wizard works, a lot of bugs | 58 radar. Because of the way the bug report wizard works, a lot of bugs |
| 52 incorrectly end up with the network label. | 59 incorrectly end up with the network label. |
| 60 | |
| 53 * The issue is assigned to an appropriate owner. | 61 * The issue is assigned to an appropriate owner. |
| 62 | |
| 54 * If there is no more specific label for a bug, it should be investigated | 63 * If there is no more specific label for a bug, it should be investigated |
| 55 until we have a good understanding of the cause of the problem, and some | 64 until we have a good understanding of the cause of the problem, and some |
| 56 idea how it should be fixed, at which point its status should be set to | 65 idea how it should be fixed, at which point its status should be set to |
| 57 Available. Future triagers should ignore bugs with this status, unless | 66 Available. Future triagers should ignore bugs with this status, unless |
| 58 investigating stale bugs. | 67 investigating stale bugs. |
| 68 | |
| 59 * Monitor UMA histograms and gasper alerts. | 69 * Monitor UMA histograms and gasper alerts. |
| 60 * For each Gasper alert that fires, the triager should determine if | |
| 61 the alert is real (not due to noise), and file a bug with the | |
| 62 appropriate label if so. Note that if no label more specific than | |
| 63 Cr-Internals-Network is appropriate, the responsibility remains | |
| 64 with the triager to continue investigating the bug, as above. | |
| 65 | 70 |
| 66 Best Effort (As you have time): | 71 * For each Gasper alert that fires, the triager should determine if the alert |
| 72 is real (not due to noise), and file a bug with the appropriate label if | |
| 73 so. Note that if no label more specific than Cr-Internals-Network is | |
| 74 appropriate, the responsibility remains with the triager to continue | |
| 75 investigating the bug, as above. | |
| 76 | |
| 77 ### Best Effort (As you have time): | |
| 67 * Investigate unowned and owned but forgotten net/ crashers that are still | 78 * Investigate unowned and owned but forgotten net/ crashers that are still |
| 68 occurring (As indicated by go/chromecrash), prioritizing frequent and long | 79 occurring (As indicated by |
| 69 standing crashers. | 80 [go/chromecrash](https://goto.google.com/chromecrash)), prioritizing frequent |
| 81 and long standing crashers. | |
| 82 | |
| 70 * Investigate old bugs, prioritizing the most recent. | 83 * Investigate old bugs, prioritizing the most recent. |
| 84 | |
| 71 * Close obsolete bugs. | 85 * Close obsolete bugs. |
| 72 | 86 |
| 73 If you've investigated an issue (in code you don't normally work on) to an | 87 If you've investigated an issue (in code you don't normally work on) to an |
| 74 extent that you know how to fix it, and the fix is simple, feel free to take | 88 extent that you know how to fix it, and the fix is simple, feel free to take |
| 75 ownership of the issue and create a patch while on triage duty, but other tasks | 89 ownership of the issue and create a patch while on triage duty, but other tasks |
| 76 should take priority. | 90 should take priority. |
| 77 | 91 |
| 78 See bug-triage-suggested-workflow.txt for suggested workflows. | 92 See [bug-triage-suggested-workflow.md](bug-triage-suggested-workflow.md) for |
| 79 See bug-triage-labels.txt for labeling tips for network and non-network bugs. | 93 suggested workflows. |
| 94 | |
| 95 See [bug-triage-labels.md](bug-triage-labels.md) for labeling tips for network | |
| 96 and non-network bugs. | |
| OLD | NEW |