2014-03-02 19:54:43 +04:00
|
|
|
# The Docker Maintainer manual
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
## Introduction
|
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
Dear maintainer. Thank you for investing the time and energy to help
|
|
|
|
make Docker as useful as possible. Maintaining a project is difficult,
|
|
|
|
sometimes unrewarding work. Sure, you will get to contribute cool
|
|
|
|
features to the project. But most of your time will be spent reviewing,
|
|
|
|
cleaning up, documenting, answering questions, justifying design
|
|
|
|
decisions - while everyone has all the fun! But remember - the quality
|
|
|
|
of the maintainers work is what distinguishes the good projects from the
|
|
|
|
great. So please be proud of your work, even the unglamourous parts,
|
|
|
|
and encourage a culture of appreciation and respect for *every* aspect
|
|
|
|
of improving the project - not just the hot new features.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
This document is a manual for maintainers old and new. It explains what
|
|
|
|
is expected of maintainers, how they should work, and what tools are
|
|
|
|
available to them.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
This is a living document - if you see something out of date or missing,
|
|
|
|
speak up!
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
## What are a maintainer's responsibility?
|
|
|
|
|
|
|
|
It is every maintainer's responsibility to:
|
|
|
|
|
|
|
|
* 1) Expose a clear roadmap for improving their component.
|
|
|
|
* 2) Deliver prompt feedback and decisions on pull requests.
|
2014-03-02 19:54:43 +04:00
|
|
|
* 3) Be available to anyone with questions, bug reports, criticism etc.
|
|
|
|
on their component. This includes IRC, GitHub requests and the mailing
|
|
|
|
list.
|
|
|
|
* 4) Make sure their component respects the philosophy, design and
|
|
|
|
roadmap of the project.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
## How are decisions made?
|
|
|
|
|
|
|
|
Short answer: with pull requests to the docker repository.
|
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
Docker is an open-source project with an open design philosophy. This
|
|
|
|
means that the repository is the source of truth for EVERY aspect of the
|
|
|
|
project, including its philosophy, design, roadmap and APIs. *If it's
|
|
|
|
part of the project, it's in the repo. It's in the repo, it's part of
|
|
|
|
the project.*
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
As a result, all decisions can be expressed as changes to the
|
|
|
|
repository. An implementation change is a change to the source code. An
|
|
|
|
API change is a change to the API specification. A philosophy change is
|
|
|
|
a change to the philosophy manifesto. And so on.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
All decisions affecting docker, big and small, follow the same 3 steps:
|
|
|
|
|
|
|
|
* Step 1: Open a pull request. Anyone can do this.
|
|
|
|
|
|
|
|
* Step 2: Discuss the pull request. Anyone can do this.
|
|
|
|
|
|
|
|
* Step 3: Accept or refuse a pull request. The relevant maintainer does this (see below "Who decides what?")
|
|
|
|
|
|
|
|
|
|
|
|
## Who decides what?
|
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
So all decisions are pull requests, and the relevant maintainer makes
|
|
|
|
the decision by accepting or refusing the pull request. But how do we
|
|
|
|
identify the relevant maintainer for a given pull request?
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
Docker follows the timeless, highly efficient and totally unfair system
|
|
|
|
known as [Benevolent dictator for
|
|
|
|
life](http://en.wikipedia.org/wiki/Benevolent_Dictator_for_Life), with
|
|
|
|
yours truly, Solomon Hykes, in the role of BDFL. This means that all
|
|
|
|
decisions are made by default by Solomon. Since making every decision
|
|
|
|
myself would be highly un-scalable, in practice decisions are spread
|
|
|
|
across multiple maintainers.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
The relevant maintainer for a pull request is assigned in 3 steps:
|
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
* Step 1: Determine the subdirectory affected by the pull request. This
|
|
|
|
might be `src/registry`, `docs/source/api`, or any other part of the repo.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
* Step 2: Find the `MAINTAINERS` file which affects this directory. If the
|
|
|
|
directory itself does not have a `MAINTAINERS` file, work your way up
|
|
|
|
the repo hierarchy until you find one.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
* Step 3: The first maintainer listed is the primary maintainer. The
|
|
|
|
pull request is assigned to him. He may assign it to other listed
|
|
|
|
maintainers, at his discretion.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
|
|
|
|
### I'm a maintainer, should I make pull requests too?
|
|
|
|
|
2014-03-02 19:54:43 +04:00
|
|
|
Yes. Nobody should ever push to master directly. All changes should be
|
|
|
|
made through a pull request.
|
2013-09-19 04:26:36 +04:00
|
|
|
|
|
|
|
### Who assigns maintainers?
|
|
|
|
|
|
|
|
Solomon.
|
|
|
|
|
|
|
|
### How is this process changed?
|
|
|
|
|
|
|
|
Just like everything else: by making a pull request :)
|