Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

Ambari use uses ReviewBoard for code reviews.

...

You may need to create a ReviewBoard login. Once you login add yourself to the "Ambari" group.

To submit a patch for review

  1. Click on "New Review Request"
  2. Select "ambari" from the drop-down and upload a diff file
  3. Go to the next page
    1. Add a summary
    2. Names of two reviewers (only one +1 is necessary for commit)
    3. Select ambari as the group
    4. Type in the Ambari JIRA id
    5. Add description of the fix
    6. Add a link to the ReviewBoard URL to the JIRA
    7. [if applicable] Fill the optional fields as needed.
    8. [if applicable] Add the unit test summary to the request
  4. Publish the review
  5. After the review is marked "Ship It" you can optionally attach the final patch to the JIRA.

For reviewers

Reviewers, after you are satisfied with the patch, mark "Ship It" through the ReviewBoard as well as add a "+1" to the JIRA.