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

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

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

Powered by Google App Engine
This is Rietveld 408576698