https://bugzilla.wikimedia.org/show_bug.cgi?id=52167

--- Comment #2 from [email protected] ---
(In reply to comment #0)
> [ Adding link to gerrit and setting PATCH__TO_REVIEW are separate
>  actions ]

This splitting is an inherent limitation of the original design of
gerrit's hooks-its plugin. With some effort, we could work around that
from within hooks-bugzilla, but hooks-its' approach also comes with a
benefit: If one of the two actions fail, the other is still carried
out.

If they were tied to a single request, they'd either both fail or both
work.

Currently, there has been some discussion about from which states one
could/would/should allow switch to PATCH_TO_REVIEW.

Forbidding this for some states means that setting the status might
fail, but we want the comment to be added nonetheless. So until the
workflow is fully settled, I guess we should not try to merge the two
separate requests into a single one.

Well ... maybe that has settled already? Andre?

> This leads to a superfluous second bugmail message whenever a commit is
> pushed.

Yes, that's true.
One can tune bugzilla's noisiness in the User preferences:
  https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
allows to opt out of getting emails for "The priority, status,
severity, or milestone changes".

That mostly mitigates the problem, but does not solve it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to