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

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

Issue 1779203002: Net triage docs: Fix links, mention crashes on other platforms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 | no next file » | 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, in rough order of priority: 9 ### Required, in rough order of priority:
10 * Identify new network bugs on the tracker. 10 * Identify new network bugs on the tracker.
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 the responsibility remains with the triager to continue investigating the 54 the responsibility remains with the triager to continue investigating the
55 bug, as above. 55 bug, as above.
56 56
57 * The triager is responsible for looking at any notification previous 57 * The triager is responsible for looking at any notification previous
58 triagers did not, so when an issue is investigated, the person who did 58 triagers did not, so when an issue is investigated, the person who did
59 so should respond to chrome-network-debugging@google.com with a short 59 so should respond to chrome-network-debugging@google.com with a short
60 email, describing their conclusions. Future triagers can then use the 60 email, describing their conclusions. Future triagers can then use the
61 fact an alert was responded to as an inidicator of which of them need 61 fact an alert was responded to as an inidicator of which of them need
62 to be followed up on. 62 to be followed up on.
63 63
64 * Investigate [Uncomfirmed / Untriaged Internals>Network issues that don't 64 * Investigate [Uncomfirmed / Untriaged Internals>Network issues that don't belon g to a more specific network component](https://bugs.chromium.org/p/chromium/iss ues/list?can=2&q=component%3DInternals%3ENetwork+status%3AUnconfirmed,Untriaged+ -label:Needs-Feedback&sort=-modified),
65 belong to a more specific network component](https://bugs.chromium.org/p/chrom ium/issues/list?can=2&q=component%3DInternals%3ENetwork+status%3AUnconfirmed,Unt riaged+-label:Needs-Feedback&sort=-modified),
66 prioritizing the most recent issues, ones with the most responsive reporters, 65 prioritizing the most recent issues, ones with the most responsive reporters,
67 and major crashers. This will generally take up the majority of your time as 66 and major crashers. This will generally take up the majority of your time as
68 triager. Continue digging until you can do one of the following: 67 triager. Continue digging until you can do one of the following:
69 68
70 * Mark it as *WontFix* (working as intended, obsolete issue) or a 69 * Mark it as *WontFix* (working as intended, obsolete issue) or a
71 duplicate. 70 duplicate.
72 71
73 * Mark it as a feature request. 72 * Mark it as a feature request.
74 73
75 * Mark it as Needs-Feedback. 74 * Mark it as Needs-Feedback.
76 75
77 * Remove the Internals>Network component, replacing it with at least one 76 * Remove the Internals>Network component, replacing it with at least one
78 more specific network component or non-network component. Replacing the 77 more specific network component or non-network component. Replacing the
79 Internals>Network component gets it off the next triager's radar, and 78 Internals>Network component gets it off the next triager's radar, and
80 in front of someone more familiar with the relevant code. Note that 79 in front of someone more familiar with the relevant code. Note that
81 due to the way the bug report wizard works, a lot of bugs incorrectly end 80 due to the way the bug report wizard works, a lot of bugs incorrectly end
82 up with the network component. 81 up with the network component.
83 82
84 * The issue is assigned to an appropriate owner, and make sure to mark it 83 * The issue is assigned to an appropriate owner, and make sure to mark it
85 as "assigned" so the next triager doesn't run into it. 84 as "assigned" so the next triager doesn't run into it.
86 85
87 * If there is no more specific component for a bug, it should be 86 * If there is no more specific component for a bug, it should be
88 investigated by the triager until we have a good understanding of the 87 investigated by the triager until we have a good understanding of the
89 cause of the problem, and some idea how it should be fixed, at which point 88 cause of the problem, and some idea how it should be fixed, at which point
90 its status should be set to Available. Future triagers should ignore bugs 89 its status should be set to Available. Future triagers should ignore bugs
91 with this status, unless investigating stale bugs. 90 with this status, unless investigating stale bugs.
92 91
93 * Follow up on [Needs-Feedback issues for all components owned by the network 92 * Follow up on [Needs-Feedback issues for all components owned by the network st ack team](https://bugs.chromium.org/p/chromium/issues/list?can=2&q=component%3AI nternals%3ENetwork%2CUI>Browser>Downloads+-component%3AInternals%3ENetwork%3EDat aProxy+-component%3AInternals%3ENetwork%3EDataUse+-component%3AInternals%3ENetwo rk%3EVPN+Needs%3DFeedback&sort=-modified).
94 stack team](https://bugs.chromium.org/p/chromium/issues/list?can=2&q=component %3AInternals%3ENetwork%2CUI>Browser>Downloads+-component%3AInternals%3ENetwork%3 EDataProxy+-component%3AInternals%3ENetwork%3EDataUse+-component%3AInternals%3EN etwork%3EVPN+Needs%3DFeedback).
mmenke 2016/03/10 17:36:15 Strangely, splitting the lines here didn't work on
Randy Smith (Not in Mondays) 2016/03/10 18:14:10 Huh. How did you check Googlesource's md engine b
mmenke 2016/03/10 19:18:35 I didn't, I'm describing how it's behaving with th
95 93
96 * Remove label once feedback is provided. Continue to investigate, if 94 * Remove label once feedback is provided. Continue to investigate, if
97 the previous section applies. 95 the previous section applies.
98 96
99 * If the Needs-Feedback label has been present for one week, ping the 97 * If the Needs-Feedback label has been present for one week, ping the
100 reporter. 98 reporter.
101 99
102 * Archive after two weeks with no feedback, telling users to file a new 100 * Archive after two weeks with no feedback, telling users to file a new
103 bug if they still have the issue, with the requested information, unless 101 bug if they still have the issue, with the requested information, unless
104 the reporter indicates they'll provide data when they can. In that case, 102 the reporter indicates they'll provide data when they can. In that case,
105 use your own judgment for further pings or archiving. 103 use your own judgment for further pings or archiving.
106 104
107 * Identify significant new browser process 105 * Identify significant new browser process
108 [crashers](https://goto.google.com/chromecrash) that are potentially network 106 [crashers](https://goto.google.com/chromecrash) that are potentially network
109 related. You should look at crashes for the most recent canary that has at 107 related. You should look at crashes for the most recent canary that has at
110 least a day of data, and if there's been a dev or beta release from the start 108 least a day of data, and if there's been a dev or beta release from the start
111 of the last triager's shift to the start of yours, you should also look at 109 of the last triager's shift to the start of yours, you should also look at
112 that once it has at least a day of data. Recent releases available 110 that once it has at least a day of data. Recent releases available
113 [here](https://omahaproxy.appspot.com/). If both dev and beta have been 111 [here](https://omahaproxy.appspot.com/). If both dev and beta have been
114 released in that period, just look at beta. File Internals>Network bugs on 112 released in that period, just look at beta. File Internals>Network bugs on
115 the tracker when new crashers are found. Bugs should only be filed for 113 the tracker when new crashers are found. Bugs should only be filed for
116 crashes that are both in the top 100 for each release and occurred for more 114 crashes that are both in the top 100 for each release and occurred for more
117 than two users. 115 than two users.
118 116
117 * Make sure to check for new crashes on all platforms, not just Windows.
118
119 ### Best Effort (As you have time): 119 ### Best Effort (As you have time):
120 120
121 * Investigate old bugs, and bugs associated with Internals>Network 121 * Investigate old bugs, and bugs associated with Internals>Network
122 subcomponents. 122 subcomponents.
123 123
124 * Investigate unowned and owned but forgotten net/ crashers that are still 124 * Investigate unowned and owned but forgotten net/ crashers that are still
125 occurring (As indicated by 125 occurring (As indicated by
126 [go/chromecrash](https://goto.google.com/chromecrash)), prioritizing frequent 126 [go/chromecrash](https://goto.google.com/chromecrash)), prioritizing frequent
127 and long standing crashers. 127 and long standing crashers.
128 128
129 * Close obsolete bugs. 129 * Close obsolete bugs.
130 130
131 See [bug-triage-suggested-workflow.md](bug-triage-suggested-workflow.md) for 131 See [bug-triage-suggested-workflow.md](bug-triage-suggested-workflow.md) for
132 suggested workflows. 132 suggested workflows.
133 133
134 See [bug-triage-labels.md](bug-triage-labels.md) for labeling tips for network 134 See [bug-triage-labels.md](bug-triage-labels.md) for labeling tips for network
135 and non-network bugs. 135 and non-network bugs.
136 136
137 See [crash-course-in-net-internals.md](crash-course-in-net-internals.md) for 137 See [crash-course-in-net-internals.md](crash-course-in-net-internals.md) for
138 some help on getting started with about:net-internals debugging. 138 some help on getting started with about:net-internals debugging.
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698