Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Side by Side Diff: net/docs/bug-triage.md

Issue 1017743002: [net] Convert bug triage documents to Markdown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | net/docs/bug-triage.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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.
OLDNEW
« no previous file with comments | « no previous file | net/docs/bug-triage.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698