Labels are carefully picked to optimize for:
A pull request should only be attributed labels documented in this section: other labels that may exist on the repository should apply to issues.
dco/no
: automatically set by a bot when one of the commits lacks proper signaturestatus/0-triage
status/1-design-review
status/2-code-review
status/3-docs-review
status/4-ready-to-merge
Special status labels:
status/failing-ci
: indicates that the PR in its current state fails the test suitestatus/needs-attention
: calls for a collective discussion during a review sessionThose labels are used to raise awareness of a particular specialty group, either because we need help in reviewing the PR, or because of the potential impact of the PR on their work:
group/distribution
group/networking
group/security
group/windows
impact/api
impact/changelog
impact/cli
impact/deprecation
impact/distribution
impact/dockerfile
An opened pull request can be in 1 of 5 distinct states, for each of which there is a corresponding label that needs to be applied.
status/0-triage
Maintainers are expected to triage new incoming pull requests by removing the status/0-triage
label and adding the correct labels (e.g. status/1-design-review
) before any other interaction with the PR. The starting label may potentially skip some steps depending on the kind of pull request: use your best judgement.
Maintainers should perform an initial, high-level, overview of the pull request before moving it to the next appropriate stage:
Possible transitions from this state:
status/1-design-review
: general casestatus/2-code-review
: e.g. trivial bugfixstatus/3-docs-review
: non-proposal documentation-only changestatus/1-design-review
Maintainers are expected to comment on the design of the pull request. Review of documentation is expected only in the context of design validation, not for stylistic changes.
Ideally, documentation should reflect the expected behavior of the code. No code review should take place in this step.
There are no strict rules on the way a design is validated: we usually aim for a consensus, although a single maintainer approval is often sufficient for obviously reasonable changes. In general, strong disagreement expressed by any of the maintainers should not be taken lightly.
Once design is approved, a maintainer should make sure to remove this label and add the next one.
Possible transitions from this state:
status/2-code-review
: general casestatus/3-docs-review
: proposals with only documentation changesstatus/2-code-review
Maintainers are expected to review the code and ensure that it is good quality and in accordance with the documentation in the PR.
New testcases are expected to be added. Ideally, those testcases should fail when the new code is absent, and pass when present. The testcases should strive to test as many variants, code paths, as possible to ensure maximum coverage.
Changes to code must be reviewed and approved (LGTM'd) by a minimum of two code maintainers. When the author of a PR is a maintainer, he still needs the approval of two other maintainers.
Once code is approved according to the rules of the subsystem, a maintainer should make sure to remove this label and add the next one. If documentation is absent but expected, maintainers should ask for documentation and move to status status/3-docs-review
for docs maintainer to follow.
Possible transitions from this state:
status/1-design-review
: new design concerns are raisedstatus/3-docs-review
: general casestatus/4-ready-to-merge
: change not impacting documentationstatus/3-docs-review
Maintainers are expected to review the documentation in its bigger context, ensuring consistency, completeness, validity, and breadth of coverage across all existing and new documentation.
They should ask for any editorial change that makes the documentation more consistent and easier to understand.
Changes and additions to docs must be reviewed and approved (LGTM'd) by a minimum of two docs sub-project maintainers. If the docs change originates with a docs maintainer, only one additional LGTM is required (since we assume a docs maintainer approves of their own PR).
Once documentation is approved (see below), a maintainer should make sure to remove this label and add the next one.
Possible transitions from this state:
status/1-design-review
: new design concerns are raisedstatus/2-code-review
: requires more code changesstatus/4-ready-to-merge
: general casestatus/4-ready-to-merge
Maintainers are expected to merge this pull request as soon as possible. They can ask for a rebase or carry the pull request themselves.
Possible transitions from this state:
After merging a pull request, the maintainer should consider applying one or multiple impact labels to ease future classification:
impact/api
signifies the patch impacted the remote APIimpact/changelog
signifies the change is significant enough to make it in the changelogimpact/cli
signifies the patch impacted a CLI commandimpact/dockerfile
signifies the patch impacted the Dockerfile syntaximpact/deprecation
signifies the patch participates in deprecating an existing featureIf a pull request is closed it is expected that sufficient justification will be provided. In particular, if there are alternative ways of achieving the same net result then those needs to be spelled out. If the pull request is trying to solve a use case that is not one that we (as a community) want to support then a justification for why should be provided.
The number of maintainers it takes to decide and close a PR is deliberately left unspecified. We assume that the group of maintainers is bound by mutual trust and respect, and that opposition from any single maintainer should be taken into consideration. Similarly, we expect maintainers to justify their reasoning and to accept debating.
Despite the previously described reviewing process, some PR might not show any progress for various reasons:
All these will eventually lead to stalled PR, where no apparent progress is made across several weeks, or even months.
Maintainers should use their best judgement and apply the status/needs-attention
label. It must be used sparingly, as each PR with such label will be discussed by a group of maintainers during a review session. The goal of that session is to agree on one of the following outcomes for the PR: