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

Unified Diff: net/docs/bug-triage.md

Issue 1017743002: [net] Convert bug triage documents to Markdown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/docs/bug-triage.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/docs/bug-triage.md
diff --git a/net/docs/bug-triage.md b/net/docs/bug-triage.md
new file mode 100644
index 0000000000000000000000000000000000000000..58d4c7491836f60bf790b846c5fac74cc34bb097
--- /dev/null
+++ b/net/docs/bug-triage.md
@@ -0,0 +1,98 @@
+# Chrome Network Bug Triage
+
+The Chrome network team uses a two day bug triage rotation. The main goals are
+to identify and label new network bugs, and investigate network bugs when no
+label seems suitable.
+
+## Responsibilities
+
+### Required:
+* Identify new crashers
+* Identify new network issues.
+* Request data about recent Cr-Internals-Network issue.
+* Investigate each recent Cr-Internals-Network issue.
+* Monitor UMA histograms and gasper alerts.
+
+### Best effort:
+* Investigate unowned and owned-but-forgotten net/ crashers
+* Investigate old bugs
+* Close obsolete bugs.
+
+All of the above is to be done on each rotation. These responsibilities should
+be tracked, and anything left undone at the end of a rotation should be handed
+off to the next triager. The downside to passing along bug investigations like
+this is each new triager has to get back up to speed on bugs the previous
+triager was investigating. The upside is that triagers don't get stuck
+investigating issues after their time after their rotation, and it results in a
+uniform, predictable two day commitment for all triagers.
+
+## Details
+
+### Required:
+
+* Identify new crashers that are potentially network related. You should check
+ the most recent canary, the previous canary (if the most recent less than a
+ day old), and any of dev/beta/stable that were released in the last couple of
+ days, for each platform. File Cr-Internals-Network bugs on the tracker when
+ new crashers are found.
+
+* Identify new network bugs, both on the bug tracker and on the crash server.
+ All Unconfirmed issues filed during your triage rotation should be scanned,
+ and, for suspected network bugs, a network label assigned. A triager is
+ responsible for looking at bugs reported from noon PST / 3:00 pm EST of the
+ last day of the previous triager's rotation until the same time on the last
+ day of their rotation.
+
+* Investigate each recent (new comment within the past week or so)
+ Cr-Internals-Network issue, driving getting information from reporters as
+ needed, until you can do one of the following:
+
+ * Mark it as *WontFix* (working as intended, obsolete issue) or a
+ duplicate.
+
+ * Mark it as a feature request.
+
+ * Remove the Cr-Internals-Network label, replacing it with at least one
+ more specific network label or non-network label. Promptly adding
+ non-network labels when appropriate is important to get new bugs in front
+ of someone familiar with the relevant code, and to remove them from the
+ next triager's radar. Because of the way the bug report wizard works, a
+ lot of bugs incorrectly end up with the network label.
+
+ * The issue is assigned to an appropriate owner.
+
+ * If there is no more specific label for a bug, it should be investigated
+ until we have a good understanding of the cause of the problem, and some
+ idea how it should be fixed, at which point its status should be set to
+ Available. Future triagers should ignore bugs with this status, unless
+ investigating stale bugs.
+
+* Monitor UMA histograms and gasper alerts.
+
+ * For each Gasper alert that fires, the triager should determine if the
+ alert is real (not due to noise), and file a bug with the appropriate
+ label if so. Note that if no label more specific than
+ Cr-Internals-Network is appropriate, the responsibility remains with the
+ triager to continue investigating the bug, as above.
+
+### Best Effort (As you have time):
+
+* Investigate unowned and owned but forgotten net/ crashers that are still
+ occurring (As indicated by
+ [go/chromecrash](https://goto.google.com/chromecrash)), prioritizing frequent
+ and long standing crashers.
+
+* Investigate old bugs, prioritizing the most recent.
+
+* Close obsolete bugs.
+
+If you've investigated an issue (in code you don't normally work on) to an
+extent that you know how to fix it, and the fix is simple, feel free to take
+ownership of the issue and create a patch while on triage duty, but other tasks
+should take priority.
+
+See [bug-triage-suggested-workflow.md](bug-triage-suggested-workflow.md) for
+suggested workflows.
+
+See [bug-triage-labels.md](bug-triage-labels.md) for labeling tips for network
+and non-network bugs.
« no previous file with comments | « no previous file | net/docs/bug-triage.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698