Skip to content

Latest commit

 

History

History
153 lines (108 loc) · 5.9 KB

MAINTAINER.md

File metadata and controls

153 lines (108 loc) · 5.9 KB

Maintainers

The team

PR review process:

  • The status needs-design-review is only used in complex/heavy/tricky PRs.
  • From 1 to 2: 1 design LGTM in comment, by a senior maintainer, if needed.
  • From 2 to 3: 3 LGTM by any maintainer.
  • If needed, a specific maintainer familiar with a particular domain can be requested for the review.

We use PRM to manage locally pull requests.

Bots

Update and Merge Pull Request

The maintainer giving the final LGTM must add the status/3-needs-merge label to trigger the merge bot.

By default, a squash-rebase merge will be carried out. If you want to preserve commits you must add bot/merge-method-rebase before status/3-needs-merge.

The status status/4-merge-in-progress is only for the bot.

If the bot is not able to perform the merge, the label bot/need-human-merge is added.
In this case you must solve conflicts/CI/... and after you only need to remove bot/need-human-merge.

A maintainer can add bot/no-merge on a PR if he want (temporarily) prevent a merge by the bot.

bot/light-review can be used to decrease required LGTM from 3 to 1 when:

  • vendor updates from previously reviewed PRs
  • merges branches into master
  • prepare release
  • closes stale issues [cron]
    • use some criterion as number of days between creation, last update, labels, ...

Manage GitHub labels

  • Add labels on new PR [GitHub WebHook]
  • Add milestone to a new PR based on a branch version (1.4, 1.3, ...) [GitHub WebHook]
  • Add and remove contributor/waiting-for-corrections label when a review request changes [GitHub WebHook]
  • Weekly report of PR status on Slack (CaptainPR) [cron]

Labels

If we open/look an issue/PR, we must add a kind/*, an area/* and a status/*.

Contributor

  • contributor/need-more-information: we need more information from the contributor in order to analyze a problem.
  • contributor/waiting-for-feedback: we need the contributor to give us feedback.
  • contributor/waiting-for-corrections: we need the contributor to take actions in order to move forward with a PR. (only for PR) [bot, humans]
  • contributor/needs-resolve-conflicts: use it only when there is some conflicts (and an automatic rebase is not possible). (only for PR) [bot, humans]

Kind

  • kind/enhancement: a new or improved feature.

  • kind/question: It's a question. (only for issue)

  • kind/proposal: proposal PR/issues need a public debate.

    • Proposal issues are design proposal that need to be refined with multiple contributors.
    • Proposal PRs are technical prototypes that need to be refined with multiple contributors.
  • kind/bug/possible: if we need to analyze to understand if it's a bug or not. (only for issues)

  • kind/bug/confirmed: we are sure, it's a bug. (only for issues)

  • kind/bug/fix: it's a bug fix. (only for PR)

Resolution

  • resolution/duplicate: it's a duplicate issue/PR.
  • resolution/declined: Rule #1 of open-source: no is temporary, yes is forever.
  • WIP: Work In Progress. (only for PR)

Platform

  • platform/windows: Windows related.

Area

  • area/acme: ACME related.
  • area/api: Traefik API related.
  • area/authentication: Authentication related.
  • area/cluster: Traefik clustering related.
  • area/documentation: regards improving/adding documentation.
  • area/infrastructure: related to CI or Traefik building scripts.
  • area/healthcheck: Health-check related.
  • area/logs: Traefik logs related.
  • area/middleware: Middleware related.
  • area/middleware/metrics: Metrics related. (Prometheus, StatsD, ...)
  • area/oxy: Oxy related.
  • area/provider: related to all providers.
  • area/provider/boltdb: Boltd DB related.
  • area/provider/consul: Consul related.
  • area/provider/docker: Docker and Swarm related.
  • area/provider/ecs: ECS related.
  • area/provider/etcd: Etcd related.
  • area/provider/eureka: Eureka related.
  • area/provider/file: file provider related.
  • area/provider/k8s: Kubernetes related.
  • area/provider/marathon: Marathon related.
  • area/provider/mesos: Mesos related.
  • area/provider/rancher: Rancher related.
  • area/provider/zk: Zoo Keeper related.
  • area/sticky-session: Sticky session related.
  • area/tls: TLS related.
  • area/websocket: WebSocket related.
  • area/webui: Web UI related.

Priority

  • priority/P0: needs hot fix. (only for issue)
  • priority/P1: need to be fixed in next release. (only for issue)
  • priority/P2: need to be fixed in the future. (only for issue)
  • priority/P3: maybe. (only for issue)

PR size

  • size/S: small PR. (only for PR) [bot only]
  • size/M: medium PR. (only for PR) [bot only]
  • size/L: Large PR. (only for PR) [bot only]

Status - Workflow

The status/* labels represent the desired state in the workflow.

  • status/0-needs-triage: all new issue or PR have this status. [bot only]
  • status/1-needs-design-review: need a design review. (only for PR)
  • status/2-needs-review: need a code/documentation review. (only for PR)
  • status/3-needs-merge: ready to merge. (only for PR)
  • status/4-merge-in-progress: merge in progress. [bot only]