Process Change: Package Reviews with Flags

[Warren Togami]

Warren Togami wrote:
> Further filing of review bugs is blocking on two issues:
> 
> 1) We must decide whether or not the review should be assigned to the 
> reviewer or package owner.  I believe package owner is more logical, 
> because that person is accountable to doing the work.  The reviewer is 
> already tracked by name in the flag itself, which too is logical.

OK, it seems the only real drawback to "ASSIGNED to owner instead of 
reviewer" is Tibbs' good point about being able to see it on frontpage.cgi.

https://bugzilla.redhat.com/bugzilla/request.cgi
dkl mentioned this interface that provides a way to query for flags.  He 
said that we can add a query like this to frontpage.cgi later after we 
figure out exactly what we want to do.  This would take care of the 
personal work-flow tracking case.  Other canned queries can take care of 
the general "see everything in this state" case.

I believe we should go ahead with this as the package review process for 
not only the mass review, but all future reviews.

Steps to make this happen:
1) File the mass review bugs, auto-assigned to the package owners.
2) FESCO should discuss and vote to ratify this process change.
3) Documentation should be updated.

I'd like to go ahead with filing step #1 on Wednesday.

Thoughts?

Warren Togami
wtogami at redhat.com