779126 - Make revision SHAs link to Treeherder not hg.mozilla.org (fixes 404s)

Status: RESOLVED FIXED

(Tree Management Graveyard :: OrangeFactor, defect, P2)

Description

[Ed Morley [:emorley]]

1) Go to: http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=759711&startday=2012-07-24&endday=2012-07-31&tree=mozilla-aurora
2) Click on one of the links in the revision column

Expected:
Taken to http://hg.mozilla.org/releases/mozilla-aurora/rev/<changeset_hash>

Actual:
Taken to http://hg.mozilla.org/mozilla-aurora/rev/<changeset_hash>

Same for beta/inbound.

Guess we either need to have two variables in orangefactor.conf (repoName and repoURL), or just switch the orangefactor.conf entries to "releases/mozilla-aurora, integration/mozilla-inbound" etc, and filter out when we just need the repo name.

Comment 2

[Ed Morley [:emorley]]

Wontfix in favour of OrangeFactor v2 which will consume Treeherder's API, and likely be written from scratch.

Comment 3

[Ed Morley [:emorley]]

Thought of an easier fix to this than trying to provide a mapping of repo prefixes - let's just link to Treeherder, since (a) it's likely more useful, and (b) it doesn't use the prefixes, so the links will always work :-)

Comment 4

[Ed Morley [:emorley]]

Attachment: Make revision SHAs link to Treeherder not hg.mozilla.org

This is both more useful for investigating the failures (easier to get to the
log) and also fixes the broken links for trees whose hg.mozilla.org location is
not at the site root.

Comment 5

[Jonathan Griffin (:jgriffin)]

Comment on attachment 8629152 [details] [diff] [review]
Make revision SHAs link to Treeherder not hg.mozilla.org

Review of attachment 8629152 [details] [diff] [review]:
-----------------------------------------------------------------

Cool, makes sense.

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/automation/orangefactor/rev/7460b67a8ef6

Deployed.