RFC: CVS Admin with Flags (Version 1)

Warren Togami wtogami at redhat.com
Thu Feb 8 05:05:40 UTC 2007


I think this proposal generally works as-is.  But perhaps we can do 
better with a small optimization.  The reviewer setting fedora-review+ 
could possibly be the one who sets fedora-cvs?.  This could reduce the 
time overhead a bit by eliminating waiting on the owner to request what 
they presumably they would need to do next.

The only drawback of this, how does the reviewer know which branches the 
owner desires to build to, and who the owner wants as initial 
co-maintainers?

A possible solution, is for the filed review itself to state these 
things up front.

Thoughts?  Unless there are any major objections, this is going ahead as 
the standard process during Thursday.



Current Crappy CVSSyncNeeded Wiki Procedure
===========================================
http://fedoraproject.org/wiki/Extras/CVSSyncNeeded
1. Request new package and branch.
2. Wait until somebody creates empty directories and edits owners.list.
3. Owner checks stuff in and builds.

Using the Wiki for this process has always sucked.  We could embed this
process within the Bugzilla review tickets themselves.

Proposal: CVS Admin with Flags
==============================
1) Review is complete, fedora-review+
2) Owner writes in the Bugzilla comment something like:
<Branches> <PackageName> <FedoraAccountName> <BugzillaName>
FC-5 FC-6 foopackage bobjoe bobjoebugzilla at gmail.com
3) Set fedora-cvs flag to ?
4) CVS Admins get e-mail about fedora-cvs flag.  All context of the
review is within the bug itself, so they can easily read all details
about the package and verify approval validity.  The Admin then creates
CVS directories and sets owner.  Sets fedora-cvs flag to BLANK.
5) Owner checks in and builds.

Benefits
========
- This fedora-cvs flag eliminates the need for CVSSyncNeeded
entirely.  An actual work queue with tickets!
- fedora-cvs can be a simple canned query for CVS admins to see.
Awesome possibilities offered via RSS too... =)
- You could also use the fedora-cvs flag with explicit instructions
within any bug to do special requests, like:
"Please remove audacious-itouch.  We made some mistake.  Blah blah."

Notes
=====
- Unlike other flags, fedora-cvs is only BLANK or ?.  fedorabugs members
may request fedora-cvs by setting it to ?.  This sends an e-mail to CVS
Admins, signifying that attention is required.


Warren Togami
wtogami at redhat.com




More information about the Fedora-maintainers mailing list