Opened on 02/22/2016 at 10:59:25 AM

Closed on 02/29/2016 at 12:14:50 PM

#3676 closed change (fixed)

Set up the hg hook that references commits in issues

Reported by: fhd Assignee: fhd
Priority: P4 Milestone:
Module: Infrastructure Keywords:
Cc: matze, trev Blocked By: #3674
Blocking: #3678 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by fhd)


#3674 adds a hook that references commits in the respective issues. Once that has landed, we need to set it up.

What to change

Since Trac doesn't have an API out of the box, we first need to set that up:

  1. Install XmlRpcPlugin.
  2. Because we're using AccountManagerPlugin, we also need to install HttpAuthPlugin and configure it for the `/login/xmlrpc` path.
  3. Set up a new abpbot Trac user and give it the XML_RPC permission.

Once that is done, we need to set up the new hook as a pretxnchangegroup hook (see #3679) in the hgrc of all repositories on hg1.

Attachments (0)

Change History (14)

comment:1 Changed on 02/22/2016 at 10:59:37 AM by fhd

  • Blocked By 3674 added

comment:2 Changed on 02/22/2016 at 11:00:59 AM by fhd

  • Description modified (diff)

comment:3 Changed on 02/22/2016 at 04:37:12 PM by fhd

  • Blocking 3678 added

comment:4 Changed on 02/22/2016 at 07:40:29 PM by fhd

  • Description modified (diff)

comment:5 Changed on 02/24/2016 at 08:34:10 PM by fhd

  • Cc matze trev added
  • Owner set to fhd

I'll have a go at this. Largely manual work, unfortunately... (Edit: No actually, I mixed something up: The Trac changes, except for creating the API user, will go through Puppet how it looks. Just the hgrc changes not, but those aren't the tricky part.)

Last edited on 02/24/2016 at 08:44:17 PM by fhd

comment:6 Changed on 02/26/2016 at 07:38:41 AM by fhd

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

The reviewable part is under review now. I'll do the manual work once that has landed.

comment:7 Changed on 02/26/2016 at 04:12:27 PM by fhd

Code change landed:

(If everything goes well, this was the last time I had to do that :D)

comment:8 Changed on 02/26/2016 at 05:01:32 PM by fhd

Created the user abpbot and granted it the XML_RPC permission.

comment:9 Changed on 02/26/2016 at 06:37:59 PM by fhd

  • Review URL(s) modified (diff)

Well there's one more infrastructure change required for this, the new hook needs to be configured on hg1. Added a review for that.

comment:10 Changed on 02/26/2016 at 07:11:10 PM by fhd

Second change landed:

(Yup, had to do it again...)

Also deployed. Now all that remains is to update the hgrcs...

comment:11 Changed on 02/29/2016 at 09:35:46 AM by fhd

Tested it successfully with the adblockplussafariios repository and #3712. Chaning all the hgrcs now...

comment:12 Changed on 02/29/2016 at 11:28:54 AM by fhd

Enabled the hook for the following repositories:

  • abp2blocklist
  • abpbot
  • abpcrawler
  • abpcustomization
  • abpwatcher
  • adblockbrowser-build
  • adblockbrowser
  • adblockplusandroid
  • adblockpluschrome
  • adblockpluscore
  • adblockplus
  • adblockplusie
  • adblockplussafariios
  • adblockplussbrowser
  • adblockplustests
  • adblockplusui
  • buildtools
  • cms
  • codingtools
  • contentblockerlists
  • downloads
  • elemhidehelper
  • infrastructure
  • libadblockplus-binaries
  • libadblockplus
  • notifications
  • screenshotter
  • sitescripts
  • urlfixer

Still wondering what exactly to do with some of the others.

comment:13 Changed on 02/29/2016 at 12:14:19 PM by fhd

Forgot two, both are not just dependencies but forks:

  • jshydra
  • mozharness

I'll leave the rest alone.

Last edited on 02/29/2016 at 12:14:42 PM by fhd

comment:14 Changed on 02/29/2016 at 12:14:50 PM by fhd

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from fhd.
Note: See TracTickets for help on using tickets.