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

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

Issue 1836343002: Tweak network doc wording, fix link, and make order in workflow match (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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
« net/docs/bug-triage.md ('K') | « net/docs/bug-triage.md ('k') | 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 : Suggested Workflow 1 # Chrome Network Bug Triage : Suggested Workflow
2 2
3 [TOC] 3 [TOC]
4 4
5 ## Identifying unlabeled network bugs on the tracker 5 ## Identifying unlabeled network bugs on the tracker
6 6
7 * Look at new uncomfirmed bugs since noon PST on the last triager's rotation. 7 * Look at new uncomfirmed bugs since noon PST on the last triager's rotation.
8 [Use this issue tracker 8 [Use this issue tracker
9 query](https://code.google.com/p/chromium/issues/list?can=2&q=status%3Aunconfi rmed&sort=-id&num=1000). 9 query](https://code.google.com/p/chromium/issues/list?can=2&q=status%3Aunconfi rmed&sort=-id&num=1000).
10 10
(...skipping 11 matching lines...) Expand all
22 * If there's just a blank form and a crash ID, just ignore the bug. 22 * If there's just a blank form and a crash ID, just ignore the bug.
23 23
24 * If network causes are possible, ask for a net-internals log (If it's not a 24 * If network causes are possible, ask for a net-internals log (If it's not a
25 browser crash) and attach the most specific internals-network label that's 25 browser crash) and attach the most specific internals-network label that's
26 applicable. If there isn't an applicable narrower component, a clear owner 26 applicable. If there isn't an applicable narrower component, a clear owner
27 for the issue, or there are multiple possibilities, attach the 27 for the issue, or there are multiple possibilities, attach the
28 Internals>Network component and proceed with further investigation. 28 Internals>Network component and proceed with further investigation.
29 29
30 * If non-network causes also seem possible, attach those components as well. 30 * If non-network causes also seem possible, attach those components as well.
31 31
32 ## Investigate UMA notifications
33
34 For each alert that fires, determine if it's a real alert and file a bug if so.
35
36 * Don't file if the alert is coincident with a major volume change. The volume
37 at a particular date can be determined by hovering the mouse over the
38 appropriate location on the alert line.
39
40 * Don't file if the alert is on a graph with very low volume (< ~200 data
41 points); it's probably noise, and we probably don't care even if it isn't.
42
43 * Don't file if the graph is really noisy (but eyeball it to decide if there is
44 an underlying important shift under the noise).
45
46 * Don't file if the alert is in the "Known Ignorable" list:
47 * SimpleCache on Windows
48 * DiskCache on Android.
49
32 ## Investigating component=Internals>Network bugs 50 ## Investigating component=Internals>Network bugs
33 51
34 * It's recommended that while on triage duty, you subscribe to the 52 * It's recommended that while on triage duty, you subscribe to the
35 Internals>Network component (but not its subcomponents). To do this, go 53 Internals>Network component (but not its subcomponents). To do this, go
36 to the issue tracker and then click "Saved Queries". 54 to the issue tracker and then click "Saved Queries".
37 Add a query with these settings: 55 Add a query with these settings:
38 * Saved query name: Network Bug Triage 56 * Saved query name: Network Bug Triage
39 * Project: chromium 57 * Project: chromium
40 * Query: component=Internals>Network 58 * Query: component=Internals>Network
41 * Subscription options: Notify Immediately 59 * Subscription options: Notify Immediately
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 * If you are having trouble with an issue, particularly for help understanding 118 * If you are having trouble with an issue, particularly for help understanding
101 net-internals logs, email the public net-dev@chromium.org list for help 119 net-internals logs, email the public net-dev@chromium.org list for help
102 debugging. If it's a crasher, or for some other reason discussion needs to 120 debugging. If it's a crasher, or for some other reason discussion needs to
103 be done in private, use chrome-network-debugging@google.com. TODO(mmenke): 121 be done in private, use chrome-network-debugging@google.com. TODO(mmenke):
104 Write up a net-internals tips and tricks docs. 122 Write up a net-internals tips and tricks docs.
105 123
106 * If it appears to be a bug in the unowned core of the network stack (i.e. no 124 * If it appears to be a bug in the unowned core of the network stack (i.e. no
107 subcomponent applies, or only the Internals>Network>HTTP subcomponent 125 subcomponent applies, or only the Internals>Network>HTTP subcomponent
108 applies, and there's no clear owner), try to figure out the exact cause. 126 applies, and there's no clear owner), try to figure out the exact cause.
109 127
110 ## Investigate UMA notifications
111
112 For each alert that fires, determine if it's a real alert and file a bug if so.
113
114 * Don't file if the alert is coincident with a major volume change. The volume
115 at a particular date can be determined by hovering the mouse over the
116 appropriate location on the alert line.
117
118 * Don't file if the alert is on a graph with very low volume (< ~200 data
119 points); it's probably noise, and we probably don't care even if it isn't.
120
121 * Don't file if the graph is really noisy (but eyeball it to decide if there is
122 an underlying important shift under the noise).
123
124 * Don't file if the alert is in the "Known Ignorable" list:
125 * SimpleCache on Windows
126 * DiskCache on Android.
127
128 ## Looking for new crashers 128 ## Looking for new crashers
129 129
130 1. Go to [go/chromecrash](https://goto.google.com/chromecrash). 130 1. Go to [go/chromecrash](https://goto.google.com/chromecrash).
131 131
132 2. For each platform, look through the releases for which releases to 132 2. For each platform, look through the releases for which releases to
133 investigate. As per bug-triage.txt, this should be the most recent canary, 133 investigate. As per bug-triage.txt, this should be the most recent canary,
134 the previous canary (if the most recent is less than a day old), and any of 134 the previous canary (if the most recent is less than a day old), and any of
135 dev/beta/stable that were released in the last couple of days. 135 dev/beta/stable that were released in the last couple of days.
136 136
137 3. For each release, in the "Process Type" frame, click on "browser". 137 3. For each release, in the "Process Type" frame, click on "browser".
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 a crash report, and seeing if there are multiple reports for the same crash. 201 a crash report, and seeing if there are multiple reports for the same crash.
202 If this is the case, it may be also be malware, or an issue with an unusual 202 If this is the case, it may be also be malware, or an issue with an unusual
203 system/chrome/network config. 203 system/chrome/network config.
204 204
205 * Dig through crash reports to figure out when the crash first appeared, and 205 * Dig through crash reports to figure out when the crash first appeared, and
206 dig through revision history in related files to try and locate a suspect CL. 206 dig through revision history in related files to try and locate a suspect CL.
207 TODO(mmenke): Add more detail here. 207 TODO(mmenke): Add more detail here.
208 208
209 * Load crash dumps, try to figure out a cause. See 209 * Load crash dumps, try to figure out a cause. See
210 http://www.chromium.org/developers/crash-reports for more information 210 http://www.chromium.org/developers/crash-reports for more information
OLDNEW
« net/docs/bug-triage.md ('K') | « net/docs/bug-triage.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698