Documentation/committer-responsibilities

   1 Expectations for Developers with Open vSwitch Repo Access
   2 =========================================================
   3
   4 Prerequisites
   5 -------------
   6
   7 Be familiar with CodingStyle and CONTRIBUTING.
   8
   9 Review
  10 ------
  11
  12 Code (yours or others') must be reviewed publicly (by you or others)
  13 before you push it to the repository. With one exception (see below),
  14 every change needs at least one review.
  15
  16 If one or more people know an area of code particularly well, code
  17 that affects that area should ordinarily get a review from one of
  18 them.
  19
  20 The riskier, more subtle, or more complicated the change, the more
  21 careful the review required. When a change needs careful review, use
  22 good judgment regarding the quality of reviews. If a change adds 1000
  23 lines of new code, and a review posted 5 minutes later says just
  24 "Looks good," then this is probably not a quality review.
  25
  26 (The size of a change is correlated with the amount of care needed in
  27 review, but it is not strictly tied to it. A search and replace
  28 across many files may not need much review, but one-line optimization
  29 changes can have widespread implications.)
  30
  31 Your own small changes to fix a recently broken build ("make") or
  32 tests ("make check"), that you believe to be visible to a large number
  33 of developers, may be checked in without review. If you are not sure,
  34 ask for review. If you do push a build fix without review, send the
  35 patch to ovs-dev afterward as usual, indicating in the email that you
  36 have already pushed it.
  37
  38 Regularly review submitted code in areas where you have expertise.
  39 Consider reviewing other code as well.
  40
  41 Git conventions
  42 ---------------
  43
  44 Do not push merge commits to the Git repository without prior
  45 discussion on ovs-dev.
  46
  47 If you apply a change (yours or another's) then it is your
  48 responsibility to handle any resulting problems, especially broken
  49 builds and other regressions. If it is someone else's change, then
  50 you can ask the original submitter to address it. Regardless, you
  51 need to ensure that the problem is fixed in a timely way. The
  52 definition of "timely" depends on the severity of the problem.
  53
  54 If a bug is present on master and other branches, fix it on master
  55 first, then backport the fix to other branches. Straightforward
  56 backports do not require additional review (beyond that for the fix on
  57 master).
  58
  59 Feature development should be done only on master. Occasionally it
  60 makes sense to add a feature to the most recent release branch, before
  61 the first actual release of that branch. These should be handled in
  62 the same way as bug fixes, that is, first implemented on master and
  63 then backported.
  64
  65 Keep the authorship of a commit clear by maintaining a correct list of
  66 "Signed-off-by:"s. If a confusing situation comes up, as it
  67 occasionally does, bring it up on the mailing list. If you explain
  68 the use of Signed-off-by: to a new developer, explain not just how but
  69 why, since the intended meaning of Signed-off-by: is more important
  70 than the syntax. As part of your explanation, quote or provide a URL
  71 to the Developer's Certificate of Origin in CONTRIBUTING.
  72
  73 Use Reported-by: and Tested-by: tags in commit messages to indicate
  74 the source of a bug report.
  75
  76 Keep the AUTHORS file up to date.