Skip to content

Latest commit

 

History

History
229 lines (197 loc) · 8.16 KB

CLA.md

File metadata and controls

229 lines (197 loc) · 8.16 KB

Sign the CLA

This page is the step-by-step guide to signing the Consensys AG Individual Contributor License Agreement.

  1. First and foremost, read [the current version of the CLA]. It is written to be as close to plain English as possible.

  2. Make an account on GitHub if you don't already have one.

  3. After creating your first pull request, you will see a merge pre-requisite requiring to you read and sign the CLA.

If you have any questions, you can reach us on Discord.

Current version of the CLA

Bug Triage Process

In the Quorum Besu Plugins repository, we have a well-defined policy regarding bug categorization.

Our policy is inspired by the Besu Defect Prioritization Policy & Bug Triage Process :

Deciding Priority

When deciding how to respond to a bug, consider the two variables of probability and severity, and apply them to a risk matrix to determine the priority and corresponding action.

Probability

How often actual customers are likely to encounter the issue during their use of the product?

A feature rarely used, with a high likelihood of failure, may have a lower overall probability than a popular feature with a low likelihood of failure (due to a high number of customers affected).

Frequent

Will occur several times in the lifetime of the product.

Probable

Likely to occur often in the lifetime of the product.

Occasional

Likely to occur some time in the lifetime of the product.

Remote

Unlikely but possible to occur in the lifetime of the product.

Improbable

So unlikely, it can be assumed occurrence may not be experienced.

Severity

How bad is the problem when it is encountered?

An issue that causes data loss/corruption is automatically classed as Catastrophic.

Catastrophic

Detrimental risk for the product as a whole.

Critical

Jeopardises feature(s) of the product, not ruining the product entirely.

Moderate

Jeopardises aspects of a feature, not ruining the feature entirely.

Marginal

Causes an inconvenience when using a feature of the product.

Insignificant

No significant threat posed and can be left unmediated. 

Risk Matrix

Catastrophic Critical Moderate Marginal Insignificant
Frequent Very High High High Medium Very Low
Probable High High Medium Medium Very Low
Occasional High Medium Medium Low Very Low
Remote Medium Medium Low Low Very Low
Improbable Medium Low Low Very Low Very Low

Examples

In a nutshell, our bugs are categorized into 5 priorities:

Priority Examples
Very High (P1)
  • Affects consensus
  • Breaks sync
  • Security issue
High (P2)
  • Degrades validator rewards
  • Incurs penalties
  • Significant performance regression
  • Unexpected behaviour of core features
Medium (P3)
  • Feature not working with a specific client library
Low (P4)
  • Node doesn't start up when the configuration file has unexpected "end-of-line" character
Very Low (P5)
  • Typo on a CLI command description
  • An inconsequential issue gets logged on ERROR level

Why is it important?

All software has bugs. And sometimes we don't find them. This is something that we need to live with. However, when we do find bugs we need to do our best to identify the potential impact of that bug in our system.

A proper triaged bug helps up to prioritize and act on bugs in a timely manner. Also, by analysing the severity and impact of our bugs, we can create/update processes to ensure we are better prepared for them in the future.

How to triage a bug?

All of our bugs are reported on GitHub issues. They are identified by the BUG label. We can make use of GitHub labels to categorize our bugs. We have 5 labels, P1, P2, ..., P5 to categorize a bug according to our policy shown above.

Step-by-step process (for maintainers)
  1. Identify that a bug doesn't have a priority (no Px label).
  2. Use the bug categorization policy to determine the final priority of the bug.
  3. Based on your assessment, add the corresponding label to the bug.
  4. (optional) Add a comment to the issue explaining the rationale (justifying the severity and probability analysis).

FAQ

What if I don't agree with the priority of a bug?

We try to be deterministic on our assessments, and most of the time it will involve some discussion. However, if you feel like the assessment on a bug priority is wrong, you can always add a comment on the issue and start a discussion.

Can I change the priority of a bug?

You can. However, if another person was responsible for the initial assessment it is expected that you will contact that person to share your thoughts before changing it.

It is a combination of being polite and reasonable. :)

What if I don't have enough knowledge of a specific area to prioritize a bug?

That's ok! Deeper understanding of the impact of a bug will come from the people that work closer to the affected area of the code. Leave it to them!