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

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

Issue 935333002: Update from https://crrev.com/316786 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 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 | « net/docs/bug-triage.txt ('k') | net/http/disk_cache_based_quic_server_info.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 Identifying unlabeled network bugs on the tracker: 1 Identifying unlabeled network bugs on the tracker:
2 * Look at new uncomfirmed bugs since noon PST on the last triager's rotation: 2 * Look at new uncomfirmed bugs since noon PST on the last triager's rotation:
3 https://code.google.com/p/chromium/issues/list?can=2&q=status%3Aunconfirmed& sort=-id&num=1000 3 https://code.google.com/p/chromium/issues/list?can=2&q=status%3Aunconfirmed& sort=-id&num=1000
4 * Press "h" to bring up a preview of the bug text. 4 * Press "h" to bring up a preview of the bug text.
5 * Use "j" and "k" to advance through bugs. 5 * Use "j" and "k" to advance through bugs.
6 * If a bug looks like it might be network/download/safe-browsing related, middle 6 * If a bug looks like it might be network/download/safe-browsing related, middle
7 click [or command-click on OSX] to open in new tab. 7 click [or command-click on OSX] to open in new tab.
8 * If a user provides a crash ID for a crasher for a bug that could be 8 * If a user provides a crash ID for a crasher for a bug that could be
9 net-related, look at the crash stack at go/crash, and see if it looks to be 9 net-related, look at the crash stack at go/crash, and see if it looks to be
10 network related. Be sure to check if other bug reports have that stack 10 network related. Be sure to check if other bug reports have that stack
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 net-internals logs, email the public net-dev@chromium.org list for help 70 net-internals logs, email the public net-dev@chromium.org list for help
71 debugging. If it's a crasher, or for some other reason discussion needs to 71 debugging. If it's a crasher, or for some other reason discussion needs to
72 be done in private, use chrome-network-debugging@google.com. 72 be done in private, use chrome-network-debugging@google.com.
73 TODO(mmenke): Write up a net-internals tips and tricks docs. 73 TODO(mmenke): Write up a net-internals tips and tricks docs.
74 * If it appears to be a bug in the unowned core of the network stack (i.e. no 74 * If it appears to be a bug in the unowned core of the network stack (i.e. no
75 sublabel applies, or only the Cr-Internals-Network-HTTP sublabel applies, 75 sublabel applies, or only the Cr-Internals-Network-HTTP sublabel applies,
76 and there's no clear owner), try to figure out the exact cause. 76 and there's no clear owner), try to figure out the exact cause.
77 77
78 Look for new crashers: 78 Look for new crashers:
79 * Go to go/chromecrash. 79 * Go to go/chromecrash.
80 * For each platform, go to the latest canary. Click on browser -> limit 1000. 80 * For each platform, go to the latest canary.
81 Search the page for "net::". Ignore crashes that only occur once, as 81 * In the "Process Type" frame, click on "browser".
82 * At the bottom of the "Magic Signature" frame, click "limit 1000".
83 Reported crashers are sorted in decreasing order of the number of reports for
84 that crash signature.
85 * Search the page for "net::". Ignore crashes that only occur once, as
82 memory corruption can easily cause one-off failures when the sample size is 86 memory corruption can easily cause one-off failures when the sample size is
83 large enough. 87 large enough.
84 * Look at the stack trace to confirm it's a network bug. 88 * Click on the number of reports field to see details of crash. Look at the
89 stack trace to confirm it's a network bug.
85 * If it is, and there's no associated bug filed, file a new bug directly from 90 * If it is, and there's no associated bug filed, file a new bug directly from
86 chromecrash, looking at earlier canaries to determine if it's a recent 91 chromecrash, looking at earlier canaries to determine if it's a recent
87 regression. Use the most specific label possible. 92 regression. Use the most specific label possible.
88 * The most recent Canary may not yet have a full day of crashes, so it may be 93 * The most recent Canary may not yet have a full day of crashes, so it may be
89 worth looking at more than one version. 94 worth looking at more than one version.
90 * If there's been a dev, beta, or stable release in the last couple days, should 95 * If there's been a dev, beta, or stable release in the last couple days, should
91 also look at those. 96 also look at those.
92 97
93 Investigating crashers: 98 Investigating crashers:
94 * Only investigate crashers that are still occurring, as identified by above 99 * Only investigate crashers that are still occurring, as identified by above
(...skipping 24 matching lines...) Expand all
119 * If a bug is over 2 months old, and the underlying problem was never 124 * If a bug is over 2 months old, and the underlying problem was never
120 reproduced or really understood: 125 reproduced or really understood:
121 * If it's over a year old, go ahead and mark the issue as Archived. 126 * If it's over a year old, go ahead and mark the issue as Archived.
122 * Otherwise, ask reporters if the issue is still present, and attach the 127 * Otherwise, ask reporters if the issue is still present, and attach the
123 Needs-Feedback label. 128 Needs-Feedback label.
124 * Old unconfirmed or untriaged Cr-Internals-Network issues can be investigated 129 * Old unconfirmed or untriaged Cr-Internals-Network issues can be investigated
125 just like newer ones. Crashers should generally be given higher priority, 130 just like newer ones. Crashers should generally be given higher priority,
126 since we can verify if they still occur, and then newer issues, as they're 131 since we can verify if they still occur, and then newer issues, as they're
127 more likely to still be present, and more likely to have a still responsive 132 more likely to still be present, and more likely to have a still responsive
128 bug reporter. 133 bug reporter.
OLDNEW
« no previous file with comments | « net/docs/bug-triage.txt ('k') | net/http/disk_cache_based_quic_server_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698