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

Issue 2959883002: Add "Known unknowns" assessment criterium. (Closed)

Created:
3 years, 5 months ago by aelias_OOO_until_Jul13
Modified:
3 years, 5 months ago
Reviewers:
Dirk Pranke, amineer
CC:
chromium-reviews, Dirk Pranke
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add "Known unknowns" assessment criterium. Severity and prevalence are often very hard to assess in practice -- many regressions follow an "iceberg" pattern where the initial minor bug is just a hint of the true scope of the problem. In such cases, coming up with an objective assessment of severity and prevalence is very difficult and it may be challenging to make the case for a blocking label purely on the basis of observations. However, developers often have a good intuition whether a particular regression falls in one of the categories of bugs prone to "icebergs", based on their past experience with the subsystem, and it should be encouraged to mark a bug ReleaseBlock when they harbor such a fear. Also add a paragraph with some extra nuances about the role that regressions play in the assessment criteria (in particular, nonregressions have much fewer unknowns). BUG=

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -0 lines) Patch
M docs/process/release_blockers.md View 2 chunks +23 lines, -0 lines 3 comments Download

Messages

Total messages: 7 (2 generated)
aelias_OOO_until_Jul13
Hi amineer@, this is what I've been able to come up with for expressing my ...
3 years, 5 months ago (2017-06-26 23:58:32 UTC) #2
Dirk Pranke
I'll leave the bulk of this review to amineer, but I'm not wild about the ...
3 years, 5 months ago (2017-06-30 01:20:49 UTC) #4
amineer
On 2017/06/30 01:20:49, Dirk Pranke wrote: > I'll leave the bulk of this review to ...
3 years, 5 months ago (2017-07-06 21:57:01 UTC) #5
amineer
https://codereview.chromium.org/2959883002/diff/1/docs/process/release_blockers.md File docs/process/release_blockers.md (right): https://codereview.chromium.org/2959883002/diff/1/docs/process/release_blockers.md#newcode90 docs/process/release_blockers.md:90: ### Known unknowns "Addressing Uncertainty" https://codereview.chromium.org/2959883002/diff/1/docs/process/release_blockers.md#newcode94 docs/process/release_blockers.md:94: * **High** ...
3 years, 5 months ago (2017-07-06 21:57:16 UTC) #6
aelias_OOO_until_Jul13
3 years, 5 months ago (2017-07-20 21:28:17 UTC) #7
Message was sent while issue was closed.
I closed this and reuploaded it as
https://chromium-review.googlesource.com/c/580293/

Powered by Google App Engine
This is Rietveld 408576698