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.