github.com/chenchun/docker@v1.3.2-0.20150629222414-20467faf132b/MAINTAINERS (about) 1 # Docker maintainers file 2 # 3 # This file describes who runs the Docker project and how. 4 # This is a living document - if you see something out of date or missing, 5 # speak up! 6 # 7 # It is structured to be consumable by both humans and programs. 8 # To extract its contents programmatically, use any TOML-compliant 9 # parser. 10 11 [Rules] 12 13 [Rules.maintainers] 14 15 title = "What is a maintainer?" 16 17 text = """ 18 There are different types of maintainers, with different responsibilities, but 19 all maintainers have 3 things in common: 20 21 1) They share responsibility in the project's success. 22 2) They have made a long-term, recurring time investment to improve the project. 23 3) They spend that time doing whatever needs to be done, not necessarily what 24 is the most interesting or fun. 25 26 Maintainers are often under-appreciated, because their work is harder to appreciate. 27 It's easy to appreciate a really cool and technically advanced feature. It's harder 28 to appreciate the absence of bugs, the slow but steady improvement in stability, 29 or the reliability of a release process. But those things distinguish a good 30 project from a great one. 31 """ 32 33 [Rules.bdfl] 34 35 title = "The Benevolent dictator for life (BDFL)" 36 37 text = """ 38 Docker follows the timeless, highly efficient and totally unfair system 39 known as [Benevolent dictator for 40 life](https://en.wikipedia.org/wiki/Benevolent_Dictator_for_Life), with 41 yours truly, Solomon Hykes, in the role of BDFL. This means that all 42 decisions are made, by default, by Solomon. Since making every decision 43 myself would be highly un-scalable, in practice decisions are spread 44 across multiple maintainers. 45 46 Ideally, the BDFL role is like the Queen of England: awesome crown, but not 47 an actual operational role day-to-day. The real job of a BDFL is to NEVER GO AWAY. 48 Every other rule can change, perhaps drastically so, but the BDFL will always 49 be there, preserving the philosophy and principles of the project, and keeping 50 ultimate authority over its fate. This gives us great flexibility in experimenting 51 with various governance models, knowing that we can always press the "reset" button 52 without fear of fragmentation or deadlock. See the US congress for a counter-example. 53 54 BDFL daily routine: 55 56 * Is the project governance stuck in a deadlock or irreversibly fragmented? 57 * If yes: refactor the project governance 58 * Are there issues or conflicts escalated by core? 59 * If yes: resolve them 60 * Go back to polishing that crown. 61 """ 62 63 [Rules.decisions] 64 65 title = "How are decisions made?" 66 67 text = """ 68 Short answer: EVERYTHING IS A PULL REQUEST. 69 70 Docker is an open-source project with an open design philosophy. This 71 means that the repository is the source of truth for EVERY aspect of the 72 project, including its philosophy, design, road map, and APIs. *If it's 73 part of the project, it's in the repo. If it's in the repo, it's part of 74 the project.* 75 76 As a result, all decisions can be expressed as changes to the 77 repository. An implementation change is a change to the source code. An 78 API change is a change to the API specification. A philosophy change is 79 a change to the philosophy manifesto, and so on. 80 81 All decisions affecting Docker, big and small, follow the same 3 steps: 82 83 * Step 1: Open a pull request. Anyone can do this. 84 85 * Step 2: Discuss the pull request. Anyone can do this. 86 87 * Step 3: Merge or refuse the pull request. Who does this depends on the nature 88 of the pull request and which areas of the project it affects. See *review flow* 89 for details. 90 91 Because Docker is such a large and active project, it's important for everyone to know 92 who is responsible for deciding what. That is determined by a precise set of rules. 93 94 * For every *decision* in the project, the rules should designate, in a deterministic way, 95 who should *decide*. 96 97 * For every *problem* in the project, the rules should designate, in a deterministic way, 98 who should be responsible for *fixing* it. 99 100 * For every *question* in the project, the rules should designate, in a deterministic way, 101 who should be expected to have the *answer*. 102 """ 103 104 [Rules.review] 105 106 title = "Review flow" 107 108 text = """ 109 Pull requests should be processed according to the following flow: 110 111 * For each subsystem affected by the change, the maintainers of the subsystem must approve or refuse it. 112 It is the responsibility of the subsystem maintainers to process patches affecting them in a timely 113 manner. 114 115 * If the change affects areas of the code which are not part of a subsystem, 116 or if subsystem maintainers are unable to reach a timely decision, it must be approved by 117 the core maintainers. 118 119 * If the change affects the UI or public APIs, or if it represents a major change in architecture, 120 the architects must approve or refuse it. 121 122 * If the change affects the operations of the project, it must be approved or rejected by 123 the relevant operators. 124 125 * If the change affects the governance, philosophy, goals or principles of the project, 126 it must be approved by BDFL. 127 128 * A pull request can be in 1 of 5 distinct states, for each of which there is a corresponding label 129 that needs to be applied. `Rules.review.states` contains the list of states with possible targets 130 for each. 131 """ 132 133 # Triage 134 [Rules.review.states.0-needs-triage] 135 136 # Maintainers are expected to triage new incoming pull requests by removing 137 # the `0-triage` label and adding the correct labels (e.g. `1-design-review`) 138 # potentially skipping some steps depending on the kind of pull request. 139 # Use common sense for judging. 140 # 141 # Checking for DCO should be done at this stage. 142 # 143 # If an owner, responsible for closing or merging, can be assigned to the PR, 144 # the better. 145 146 close = "e.g. unresponsive contributor without DCO" 147 3-docs-review = "non-proposal documentation-only change" 148 2-code-review = "e.g. trivial bugfix" 149 1-design-review = "general case" 150 151 # Design review 152 [Rules.review.states.1-needs-design-review] 153 154 # Maintainers are expected to comment on the design of the pull request. 155 # Review of documentation is expected only in the context of design validation, 156 # not for stylistic changes. 157 # 158 # Ideally, documentation should reflect the expected behavior of the code. 159 # No code review should take place in this step. 160 # 161 # Once design is approved, a maintainer should make sure to remove this label 162 # and add the next one. 163 164 close = "design rejected" 165 3-docs-review = "proposals with only documentation changes" 166 2-code-review = "general case" 167 168 # Code review 169 [Rules.review.states.2-needs-code-review] 170 171 # Maintainers are expected to review the code and ensure that it is good 172 # quality and in accordance with the documentation in the PR. 173 # 174 # If documentation is absent but expected, maintainers should ask for documentation. 175 # 176 # All tests should pass. 177 # 178 # Once code is approved according to the rules of the subsystem, a maintainer 179 # should make sure to remove this label and add the next one. 180 181 close = "" 182 1-design-review = "raises design concerns" 183 4-merge = "trivial change not impacting documentation" 184 3-docs-review = "general case" 185 186 # Docs review 187 [Rules.review.states.3-needs-docs-review] 188 189 # Maintainers are expected to review the documentation in its bigger context, 190 # ensuring consistency, completeness, validity, and breadth of coverage across 191 # all extent and new documentation. 192 # 193 # They should ask for any editorial change that makes the documentation more 194 # consistent and easier to understand. 195 # 196 # Changes and additions to docs must be reviewed and approved (LGTM'd) by a minimum of 197 # two docs sub-project maintainers. If the docs change originates with a docs 198 # maintainer, only one additional LGTM is required (since we assume a docs maintainer 199 # approves of their own PR). 200 # 201 # Once documentation is approved (see below), a maintainer should make sure to remove this 202 # label and add the next one. 203 204 close = "" 205 2-code-review = "requires more code changes" 206 1-design-review = "raises design concerns" 207 4-merge = "general case" 208 209 # Merge 210 [Rules.review.states.4-needs-merge] 211 212 # Maintainers are expected to merge this pull request as soon as possible. 213 # They can ask for a rebase, or carry the pull request themselves. 214 # These should be the easy PRs to merge. 215 216 close = "carry PR" 217 merge = "" 218 219 [Rules.DCO] 220 221 title = "Helping contributors with the DCO" 222 223 text = """ 224 The [DCO or `Sign your work`]( 225 https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work) 226 requirement is not intended as a roadblock or speed bump. 227 228 Some Docker contributors are not as familiar with `git`, or have used a web based 229 editor, and thus asking them to `git commit --amend -s` is not the best way forward. 230 231 In this case, maintainers can update the commits based on clause (c) of the DCO. The 232 most trivial way for a contributor to allow the maintainer to do this, is to add 233 a DCO signature in a Pull Requests's comment, or a maintainer can simply note that 234 the change is sufficiently trivial that it does not substantivly change the existing 235 contribution - i.e., a spelling change. 236 237 When you add someone's DCO, please also add your own to keep a log. 238 """ 239 240 [Rules.holiday] 241 242 title = "I'm a maintainer, and I'm going on holiday" 243 244 text = """ 245 Please let your co-maintainers and other contributors know by raising a pull 246 request that comments out your `MAINTAINERS` file entry using a `#`. 247 """ 248 249 [Rules."no direct push"] 250 251 title = "I'm a maintainer. Should I make pull requests too?" 252 253 text = """ 254 Yes. Nobody should ever push to master directly. All changes should be 255 made through a pull request. 256 """ 257 258 [Rules.meta] 259 260 title = "How is this process changed?" 261 262 text = "Just like everything else: by making a pull request :)" 263 264 # Current project organization 265 [Org] 266 267 bdfl = "shykes" 268 269 # The chief architect is responsible for the overall integrity of the technical architecture 270 # across all subsystems, and the consistency of APIs and UI. 271 # 272 # Changes to UI, public APIs and overall architecture (for example a plugin system) must 273 # be approved by the chief architect. 274 "Chief Architect" = "shykes" 275 276 # The Chief Operator is responsible for the day-to-day operations of the project including: 277 # - facilitating communications amongst all the contributors; 278 # - tracking release schedules; 279 # - managing the relationship with downstream distributions and upstream dependencies; 280 # - helping new contributors to get involved and become successful contributors and maintainers 281 # 282 # The role is also responsible for managing and measuring the success of the overall project 283 # and ensuring it is governed properly working in concert with the Docker Governance Advisory Board (DGAB). 284 "Chief Operator" = "spf13" 285 286 [Org.Operators] 287 288 # The operators make sure the trains run on time. They are responsible for overall operations 289 # of the project. This includes facilitating communication between all the participants; helping 290 # newcomers get involved and become successful contributors and maintainers; tracking the schedule 291 # of releases; managing the relationship with downstream distributions and upstream dependencies; 292 # define measures of success for the project and measure progress; Devise and implement tools and 293 # processes which make contributors and maintainers happier and more efficient. 294 295 296 [Org.Operators.security] 297 298 people = [ 299 "erw", 300 "diogomonica", 301 "nathanmccauley" 302 ] 303 304 [Org.Operators."monthly meetings"] 305 306 people = [ 307 "sven", 308 "tianon" 309 ] 310 311 [Org.Operators.infrastructure] 312 313 people = [ 314 "jfrazelle", 315 "crosbymichael" 316 ] 317 318 [Org.Operators.community] 319 people = [ 320 "theadactyl" 321 ] 322 323 # The chief maintainer is responsible for all aspects of quality for the project including 324 # code reviews, usability, stability, security, performance, etc. 325 # The most important function of the chief maintainer is to lead by example. On the first 326 # day of a new maintainer, the best advice should be "follow the C.M.'s example and you'll 327 # be fine". 328 "Chief Maintainer" = "crosbymichael" 329 330 # The community manager is responsible for serving the project community, including users, 331 # contributors and partners. This involves: 332 # - facilitating communication between maintainers, contributors and users 333 # - organizing contributor and maintainer events 334 # - helping new contributors get involved 335 # - anything the project community needs to be successful 336 # 337 # The community manager is a point of contact for any contributor who has questions, concerns 338 # or feedback about project operations. 339 "Community Manager" = "theadactyl" 340 341 [Org."Core maintainers"] 342 343 # The Core maintainers are the ghostbusters of the project: when there's a problem others 344 # can't solve, they show up and fix it with bizarre devices and weaponry. 345 # They have final say on technical implementation and coding style. 346 # They are ultimately responsible for quality in all its forms: usability polish, 347 # bugfixes, performance, stability, etc. When ownership can cleanly be passed to 348 # a subsystem, they are responsible for doing so and holding the 349 # subsystem maintainers accountable. If ownership is unclear, they are the de facto owners. 350 351 # For each release (including minor releases), a "release captain" is assigned from the 352 # pool of core maintainers. Rotation is encouraged across all maintainers, to ensure 353 # the release process is clear and up-to-date. 354 # 355 # It is common for core maintainers to "branch out" to join or start a subsystem. 356 357 358 359 people = [ 360 "calavera", 361 "crosbymichael", 362 "erikh", 363 "estesp", 364 "icecrime", 365 "jfrazelle", 366 "lk4d4", 367 "runcom", 368 "tibor", 369 "unclejack", 370 "vbatts", 371 "vieux", 372 "vishh" 373 ] 374 375 376 [Org.Subsystems] 377 378 # As the project grows, it gets separated into well-defined subsystems. Each subsystem 379 # has a dedicated group of maintainers, which are dedicated to that subsytem and responsible 380 # for its quality. 381 # This "cellular division" is the primary mechanism for scaling maintenance of the project as it grows. 382 # 383 # The maintainers of each subsytem are responsible for: 384 # 385 # 1. Exposing a clear road map for improving their subsystem. 386 # 2. Deliver prompt feedback and decisions on pull requests affecting their subsystem. 387 # 3. Be available to anyone with questions, bug reports, criticism etc. 388 # on their component. This includes IRC, GitHub requests and the mailing 389 # list. 390 # 4. Make sure their subsystem respects the philosophy, design and 391 # road map of the project. 392 # 393 # #### How to review patches to your subsystem 394 # 395 # Accepting pull requests: 396 # 397 # - If the pull request appears to be ready to merge, give it a `LGTM`, which 398 # stands for "Looks Good To Me". 399 # - If the pull request has some small problems that need to be changed, make 400 # a comment adressing the issues. 401 # - If the changes needed to a PR are small, you can add a "LGTM once the 402 # following comments are adressed..." this will reduce needless back and 403 # forth. 404 # - If the PR only needs a few changes before being merged, any MAINTAINER can 405 # make a replacement PR that incorporates the existing commits and fixes the 406 # problems before a fast track merge. 407 # 408 # Closing pull requests: 409 # 410 # - If a PR appears to be abandoned, after having attempted to contact the 411 # original contributor, then a replacement PR may be made. Once the 412 # replacement PR is made, any contributor may close the original one. 413 # - If you are not sure if the pull request implements a good feature or you 414 # do not understand the purpose of the PR, ask the contributor to provide 415 # more documentation. If the contributor is not able to adequately explain 416 # the purpose of the PR, the PR may be closed by any MAINTAINER. 417 # - If a MAINTAINER feels that the pull request is sufficiently architecturally 418 # flawed, or if the pull request needs significantly more design discussion 419 # before being considered, the MAINTAINER should close the pull request with 420 # a short explanation of what discussion still needs to be had. It is 421 # important not to leave such pull requests open, as this will waste both the 422 # MAINTAINER's time and the contributor's time. It is not good to string a 423 # contributor on for weeks or months, having them make many changes to a PR 424 # that will eventually be rejected. 425 426 [Org.Subsystems.Documentation] 427 428 people = [ 429 "fredlf", 430 "james", 431 "moxiegirl", 432 "thaJeztah", 433 "jamtur01", 434 "spf13", 435 "sven" 436 ] 437 438 [Org.Subsystems.libcontainer] 439 440 people = [ 441 "crosbymichael", 442 "jnagal", 443 "lk4d4", 444 "mpatel", 445 "vmarmol" 446 ] 447 448 [Org.Subsystems.registry] 449 450 people = [ 451 "dmcg", 452 "dmp42", 453 "jlhawn", 454 "samalba", 455 "sday", 456 "vbatts" 457 ] 458 459 [Org.Subsystems."build tools"] 460 461 people = [ 462 "shykes", 463 "tianon" 464 ] 465 466 [Org.Subsystem."remote api"] 467 468 people = [ 469 "vieux" 470 ] 471 472 [Org.Subsystem.swarm] 473 474 people = [ 475 "aluzzardi", 476 "vieux" 477 ] 478 479 [Org.Subsystem.machine] 480 481 people = [ 482 "bfirsh", 483 "ehazlett" 484 ] 485 486 [Org.Subsystem.compose] 487 488 people = [ 489 "aanand" 490 ] 491 492 [Org.Subsystem.builder] 493 494 people = [ 495 "duglin", 496 "erikh", 497 "tibor" 498 ] 499 500 [Org.Curators] 501 502 # The curators help ensure that incoming issues and pull requests are properly triaged and 503 # that our various contribution and reviewing processes are respected. With their knowledge of 504 # the repository activity, they can also guide contributors to relevant material or 505 # discussions. 506 # 507 # They are neither code nor docs reviewers, so they are never expected to merge. They can 508 # however: 509 # - close an issue or pull request when it's an exact duplicate 510 # - close an issue or pull request when it's inappropriate or off-topic 511 512 people = [ 513 "thajeztah" 514 ] 515 516 517 [people] 518 519 # A reference list of all people associated with the project. 520 # All other sections should refer to people by their canonical key 521 # in the people section. 522 523 # ADD YOURSELF HERE IN ALPHABETICAL ORDER 524 525 [people.aanand] 526 Name = "Aanand Prasad" 527 Email = "aanand@docker.com" 528 GitHub = "aanand" 529 530 [people.aluzzardi] 531 Name = "Andrea Luzzardi" 532 Email = "aluzzardi@docker.com" 533 GitHub = "aluzzardi" 534 535 [people.bfirsh] 536 Name = "Ben Firshman" 537 Email = "ben@firshman.co.uk" 538 GitHub = "bfirsh" 539 540 [people.calavera] 541 Name = "David Calavera" 542 Email = "david.calavera@gmail.com" 543 GitHub = "calavera" 544 545 [people.cpuguy83] 546 Name = "Brian Goff" 547 Email = "cpuguy83@gmail.com" 548 Github = "cpuguy83" 549 550 [people.crosbymichael] 551 Name = "Michael Crosby" 552 Email = "crosbymichael@gmail.com" 553 GitHub = "crosbymichael" 554 555 [people.diogomonica] 556 Name = "Diogo Monica" 557 Email = "diogo@docker.com" 558 GitHub = "diogomonica" 559 560 [people.duglin] 561 Name = "Doug Davis" 562 Email = "dug@us.ibm.com" 563 GitHub = "duglin" 564 565 [people.dmcg] 566 Name = "Derek McGowan" 567 Email = "derek@docker.com" 568 Github = "dmcgowan" 569 570 [people.dmp42] 571 Name = "Olivier Gambier" 572 Email = "olivier@docker.com" 573 Github = "dmp42" 574 575 [people.ehazlett] 576 Name = "Evan Hazlett" 577 Email = "ejhazlett@gmail.com" 578 GitHub = "ehazlett" 579 580 [people.erikh] 581 Name = "Erik Hollensbe" 582 Email = "erik@docker.com" 583 GitHub = "erikh" 584 585 [people.erw] 586 Name = "Eric Windisch" 587 Email = "eric@windisch.us" 588 GitHub = "ewindisch" 589 590 [people.estesp] 591 Name = "Phil Estes" 592 Email = "estesp@linux.vnet.ibm.com" 593 GitHub = "estesp" 594 595 [people.fredlf] 596 Name = "Fred Lifton" 597 Email = "fred.lifton@docker.com" 598 GitHub = "fredlf" 599 600 [people.icecrime] 601 Name = "Arnaud Porterie" 602 Email = "arnaud@docker.com" 603 GitHub = "icecrime" 604 605 [people.jfrazelle] 606 Name = "Jessie Frazelle" 607 Email = "j@docker.com" 608 GitHub = "jfrazelle" 609 610 [people.jlhawn] 611 Name = "Josh Hawn" 612 Email = "josh.hawn@docker.com" 613 Github = "jlhawn" 614 615 [people.lk4d4] 616 Name = "Alexander Morozov" 617 Email = "lk4d4@docker.com" 618 GitHub = "lk4d4" 619 620 [people.moxiegirl] 621 Name = "Mary Anthony" 622 Email = "mary.anthony@docker.com" 623 GitHub = "moxiegirl" 624 625 [people.nathanmccauley] 626 Name = "Nathan McCauley" 627 Email = "nathan.mccauley@docker.com" 628 GitHub = "nathanmccauley" 629 630 [people.runcom] 631 Name = "Antonio Murdaca" 632 Email = "me@runcom.ninja" 633 GitHub = "runcom" 634 635 [people.sday] 636 Name = "Stephen Day" 637 Email = "stephen.day@docker.com" 638 Github = "stevvooe" 639 640 [people.shykes] 641 Name = "Solomon Hykes" 642 Email = "solomon@docker.com" 643 GitHub = "shykes" 644 645 [people.spf13] 646 Name = "Steve Francia" 647 Email = "steve.francia@gmail.com" 648 GitHub = "spf13" 649 650 [people.sven] 651 Name = "Sven Dowideit" 652 Email = "SvenDowideit@home.org.au" 653 GitHub = "SvenDowideit" 654 655 [people.thajeztah] 656 Name = "Sebastiaan van Stijn" 657 Email = "github@gone.nl" 658 GitHub = "thaJeztah" 659 660 [people.theadactyl] 661 Name = "Thea Lamkin" 662 Email = "thea@docker.com" 663 GitHub = "theadactyl" 664 665 [people.tianon] 666 Name = "Tianon Gravi" 667 Email = "admwiggin@gmail.com" 668 GitHub = "tianon" 669 670 [people.tibor] 671 Name = "Tibor Vass" 672 Email = "tibor@docker.com" 673 GitHub = "tiborvass" 674 675 [people.vbatts] 676 Name = "Vincent Batts" 677 Email = "vbatts@redhat.com" 678 GitHub = "vbatts" 679 680 [people.vieux] 681 Name = "Victor Vieux" 682 Email = "vieux@docker.com" 683 GitHub = "vieux" 684 685 [people.vmarmol] 686 Name = "Victor Marmol" 687 Email = "vmarmol@google.com" 688 GitHub = "vmarmol" 689 690 [people.jnagal] 691 Name = "Rohit Jnagal" 692 Email = "jnagal@google.com" 693 GitHub = "rjnagal" 694 695 [people.mpatel] 696 Name = "Mrunal Patel" 697 Email = "mpatel@redhat.com" 698 GitHub = "mrunalp" 699 700 [people.unclejack] 701 Name = "Cristian Staretu" 702 Email = "cristian.staretu@gmail.com" 703 GitHub = "unclejack" 704 705 [people.vishh] 706 Name = "Vishnu Kannan" 707 Email = "vishnuk@google.com" 708 GitHub = "vishh"