On Wed, 2013-06-05 at 15:29 -0700, Jon Robson wrote:
I've been using ASSIGNED and assignee name to mean
this... to me if
someone is assigned to something it is a promise to attempt to fix it.
Ideally I'd use ASSIGNED when seriously(TM) planning to work on a patch.
I mean, seriously. But I know that there are small differences in
interpretation out there, and whatever works for a developer or a
development team also works for me.
I'm not sure how patch to review would be
useful... Gerrit should be
the place to look for things to review - not bugzilla.
I agree. This is more to minimize the small status mismatch we have to
due to manual activity in Bugzilla (bug reports still open though
patches have been merged which are supposed to fix the reports).
A RELEASED would however would be useful.
That's one of the next topics, see my PS at the bottom. :)
andre
On Wed, Jun 5, 2013 at 3:15 PM, Andre Klapper
<aklapper(a)wikimedia.org> wrote:
Hi everybody,
in December I mentioned the idea of having a "PATCH_AVAILABLE" or
"PATCH_TO_REVIEW" status in Bugzilla [1] and that we should re-evaluate
the idea once we have automatic notifications from Gerrit into Bugzilla
in place [2]. This is now the case [3].
From the Amsterdam Hackathon I know that some developers would like to
filter on bug reports that have or don't have a patch in Gerrit, and
easier finding of bug reports with a corresponding patch && lack of
recent changes might provide another entry point for new developers
(pick up the existing patch and finish it).
Hence I propose
* to remove the manually set and error-prone Bugzilla keyword
"patch-in-gerrit": Every bug on its way to get RESOLVED FIXED
has to pass this stage anyway so a status feels more
appropriate, and
* to make the "Gerrit Notification Bot" automatically change the
bug report status to "PATCH_AVAILABLE"/"PATCH_TO_REVIEW" in
Bugzilla when a patch for that bug report has been committed
(not: merged) to Gerrit.
Comments?
andre
[1]
http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/065046.html
[2]
http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/065226.html
[3]
https://bugzilla.wikimedia.org/show_bug.cgi?id=17322
PS: Making the Gerrit notification bot automatically close bug reports
in Bugzilla after merging a patch in Gerrit, or differentiating in
Bugzilla between "RESOLVED FIXED" (fix merged) and "RELEASED" (fix
deployed on the Wikimedia wikisites) are also interesting topics to
discuss at some point, but not in this thread. One step at a time.
--
Andre Klapper | Wikimedia Bugwrangler
http://blogs.gnome.org/aklapper/
_______________________________________________
Wikitech-l mailing list
Wikitech-l(a)lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l
--
Andre Klapper | Wikimedia Bugwrangler
http://blogs.gnome.org/aklapper/