Hi Donald, all As discuss yesterday on Slack, here it is my proposal to manage Pull Request to be discuss during the audio. Regards Olivier --- FRR Pull request management --- - Once a Pull Request is submitted, it is assigned to a code reviewer. The code reviewer has 2 weeks to review the code and commit it into the master. If the assigned code reviewer is too busy to manage the Pull Request he must delegate the review to another person within 3 days. - Code reviewer has not necessary the right to commit directly the code. If he hasn't the right, a maintainer could do that once the review is done and the Pull request accepted. - Code reviewer is responsible to interact with the owner of the Pull Request if modifications are necessary - Code reviewers are designate based on volunteer who explicitly indicate which part of the code he/she is confident and able to review - To start, helping people becoming a code reviewer and facilitate Pull request assignation to a code reviewer, we could fulfil the following table, which is give as example. The idea is at least to split the project into its various routing protocols and some transverses functions. FRR Code Primary Reviewer Backup Reviewer bgpd isisd ldpd lib ospf6d ospfd pimd ripd ripngd vtysh zebra packaging compilation tools Le 10/01/2017 à 14:43, Donald Sharp a écrit :
1) Status
2) Pull Request : https://github.com/freerangerouting/frr/pull/44 -> RD/RT cli discussion -> Functionality how does this properly fit into vnc?
3) Anything else?
donald
_______________________________________________ frr mailing list frr@lists.nox.tf https://lists.nox.tf/listinfo/frr