[LU-6907] Gerrit checkpatch (and derived scripts) is too noisy Created: 27/Jul/15 Updated: 28/Jul/15 Resolved: 27/Jul/15 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.8.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Oleg Drokin | Assignee: | Oleg Drokin |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
Currently every time gerrit checkpatch reviewer makes a comment, it's sent to everybody subscribed to the ticket. But in fact gerrit REST API allows the script to have control over who gets the emails. So I propose to change gerrit checkpatch reviewer to only send email to patch owner on error, and not send any emails to anybody on success. Proposed patch is to follow shortly. |
| Comments |
| Comment by Gerrit Updater [ 27/Jul/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: http://review.whamcloud.com/15729 |
| Comment by Gerrit Updater [ 27/Jul/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/15729/ |
| Comment by Oleg Drokin [ 27/Jul/15 ] |
|
patch landed to master for 2.8.0 |