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): |
https://codereview.adblockplus.org/29337751/ |
||
Description (last modified by fhd)
Background
#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:
- Install XmlRpcPlugin.
- Because we're using AccountManagerPlugin, we also need to install HttpAuthPlugin and configure it for the `/login/xmlrpc` path.
- 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:3 Changed on 02/22/2016 at 04:37:12 PM by fhd
- Blocking 3678 added
comment:5 Changed on 02/24/2016 at 08:34:10 PM by fhd
- Cc matze trev added
- Owner set to 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: https://hg.adblockplus.org/infrastructure/rev/887722028fd2
(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: https://hg.adblockplus.org/infrastructure/rev/4938cecd1473
(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
- facebook.adblockplus.me
- infrastructure
- libadblockplus-binaries
- libadblockplus
- notifications
- screenshotter
- share.adblockplus.org
- sitescripts
- subscribe.adblockplus.org
- testpages.adblockplus.org
- urlfixer
- web.acceptableads.org
- web.adblockbrowser.org
- web.adblockplus.org
- web.eyeo.com
- youtube.adblockplus.me
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.
comment:14 Changed on 02/29/2016 at 12:14:50 PM by fhd
- Resolution set to fixed
- Status changed from reviewing to closed
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.)