1315572 - Disable posting comment for try push by default

Status: RESOLVED WONTFIX

(Developer Services :: Mercurial: bzpost, defect)

Description

[Xidorn Quan [:xidorn] UTC+10]

This information is not quite useful for majority of bug subscribers as it doesn't contain any discussion nor indication of the progress of the bug.

In addition, when someone tries to trigger lots of try pushes for a bug, it could also become annoying as it takes a long part in the bug page, and creates lots of bug mail traffic.

(I feel a bit annoyed by, for example, bug 1307546 comment 40 to 53).

But some people do think it is useful to have records in bug page for all related things, and I think we should resolve that via having a filter in treeherder and link that in the bugzilla ui, instead of recording try pushes as comments.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

I think it is useful to record `./mach try` pushed to bugs so that all information relating to progress on fixing a bug is present in one public place.

Recording try pushes to bugs would be more useful however if the commit messages for try pushes were recorded in bugs as well, so that the purpose of the try push was clearer.

Additionally, it would be useful to have some easily identifiable string in the comment, so that people can filter out try push recording comments easier.

Comment 2

[Gregory Szorc [:gps]]

Since pulsebot handles bug updates for most repos, at this point the only value bzpost is providing is publishing Try pushes (which pulsebot could do if we really wanted).

So, if you want to stop posting comments about Try pushes, just disable the bzpost extension.

Regarding filtering messages in Bugzilla, that could be facilitated by adding a "tag" to the comment. But that would be a separate feature request. And at the point we can do that, I'd just as soon have pulsebot post with that tag, have Bugzilla hide comments with that tag by default, and delete bzpost.

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

That wouldn't stop people from receiving email of try pushes. Most of time, receiving email about try pushes is fine, but sometimes people may do lots of try pushes in a short time, which could be annoying. That's why I think it should be disabled, and we should have a different entry which allows people to browse all related try pushes but not record them in bug comments directly.