Closed Bug 1306839 Opened 8 years ago Closed 8 years ago

push_printurls should use 40 char SHAs for the revision in Treeherder/Perfherder URLs

Categories

(Developer Services :: Mercurial: hg.mozilla.org, defect)

Product:
Developer Services
Component:
Mercurial: hg.mozilla.org
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: emorley, Assigned: gps)

References

Details

The push_printurls hghook currently uses 12 character rather than 40 character SHAs for the revision in Treeherder & Perfherder URLs. https://hg.mozilla.org/hgcustom/version-control-tools/file/tip/hghooks/mozhghooks/push_printurls.py#l20 tip_node = short(repo.changectx(tip).node()) ... ui.write(' %s/#/jobs?repo=%s&revision=%s\n' % (treeherder_base_url, treeherder_repo, tip_node)) ... ui.write(' %s/perf.html#/comparechooser' '?newProject=try&newRevision=%s\n' % ( treeherder_base_url, tip_node))
Flags: needinfo?(gps)
Assignee: nobody → gps
Status: NEW → ASSIGNED
Flags: needinfo?(gps)
Pushed by gszorc@mozilla.com: https://hg.mozilla.org/hgcustom/version-control-tools/rev/1759fdffd03f hghooks: use full SHA-1 hash in push_printurls hook
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Blocks: 1311749
You need to log in before you can comment on or make changes to this bug.