Difference between revisions of "StarlingX/CodeSubmissionGuidelines"
Ghada.khalil (talk | contribs) |
Ghada.khalil (talk | contribs) m |
||
Line 15: | Line 15: | ||
Closes-Bug: $bug_id | Closes-Bug: $bug_id | ||
Example: https://review.openstack.org/596305 | Example: https://review.openstack.org/596305 | ||
− | * Pre-Submission Testing | + | * Pre-Submission / Pre-Gerrit Testing |
** At a minimum, make sure the code builds and runs | ** At a minimum, make sure the code builds and runs | ||
** Verify basic functional testing on a builtISO (ensure the new code get executed) | ** Verify basic functional testing on a builtISO (ensure the new code get executed) | ||
** Include automated unit tests when applicable. As we build out the Zuul infrastructure, those tests will run per merge | ** Include automated unit tests when applicable. As we build out the Zuul infrastructure, those tests will run per merge | ||
** Consult with the component core reviewers for any required/recommended testing | ** Consult with the component core reviewers for any required/recommended testing |
Revision as of 00:59, 4 September 2018
StarlingX Code Submission Guidelines
- Use Gerrit for StarlingX code reviews
- Add the core reviewers for the affected sub-project to the review
- The core reviewers are listed on each sub-project wiki pages. The list of sub-projects is available here
- All code changes must be pushed to master first and then cherry-picked to the appropriate release branch as needed
- Exception: Feature branches used during development
- Link your code change to a StoryBoard Story or Launchpad Bug
- For traceability, always link your code change to a story or bug. Gerrit will update the status of the story/bug automatically once the code is merged.
- Linking to StoryBoard Stories: Specify the story and task ID in the commit message as follows:
Story: $story_id Task: $task_id
Example: https://review.openstack.org/#/c/590083/
- Linking to Launchpad Bugs: Specify the Bug ID in the commit message as follows:
Closes-Bug: $bug_id
Example: https://review.openstack.org/596305
- Pre-Submission / Pre-Gerrit Testing
- At a minimum, make sure the code builds and runs
- Verify basic functional testing on a builtISO (ensure the new code get executed)
- Include automated unit tests when applicable. As we build out the Zuul infrastructure, those tests will run per merge
- Consult with the component core reviewers for any required/recommended testing