Page MenuHomePhabricator

Create a Github notification bot for phabricator
Open, MediumPublic

Description

It would be useful to have a connection between Github and Phabricator, for projects that have repos on Github. This could be done by a Phab-bot, using webhooks to get information from Github and conduit to do things with Phab-tasks.

Things that should be handled like @gerritbot:

  • When a pull-request is created on Github, webhooks will trigger the bot.
  • If the bot finds Bug: TXXX in PR or commit message, it should post a message on the phab ticket with Patch-For-Review tag, and potentially assign this ticket to commiter (how ?)
  • When a pull-request is merged on Github, webhooks will trigger the bot.
  • The bot will post a message on the phab ticket.

What the bot should do in other words:

  • Assigning a reviewer on Github adds a comment with link to the PR and to the related task. Also adds Patch-For-Review.
  • Merging a PR adds comment to the related task.

Requirements:

  • PR:s need to provide task id(s).

Tool name: github-notif-bot - contact @Framawiki for access

Event Timeline

Framawiki awarded a token.
Framawiki added a subscriber: Framawiki.

It will be very useful for projects like Huggle.
I would like to work on this, by create a Python bot.

Framawiki triaged this task as Medium priority.May 10 2017, 7:45 AM
Framawiki updated the task description. (Show Details)May 10 2017, 9:23 AM

It will be very useful for projects like Huggle.
I would like to work on this, by create a Python bot.

Great if you would take this on! It is something we would need for a couple of our projects as well we just never get around to building it =)

Framawiki renamed this task from Automate Github to Phabricator actions to Create a Github notification bot for phabricator.May 11 2017, 4:22 PM
Framawiki added a comment.EditedMay 11 2017, 4:27 PM

Some research:

bd808 updated the task description. (Show Details)May 11 2017, 4:54 PM
bd808 awarded a token.

https://phab-01.wmflabs.org/p/Github-notif-bot/

>>> phab.maniphest.edit(objectIdentifier='T183', transactions=[{'type':'comment', 'value':'valeur'}, {'type':'projects.add', 'value':['Patch-For-Review']}])

:-)

Framawiki updated the task description. (Show Details)May 11 2017, 7:31 PM

I'll now create tasks in Github-notif-bot project and continue to make general reports here.

Framawiki moved this task from Backlog to V2 on the Github-notif-bot board.May 15 2017, 5:02 PM
Framawiki moved this task from V2 to Backlog on the Github-notif-bot board.May 15 2017, 5:09 PM
Framawiki moved this task from To Triage to Not ready to announce on the User-notice board.
Abbe98 added a subscriber: Abbe98.Sep 1 2017, 11:05 AM
MusikAnimal added a subscriber: MusikAnimal.

WMDE, especially the WMDE-FUN-Team might be interested in this as well.

Sebastian_Berlin-WMSE moved this task from Backlog to IT on the WMSE board.Jan 23 2019, 8:19 AM
Sebastian_Berlin-WMSE edited projects, added WMSE (IT); removed WMSE.
He7d3r added a subscriber: He7d3r.May 6 2020, 6:01 PM
Aklapper removed Framawiki as the assignee of this task.Jun 19 2020, 4:21 PM
Aklapper added a subscriber: Aklapper.

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)