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

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

Issue 2844323002: A couple of minor updates to triage instructions. (Closed)
Patch Set: Created 3 years, 7 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 : 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 unconfirmed bugs since noon PST on the last triager's rotation. 7 * Look at new unconfirmed bugs since noon PST on the last triager's rotation.
8 [Use this issue tracker 8 [Use this issue tracker
9 query](https://bugs.chromium.org/p/chromium/issues/list?q=status%3Aunconfirmed &sort=-id&num=1000). 9 query](https://bugs.chromium.org/p/chromium/issues/list?q=status%3Aunconfirmed &sort=-id&num=1000).
10 10
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 42
43 * Don't file if the graph is really noisy (but eyeball it to decide if there is 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). 44 an underlying important shift under the noise).
45 45
46 * Don't file if the alert is in the "Known Ignorable" list: 46 * Don't file if the alert is in the "Known Ignorable" list:
47 * SimpleCache on Windows 47 * SimpleCache on Windows
48 * DiskCache on Android. 48 * DiskCache on Android.
49 49
50 ## Investigating component=Internals>Network bugs 50 ## Investigating component=Internals>Network bugs
51 51
52 * Note that you may want to investigate Needs-Feedback bugs first, as
53 that may result in some bugs being added to this list.
54
52 * It's recommended that while on triage duty, you subscribe to the 55 * It's recommended that while on triage duty, you subscribe to the
53 Internals>Network component (but not its subcomponents). To do this, go 56 Internals>Network component (but not its subcomponents). To do this, go
54 to the issue tracker and then click "Saved Queries". 57 to the issue tracker and then click "Saved Queries".
55 Add a query with these settings: 58 Add a query with these settings:
56 * Saved query name: Network Bug Triage 59 * Saved query name: Network Bug Triage
57 * Project: chromium 60 * Project: chromium
58 * Query: component=Internals>Network 61 * Query: component=Internals>Network
59 * Subscription options: Notify Immediately 62 * Subscription options: Notify Immediately
60 63
61 * Look through unconfirmed and untriaged component=Internals>Network bugs, 64 * Look through unconfirmed and untriaged component=Internals>Network bugs,
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 3. Removing the "product.version='X.Y.Z.W' AND" string and clicking 173 3. Removing the "product.version='X.Y.Z.W' AND" string and clicking
171 "Update". 174 "Update".
172 4. Clicking "Limit 1000" in the Product Version list in the 175 4. Clicking "Limit 1000" in the Product Version list in the
173 resulting page (without this, the listing will be restricted to 176 resulting page (without this, the listing will be restricted to
174 the releases in which the signature is most common, which will 177 the releases in which the signature is most common, which will
175 often not include the canary/dev release being investigated). 178 often not include the canary/dev release being investigated).
176 5. Choose some subset of that list, or all of it, to include in the 179 5. Choose some subset of that list, or all of it, to include in the
177 bug. Make sure to indicate if there is a defined point in the 180 bug. Make sure to indicate if there is a defined point in the
178 past before which the signature is not present. 181 past before which the signature is not present.
179 182
183 As an alternative to the above, you can use [Eric Roman's new crash
184 tool](https://ericroman.users.x20web.corp.google.com/www/net-crash-triage/index. html)
185 (internal link). Note that it isn't a perfect fit with the triage
186 responsibilities, specifically:
187
188 * It's only showing Windows releases; Android, iOS, and WebView are
189 usually different, and Mac is sometimes different.
190 * The instructions are to look at the latest canary which has a days
191 worth of data. If canaries are being pushed fast, that may be more
192 than one canary into the past, and hence not visible on the tool.
193 * Eric's tool filters based on files in "src/net" rather than looking
194 for magic signature's including the string "net::" ("src/net" is
195 probably the better filter).
196
180 ## Investigating crashers 197 ## Investigating crashers
181 198
182 * Only investigate crashers that are still occurring, as identified by above 199 * Only investigate crashers that are still occurring, as identified by above
183 section. If a search on go/crash indicates a crasher is no longer occurring, 200 section. If a search on go/crash indicates a crasher is no longer occurring,
184 mark it as WontFix. 201 mark it as WontFix.
185 202
186 * On Windows, you may want to look for weird dlls associated with the crashes. 203 * On Windows, you may want to look for weird dlls associated with the crashes.
187 This generally needs crashes from a fair number of different users to reach 204 This generally needs crashes from a fair number of different users to reach
188 any conclusions. 205 any conclusions.
189 * To get a list of loaded modules in related crash dumps, select 206 * To get a list of loaded modules in related crash dumps, select
(...skipping 12 matching lines...) Expand all
202 a crash report, and seeing if there are multiple reports for the same crash. 219 a crash report, and seeing if there are multiple reports for the same crash.
203 If this is the case, it may be also be malware, or an issue with an unusual 220 If this is the case, it may be also be malware, or an issue with an unusual
204 system/chrome/network config. 221 system/chrome/network config.
205 222
206 * Dig through crash reports to figure out when the crash first appeared, and 223 * Dig through crash reports to figure out when the crash first appeared, and
207 dig through revision history in related files to try and locate a suspect CL. 224 dig through revision history in related files to try and locate a suspect CL.
208 TODO(mmenke): Add more detail here. 225 TODO(mmenke): Add more detail here.
209 226
210 * Load crash dumps, try to figure out a cause. See 227 * Load crash dumps, try to figure out a cause. See
211 http://www.chromium.org/developers/crash-reports for more information 228 http://www.chromium.org/developers/crash-reports for more information
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