Difference between revisions of "CSIT/Submitting Patches"

From fd.io
Jump to: navigation, search
(Created page with "__TOC__ == Submitting Patches == The VPP wiki has a nice set of instructions for VPP/Pulling,_Building,_Running,_Hacking_and_Pushing_VPP_Code#Pushing_Code_with_git_review |...")
 
(Replaced content with "__TOC__")
 
(5 intermediate revisions by 2 users not shown)
Line 1: Line 1:
 
__TOC__
 
__TOC__
 
== Submitting Patches ==
 
The VPP wiki has a nice set of instructions for [[VPP/Pulling,_Building,_Running,_Hacking_and_Pushing_VPP_Code#Pushing_Code_with_git_review | Pushing Code with git review]].  The same process is recommended for submitting patches to the CSIT git repository.
 
 
== Commit Message Headline Nomenclature ==
 
When committing code to the CSIT repository and pushing patches to gerrit, please add a prefix to the headline of the commit message that consists of "DO_NOT_MERGE:", "CSIT-<Jira ID>:", or both (with DO_NOT_MERGE before the Jira ticket ID).  Here are some examples:
 
 
CSIT-60: Create l2-vlan-dot1ad libraries for perf testing
 
 
DO_NOT_MERGE: test VXLAN + VLAN on newer version of VPP.
 
 
DO_NOT_MERGE CSIT-156: DPDK support for Nested VM image
 
 
Using this nomenclature makes the [https://gerrit.fd.io/r/#/q/status:open+project:csit CSIT gerrit open patches list] easier to manage for CSIT committers and code reviewers.
 

Latest revision as of 10:50, 23 June 2023