Hey,
You probably know about the tag, it has been added automatically to phabricator tickets when someone makes a patch to gerrit and mentions the pabricator ticket (For example [0]). The problem is when the patch gets merged the tag doesn't get removed automatically because there might be other outstanding patches, etc. Now, lots of time of our developers is being spent on removing those tags when they don't apply which is tedious and with lots of tasks incorrectly having the tag, the tag itself is useless. So I wrote a bot to clean those tags. In order to make sure I don't remove anything by mistake, the bot doesn't touch the ticket if any of these conditions apply: * If the tag is added by someone else than the gerrit bot * If there's a link to github.com anywhere * If there are still patches that are not merged (ta-duh) * If gerritbot in any way acts out of ordinary (for example, comments twice for adding one patch) It's going to spam your inbox. Sorry :( I'm working on it to find a less spammy way and will definitely run it with slow pace. [0]: https://phabricator.wikimedia.org/T155878#5190647 -- Amir (he/him) _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
On 21/05/2019 23:40, Amir Sarabadani wrote:
> Hey, > You probably know about the tag, it has been added automatically to > phabricator tickets when someone makes a patch to gerrit and mentions the > pabricator ticket (For example [0]). The problem is when the patch gets > merged the tag doesn't get removed automatically because there might be > other outstanding patches, etc. > > Now, lots of time of our developers is being spent on removing those tags > when they don't apply which is tedious and with lots of tasks incorrectly > having the tag, the tag itself is useless. So I wrote a bot to clean those > tags. > > In order to make sure I don't remove anything by mistake, the bot doesn't > touch the ticket if any of these conditions apply: > * If the tag is added by someone else than the gerrit bot > * If there's a link to github.com anywhere > * If there are still patches that are not merged (ta-duh) > * If gerritbot in any way acts out of ordinary (for example, comments twice > for adding one patch) > > It's going to spam your inbox. Sorry:( I'm working on it to find a less > spammy way and will definitely run it with slow pace. > > [0]:https://phabricator.wikimedia.org/T155878#5190647 Hello, I appreciate your initiative to cleanup Phabricator but in this case I do not see a point of removing #patch-for-review on tasks that are marked as resolved and are months, if not years old. Can you please stop this bot because it is flooding my email inbox. Or at least exclude tasks marked as resolved! Thank you! -- Antoine "hashar" Musso _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
Hey,
It's already stopped and running them on open tasks only for now sounds like a good idea. Let me do that. On Tue, May 21, 2019 at 11:51 PM Antoine Musso <[hidden email]> wrote: > On 21/05/2019 23:40, Amir Sarabadani wrote: > > Hey, > > You probably know about the tag, it has been added automatically to > > phabricator tickets when someone makes a patch to gerrit and mentions the > > pabricator ticket (For example [0]). The problem is when the patch gets > > merged the tag doesn't get removed automatically because there might be > > other outstanding patches, etc. > > > > Now, lots of time of our developers is being spent on removing those tags > > when they don't apply which is tedious and with lots of tasks incorrectly > > having the tag, the tag itself is useless. So I wrote a bot to clean > those > > tags. > > > > In order to make sure I don't remove anything by mistake, the bot doesn't > > touch the ticket if any of these conditions apply: > > * If the tag is added by someone else than the gerrit bot > > * If there's a link to github.com anywhere > > * If there are still patches that are not merged (ta-duh) > > * If gerritbot in any way acts out of ordinary (for example, comments > twice > > for adding one patch) > > > > It's going to spam your inbox. Sorry:( I'm working on it to find a less > > spammy way and will definitely run it with slow pace. > > > > [0]:https://phabricator.wikimedia.org/T155878#5190647 > > Hello, > > I appreciate your initiative to cleanup Phabricator but in this case I > do not see a point of removing #patch-for-review on tasks that are > marked as resolved and are months, if not years old. > > Can you please stop this bot because it is flooding my email inbox. Or > at least exclude tasks marked as resolved! > > Thank you! > > -- > Antoine "hashar" Musso > > -- Amir (he/him) _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
If you do the edits as a bulk edit job I can flag it to run silently which
will spare people the email spam. On Tue, May 21, 2019 at 4:53 PM Amir Sarabadani <[hidden email]> wrote: > Hey, > It's already stopped and running them on open tasks only for now sounds > like a good idea. > > Let me do that. > > On Tue, May 21, 2019 at 11:51 PM Antoine Musso <[hidden email]> wrote: > > > On 21/05/2019 23:40, Amir Sarabadani wrote: > > > Hey, > > > You probably know about the tag, it has been added automatically to > > > phabricator tickets when someone makes a patch to gerrit and mentions > the > > > pabricator ticket (For example [0]). The problem is when the patch gets > > > merged the tag doesn't get removed automatically because there might be > > > other outstanding patches, etc. > > > > > > Now, lots of time of our developers is being spent on removing those > tags > > > when they don't apply which is tedious and with lots of tasks > incorrectly > > > having the tag, the tag itself is useless. So I wrote a bot to clean > > those > > > tags. > > > > > > In order to make sure I don't remove anything by mistake, the bot > doesn't > > > touch the ticket if any of these conditions apply: > > > * If the tag is added by someone else than the gerrit bot > > > * If there's a link to github.com anywhere > > > * If there are still patches that are not merged (ta-duh) > > > * If gerritbot in any way acts out of ordinary (for example, comments > > twice > > > for adding one patch) > > > > > > It's going to spam your inbox. Sorry:( I'm working on it to find a > less > > > spammy way and will definitely run it with slow pace. > > > > > > [0]:https://phabricator.wikimedia.org/T155878#5190647 > > > > Hello, > > > > I appreciate your initiative to cleanup Phabricator but in this case I > > do not see a point of removing #patch-for-review on tasks that are > > marked as resolved and are months, if not years old. > > > > Can you please stop this bot because it is flooding my email inbox. Or > > at least exclude tasks marked as resolved! > > > > Thank you! > > > > -- > > Antoine "hashar" Musso > > > > > > -- > Amir (he/him) > _______________________________________________ > Wikitech-l mailing list > [hidden email] > https://lists.wikimedia.org/mailman/listinfo/wikitech-l Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
In reply to this post by Amir Sarabadani-2
On Tue, 2019-05-21 at 23:40 +0200, Amir Sarabadani wrote:
> So I wrote a bot to clean those tags. Thanks! Looks like you fixed https://phabricator.wikimedia.org/T1149 ? Currently the bot seems to use the @Ladsgroup Phab account? Have you considered using a dedicated bot account for these actions? https://www.mediawiki.org/wiki/Phabricator/Bots This would allow to separate #Patch-For-Review removal notifications from activity by Ladsgroup The Human. andre -- Andre Klapper | Bugwrangler / Developer Advocate https://blogs.gnome.org/aklapper/ _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
On Wed, 2019-05-22 at 11:56 +0200, Andre Klapper wrote:
> Have you considered using a dedicated bot account for these actions? Ah, we're discussing this in https://phabricator.wikimedia.org/T224099 andre -- Andre Klapper | Bugwrangler / Developer Advocate https://blogs.gnome.org/aklapper/ _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
In reply to this post by Mukunda Modell
On Wed, May 22, 2019 at 12:12 PM Mukunda Modell <[hidden email]>
wrote: > If you do the edits as a bulk edit job I can flag it to run silently which > will spare people the email spam. > I would love to have this done :-) I already have lots of phab traffic, so keeping this one to a minimum would be nice! Thanks! Manuel. > > On Tue, May 21, 2019 at 4:53 PM Amir Sarabadani <[hidden email]> > wrote: > > > Hey, > > It's already stopped and running them on open tasks only for now sounds > > like a good idea. > > > > Let me do that. > > > > On Tue, May 21, 2019 at 11:51 PM Antoine Musso <[hidden email]> > wrote: > > > > > On 21/05/2019 23:40, Amir Sarabadani wrote: > > > > Hey, > > > > You probably know about the tag, it has been added automatically to > > > > phabricator tickets when someone makes a patch to gerrit and mentions > > the > > > > pabricator ticket (For example [0]). The problem is when the patch > gets > > > > merged the tag doesn't get removed automatically because there might > be > > > > other outstanding patches, etc. > > > > > > > > Now, lots of time of our developers is being spent on removing those > > tags > > > > when they don't apply which is tedious and with lots of tasks > > incorrectly > > > > having the tag, the tag itself is useless. So I wrote a bot to clean > > > those > > > > tags. > > > > > > > > In order to make sure I don't remove anything by mistake, the bot > > doesn't > > > > touch the ticket if any of these conditions apply: > > > > * If the tag is added by someone else than the gerrit bot > > > > * If there's a link to github.com anywhere > > > > * If there are still patches that are not merged (ta-duh) > > > > * If gerritbot in any way acts out of ordinary (for example, comments > > > twice > > > > for adding one patch) > > > > > > > > It's going to spam your inbox. Sorry:( I'm working on it to find a > > less > > > > spammy way and will definitely run it with slow pace. > > > > > > > > [0]:https://phabricator.wikimedia.org/T155878#5190647 > > > > > > Hello, > > > > > > I appreciate your initiative to cleanup Phabricator but in this case I > > > do not see a point of removing #patch-for-review on tasks that are > > > marked as resolved and are months, if not years old. > > > > > > Can you please stop this bot because it is flooding my email inbox. Or > > > at least exclude tasks marked as resolved! > > > > > > Thank you! > > > > > > -- > > > Antoine "hashar" Musso > > > > > > > > > > -- > > Amir (he/him) > > _______________________________________________ > > Wikitech-l mailing list > > [hidden email] > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > _______________________________________________ > Wikitech-l mailing list > [hidden email] > https://lists.wikimedia.org/mailman/listinfo/wikitech-l Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
In lieu of a user mute feature[0], you can identify messages from the bot
in Gmail like: from:(Maintenance_bot [hidden email]) actor(@Maintenance_bot) You have to enable the "send stamps" option under Phabricator settings -> email format. [0] https://phabricator.wikimedia.org/T164542 On Thu, May 23, 2019 at 4:48 AM Manuel Arostegui <[hidden email]> wrote: > On Wed, May 22, 2019 at 12:12 PM Mukunda Modell <[hidden email]> > wrote: > > > If you do the edits as a bulk edit job I can flag it to run silently > which > > will spare people the email spam. > > > > I would love to have this done :-) > I already have lots of phab traffic, so keeping this one to a minimum would > be nice! > > Thanks! > Manuel. > > > > > > On Tue, May 21, 2019 at 4:53 PM Amir Sarabadani <[hidden email]> > > wrote: > > > > > Hey, > > > It's already stopped and running them on open tasks only for now sounds > > > like a good idea. > > > > > > Let me do that. > > > > > > On Tue, May 21, 2019 at 11:51 PM Antoine Musso <[hidden email]> > > wrote: > > > > > > > On 21/05/2019 23:40, Amir Sarabadani wrote: > > > > > Hey, > > > > > You probably know about the tag, it has been added automatically to > > > > > phabricator tickets when someone makes a patch to gerrit and > mentions > > > the > > > > > pabricator ticket (For example [0]). The problem is when the patch > > gets > > > > > merged the tag doesn't get removed automatically because there > might > > be > > > > > other outstanding patches, etc. > > > > > > > > > > Now, lots of time of our developers is being spent on removing > those > > > tags > > > > > when they don't apply which is tedious and with lots of tasks > > > incorrectly > > > > > having the tag, the tag itself is useless. So I wrote a bot to > clean > > > > those > > > > > tags. > > > > > > > > > > In order to make sure I don't remove anything by mistake, the bot > > > doesn't > > > > > touch the ticket if any of these conditions apply: > > > > > * If the tag is added by someone else than the gerrit bot > > > > > * If there's a link to github.com anywhere > > > > > * If there are still patches that are not merged (ta-duh) > > > > > * If gerritbot in any way acts out of ordinary (for example, > comments > > > > twice > > > > > for adding one patch) > > > > > > > > > > It's going to spam your inbox. Sorry:( I'm working on it to find a > > > less > > > > > spammy way and will definitely run it with slow pace. > > > > > > > > > > [0]:https://phabricator.wikimedia.org/T155878#5190647 > > > > > > > > Hello, > > > > > > > > I appreciate your initiative to cleanup Phabricator but in this case > I > > > > do not see a point of removing #patch-for-review on tasks that are > > > > marked as resolved and are months, if not years old. > > > > > > > > Can you please stop this bot because it is flooding my email inbox. > Or > > > > at least exclude tasks marked as resolved! > > > > > > > > Thank you! > > > > > > > > -- > > > > Antoine "hashar" Musso > > > > > > > > > > > > > > -- > > > Amir (he/him) > > > _______________________________________________ > > > Wikitech-l mailing list > > > [hidden email] > > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > > _______________________________________________ > > Wikitech-l mailing list > > [hidden email] > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > _______________________________________________ > Wikitech-l mailing list > [hidden email] > https://lists.wikimedia.org/mailman/listinfo/wikitech-l Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
Some updates:
* Using a batch job I removed around 140 tags from open tasks (An example: https://phabricator.wikimedia.org/T222963#5216986) You should not get any notification for those. This is basically the first and only big cleanup * The bot is active and removes the tag on tasks that are open and have been updated in the last hour, every hour. You will get notification for those. It's similar to release tagger bot and I recommand filtering them on your mail client if you get too many of them (as Stephen pointed out). This is rather low-volume action to avoid piling those up in the future. * I'm not touching closed tasks as it's lots of tickets and won't gain us much benefit, if you want me to do it for any reason you want. Let me know. * Now, the tag is only used on tickets that actually have a patch attached to them (open tickets, closed ones is a mess), making us be able to use the tag efficiently. For example, this is the tickets about wikidata that have outstanding patch: https://phabricator.wikimedia.org/project/board/71/query/3IFPR_mx6Nll/ Best On Tue, May 28, 2019 at 5:14 PM Stephen Niedzielski < [hidden email]> wrote: > In lieu of a user mute feature[0], you can identify messages from the bot > in Gmail like: > > from:(Maintenance_bot [hidden email]) > actor(@Maintenance_bot) > > You have to enable the "send stamps" option under Phabricator settings -> > email format. > > [0] https://phabricator.wikimedia.org/T164542 > > On Thu, May 23, 2019 at 4:48 AM Manuel Arostegui <[hidden email] > > > wrote: > > > On Wed, May 22, 2019 at 12:12 PM Mukunda Modell <[hidden email]> > > wrote: > > > > > If you do the edits as a bulk edit job I can flag it to run silently > > which > > > will spare people the email spam. > > > > > > > I would love to have this done :-) > > I already have lots of phab traffic, so keeping this one to a minimum > would > > be nice! > > > > Thanks! > > Manuel. > > > > > > > > > > On Tue, May 21, 2019 at 4:53 PM Amir Sarabadani <[hidden email]> > > > wrote: > > > > > > > Hey, > > > > It's already stopped and running them on open tasks only for now > sounds > > > > like a good idea. > > > > > > > > Let me do that. > > > > > > > > On Tue, May 21, 2019 at 11:51 PM Antoine Musso <[hidden email]> > > > wrote: > > > > > > > > > On 21/05/2019 23:40, Amir Sarabadani wrote: > > > > > > Hey, > > > > > > You probably know about the tag, it has been added automatically > to > > > > > > phabricator tickets when someone makes a patch to gerrit and > > mentions > > > > the > > > > > > pabricator ticket (For example [0]). The problem is when the > patch > > > gets > > > > > > merged the tag doesn't get removed automatically because there > > might > > > be > > > > > > other outstanding patches, etc. > > > > > > > > > > > > Now, lots of time of our developers is being spent on removing > > those > > > > tags > > > > > > when they don't apply which is tedious and with lots of tasks > > > > incorrectly > > > > > > having the tag, the tag itself is useless. So I wrote a bot to > > clean > > > > > those > > > > > > tags. > > > > > > > > > > > > In order to make sure I don't remove anything by mistake, the bot > > > > doesn't > > > > > > touch the ticket if any of these conditions apply: > > > > > > * If the tag is added by someone else than the gerrit bot > > > > > > * If there's a link to github.com anywhere > > > > > > * If there are still patches that are not merged (ta-duh) > > > > > > * If gerritbot in any way acts out of ordinary (for example, > > comments > > > > > twice > > > > > > for adding one patch) > > > > > > > > > > > > It's going to spam your inbox. Sorry:( I'm working on it to > find a > > > > less > > > > > > spammy way and will definitely run it with slow pace. > > > > > > > > > > > > [0]:https://phabricator.wikimedia.org/T155878#5190647 > > > > > > > > > > Hello, > > > > > > > > > > I appreciate your initiative to cleanup Phabricator but in this > case > > I > > > > > do not see a point of removing #patch-for-review on tasks that are > > > > > marked as resolved and are months, if not years old. > > > > > > > > > > Can you please stop this bot because it is flooding my email inbox. > > Or > > > > > at least exclude tasks marked as resolved! > > > > > > > > > > Thank you! > > > > > > > > > > -- > > > > > Antoine "hashar" Musso > > > > > > > > > > > > > > > > > > -- > > > > Amir (he/him) > > > > _______________________________________________ > > > > Wikitech-l mailing list > > > > [hidden email] > > > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > > > _______________________________________________ > > > Wikitech-l mailing list > > > [hidden email] > > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > > _______________________________________________ > > Wikitech-l mailing list > > [hidden email] > > https://lists.wikimedia.org/mailman/listinfo/wikitech-l > _______________________________________________ > Wikitech-l mailing list > [hidden email] > https://lists.wikimedia.org/mailman/listinfo/wikitech-l -- Amir (he/him) _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
In reply to this post by Stephen Niedzielski
On Tue, 2019-05-28 at 09:13 -0600, Stephen Niedzielski wrote:
> you can identify messages from the bot in Gmail like: > > from:(Maintenance_bot [hidden email]) > actor(@Maintenance_bot) > > You have to enable the "send stamps" option under Phabricator settings -> > email format. See https://www.mediawiki.org/wiki/Phabricator/Help/Managing_mail for more information on who to handle Phabricator notifications. andre -- Andre Klapper | Bugwrangler / Developer Advocate https://blogs.gnome.org/aklapper/ _______________________________________________ Wikitech-l mailing list [hidden email] https://lists.wikimedia.org/mailman/listinfo/wikitech-l |
Free forum by Nabble | Edit this page |