On notifications:  Gerrit has built-in support to send email notifications on 
code changes.  There are two methods:


1.       The contributor can explicitly add a list of reviewers to the proposed 
change, which will trigger a notification to those individuals.  This should 
ideally be done after all expected amendments to the change are completed, and 
after Jenkins has verified the change.

2.       Anyone can subscribe to be notified of changes in any particular repo, 
via the "Watched Projects" section in their profile settings.

I would recommend that all committers subscribe to watch changes to the 
projects for which they are committers for.

Thanks,
Gary

From: onap-tsc-boun...@lists.onap.org [mailto:onap-tsc-boun...@lists.onap.org] 
On Behalf Of eric.deb...@orange.com
Sent: Thursday, August 10, 2017 6:30 AM
To: onap-...@lists.onap.org
Cc: RICHOMME Morgan IMT/OLN <morgan.richo...@orange.com>
Subject: [onap-tsc] Code Review: how to avoid long delay to review

Hello

There was a code review pending since 5 days due to a lack of reviewers. I 
think we should adopt some best practices to avoid such delay.
A committer should inform the PTL and a list of reviewers by mail (typically 
the project committers and other people involved in the project)


OPNFV is listing all the committers (they can provide a +2 code review) in a 
dedicated file INFO for each repo:

https://git.opnfv.org/functest/tree/INFO

https://git.opnfv.org/promise/tree/INFO



OPNFV also defined some rules for committers promotions and removal that should 
inspire us.

https://wiki.opnfv.org/display/DEV/Committer+Promotions

https://wiki.opnfv.org/display/DEV/Committer+Removal

Thanks @Morgan (OPNFV Ambassador & TSC Member) for the links

Regards

Eric

_________________________________________________________________________________________________________________________



Ce message et ses pieces jointes peuvent contenir des informations 
confidentielles ou privilegiees et ne doivent donc

pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce 
message par erreur, veuillez le signaler

a l'expediteur et le detruire ainsi que les pieces jointes. Les messages 
electroniques etant susceptibles d'alteration,

Orange decline toute responsabilite si ce message a ete altere, deforme ou 
falsifie. Merci.



This message and its attachments may contain confidential or privileged 
information that may be protected by law;

they should not be distributed, used or copied without authorisation.

If you have received this email in error, please notify the sender and delete 
this message and its attachments.

As emails may be altered, Orange is not liable for messages that have been 
modified, changed or falsified.

Thank you.
_______________________________________________
onap-discuss mailing list
onap-discuss@lists.onap.org
https://lists.onap.org/mailman/listinfo/onap-discuss

Reply via email to