Globally install push_urls hook

RESOLVED FIXED

Status

Developer Services
Mercurial: hg.mozilla.org
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ewong, Assigned: gps)

Tracking

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Currently when pushing to http://hg.mozilla.org/SeaMonkey/puppet, a
commit hook message appears; but it shows that I had pushed to
http://hg.mozilla.org/build/puppet/rev/<cset>, which isn't the case.

I believe it would lessen the confusion if the commit hook (or whatever
it is run when someone pushes to that repo) is changed to show
that the commit was pushed to http://hg.mozilla.org/SeaMonkey/puppet.
(Assignee)

Comment 1

3 years ago
I'm sick and tired of these one-off bugs related to the push_urls hook. Let's just globally install the hook and be done with it.

We'll want to remove the hook for existing hgrc files once this is deployed or we may have the hook run multiple times.

I should have time to look at this in the next day or two, once bug 1189449 is out of the way.
Assignee: nobody → gps
Status: NEW → ASSIGNED
Priority: -- → P1
Summary: change commit hook for hg.mozilla.org/SeaMonkey/puppet → Globally install push_urls hook
(Assignee)

Comment 2

3 years ago
Created attachment 8641857 [details]
MozReview Request: ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r?fubar

ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r?fubar

This hook is manually installed on many repos. It is generally useful.
Let's just install it globally so we don't have to continue spending
time managing it on all the repos.

The added config value matches exactly what exists in 94 repos on
hg.mozilla.org. Therefore, this is safe to deploy without any additional
changes to existing repository configs. There won't be multiple
invocations of the hook or anything like that.
Attachment #8641857 - Flags: review?(klibby)
(Assignee)

Comment 3

3 years ago
Comment on attachment 8641857 [details]
MozReview Request: ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r?fubar

ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r?fubar

This hook is manually installed on many repos. It is generally useful.
Let's just install it globally so we don't have to continue spending
time managing it on all the repos.

The added config value matches exactly what exists in 94 repos on
hg.mozilla.org. Therefore, this is safe to deploy without any additional
changes to existing repository configs. There won't be multiple
invocations of the hook or anything like that.
Comment on attachment 8641857 [details]
MozReview Request: ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r?fubar

https://reviewboard.mozilla.org/r/14619/#review13219

Ship It!
Attachment #8641857 - Flags: review?(klibby) → review+
(Assignee)

Comment 5

3 years ago
url:        https://hg.mozilla.org/hgcustom/version-control-tools/rev/a502fb4b12f77cd340387cf115096a164d3f26b8
changeset:  a502fb4b12f77cd340387cf115096a164d3f26b8
user:       Gregory Szorc <gps@mozilla.com>
date:       Mon Aug 03 10:18:45 2015 -0700
description:
ansible/hg-ssh: globally install push_printurls hook (bug 1189566); r=fubar

This hook is manually installed on many repos. It is generally useful.
Let's just install it globally so we don't have to continue spending
time managing it on all the repos.

The added config value matches exactly what exists in 94 repos on
hg.mozilla.org. Therefore, this is safe to deploy without any additional
changes to existing repository configs. There shouldn't be multiple
invocations of the hook or anything like that.
(Assignee)

Comment 6

3 years ago
I'll deploy this shortly.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Blocks: 1190494
(Assignee)

Updated

3 years ago
Duplicate of this bug: 1188858
You need to log in before you can comment on or make changes to this bug.