Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Add reference to Patch Inspection guidelines

...

  1. A JIRA ticket has been opened to track the issue
  2. Test and commit your patch locally using acceptance-small.sh
  3. The patch follows the Requirements for patch submission:
    1. The Commit Comments are well formatted and useful
    2. Verify patch follows Lustre Coding Guidelines (at least "git show | build/checkpatch.pl -" passes)
    3. A regression test has been created that fails without the patch and passes with the patch
    4. The patch has the appropriate signed off by line
  4. The patch has been uploaded to Gerrit
  5. Request at least two inspectors Patch Inspection approvals (preferably ones with experience in this area of code) on the Gerrit change
  6. Record the Gerrit change URL in the Jira ticket
  7. Additional test results appropriate for the patch being landed (interoperability, performance, etc) should be attached to the Jira ticket.
  8. Once Maloo has reported the tests all pass, add the branch gatekeeper Gerrit Gatekeeper as inspector on Gerrit the patch to notify him/her that the patch is ready to be merged
  9. The branch gatekeeper will review the patch, confirm the test results, and submit it when everything goes well
  10. If the submission failed due to patch conflict(s), the gatekeeper will ask you to rebase your patch with the target branch and repeat above steps.
  11. If your patch is needed for other branch(es), please repeat above steps against corresponding branch(es).

...