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

Unified Diff: docs/process/release_blockers.md

Issue 2959883002: Add "Known unknowns" assessment criterium. (Closed)
Patch Set: Created 3 years, 6 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: docs/process/release_blockers.md
diff --git a/docs/process/release_blockers.md b/docs/process/release_blockers.md
index 937925fba0ee9221e39f9cebfdd4eca4ce8aa840..899c6ee8f81d18e294bbd061132c3132c8052e9a 100644
--- a/docs/process/release_blockers.md
+++ b/docs/process/release_blockers.md
@@ -87,6 +87,23 @@ would still be considered to affect all users, and a bug affecting all
Enterprise Windows users (but not all consumer Windows users) could also be
considered to affect all users.
+### Known unknowns
amineer 2017/07/06 21:57:16 "Addressing Uncertainty"
+
+Known unknowns are defined as the assessable likelihood that the actual true
+severity and prevalence may be higher than initially observed. Examples:
+* **High** - A recently introduced race condition causing tabs to never load
amineer 2017/07/06 21:57:16 I don't know that adding dimensions here (e.g. bre
+ (without crashing, and thus without generating a report for our metrics).
+ There are few user complaints so far, but there's reason to believe it might
+ show up more frequently with Arabic language builds, where we tend to
+ receive feedback more slowly due to the language barrier.
+* **Medium** - A recently introduced CSS bug causing slightly misaligned
+ layout. But there's a slim possibility that on a different site we haven't
+ tested yet, the content might be misaligned all the way off the screen.
+* **None** - A longstanding null pointer crash with a simple local repro and
+ trustworthy prevalence metrics from Stable channel users.
+* **N/A** - The person adding the releaseblock label doesn't have enough
+ information to assess this.
+
### Customization
The definitions provided above are examples; teams are encouraged to customize
@@ -141,3 +158,9 @@ need to address, and we should focus on the most important. To ensure we
maintain a high bar for product quality, we should track the number of
introduced versus escaped regressions, and follow up if the number starts to
rise.
+
+In practice, it is still expected that the majority of release blockers filed
+will be recent regressions, because on average they have higher severity,
+prevalence and unknowns than longstanding bugs. (A rule of thumb is that the
+longer a bug has been present on stable channel without intense pressure to fix,
+the greater the odds that severity/prevalence/unknowns are actually low.)
amineer 2017/07/06 21:57:16 s/unknowns/uncertainty
« 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