My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 992: Urgency flag for a submit for immediate attention of reviewers, approvers and submitters
1 person starred this issue and may be notified of changes. Back to list
Status:  Duplicate
Merged:  issue 287
Owner:  ----
Closed:  Sep 2014

Blocked on:
issue 287


Sign in to add a comment
 
Reported by sachini...@gmail.com, Jun 1, 2011
Hi,

Urgency flag = High Priority change
When someone pushes a change on git/Gerrit for review.
Some changes are like build fixes, regression fixes, quick release fixes, showstopper bug fixes which needs quick response.
Normally the change is reviewed by domain experts and approved and then merged by submitter.
Also, reviewers have a list of changes to review in a day apart from their daily work routine.
So, such urgent changes needs an immediate attention of all these experts to get that change merge ASAP so as to unblock and pace up the things.

Introduce a Urgency flag for a change which notifies the reviewers and submitters to take a quick action.

Urgency flag can have similar interface as Starred changes however visible to all.

Optional Note:
The right to set the Urgency flag can be given to the person who has right to give +2 score.

Thanks and Regards,
Sachin Nikam
NVIDIA, Pune.


Jun 1, 2011
#1 sachini...@gmail.com
Read Optional Note as Additional Note.

Jun 2, 2011
Project Member #2 bklarson@gmail.com
I think this would fit in really well as an arbitrary label/tag ( Issue 287 ).  Also a plugin could look for changes labeled as 'Urgent' and highlight them in some way ( Issue 861 ) and/or send out extra notifications.
Blockedon: 287
Jun 2, 2011
#3 sachini...@gmail.com
 Issue 287  talks about the risk of the change to be merged that is it needs sanity as well as comprehensive testing.
 Issue 861  talks about the sending the mail notification plugin about the change.

What I am proposing is every change in gerrit review can be Flagged for Urgent for quick action by reviewers and submitters considering the criticality of the change.
Right to set the Urgency flag can be given to the person who has right to give +2 score so as to avoid any misuse.
As said in above 2 issues that change can be highlighted and notification mail/SMS can be sent to the concerned people if the change is flagged as Urgent.

Thanks,
Sachin Nikam.

Jun 2, 2011
Project Member #5 bklarson@gmail.com
 Issue 287  really talks about applying arbitrary labels on changes.  The use-case was for riskiness of a change, but the idea is for any sort of label to be applied to a change (high-risk, urgent, impacts-power-usage, dont-merge-yet, ready-to-test, etc, etc)

 Issue 861  talks about being able to extend Gerrit with plugins.  One use-case is supporting other notification systems, but there are lots of possible plugins.  Another use case is for a plugin to identify changes with the 'urgent' label and do special stuff with them.

I understand your proposal.  I'm suggesting that it can be handled in a generic way using these 2 features (Issues 287 and 861) rather than adding this capability directly to Gerrit.
Jul 18, 2011
#6 sachini...@gmail.com
Please close this issue.
Thanks.
Sep 15, 2014
Project Member #7 gustaf.l...@sonyericsson.com
Closing on the request of sachinikam. Arbitrary labels support has also been mergedin 287.
Status: Duplicate
Mergedinto: 287
Blockedon: -gerrit:287 gerrit:287
Sign in to add a comment

Powered by Google Project Hosting