1043249 - Move platform 'friendly name' mapping from the frontend into the API

Status: RESOLVED WONTFIX

(Tree Management :: Treeherder: API, defect, P3)

Description

[Ed Morley [:emorley]]

One of the future advantages of treeherder is that in the future a whole raft of consumers can use its API to save the continual re-inventing the wheel when it comes to platform/test suite/parser regexes etc.

As part of bug 1042581 I noticed that we seem to only handle the mapping of platforms to their friendly name (ie: "b2g-linux32" -> "B2G Desktop Linux") in treeherder-ui, rather than in treeherder-service.

This means other consumers will need to duplicate this mapping, which seems like something that will limit us in the future.

Also, the "raw platform names" (eg: "b2g-linux32", "android-2-3-armv6") are purely a TBPL carryover - ie: they are arbitrary internal names used by TBPL and don't match anything in buildbot [1]. As such, they are not a huge amount more less likely to change than the "friendly name".

[1] Albeit they are not as internal-to-TBPL as I'd like, since TBPL submits them as-is to OrangeFactor, which isn't ideal but means OF needs to map against them at present, though once OrangeFactor switches to using treeherder's API this will go away.

Comment 2

[Ed Morley [:emorley]]

As part of this, it will be worth considering whether:
(a) we see ourselves standardising on platform names used elsewhere (eg mozinfo or taskcluster) instead of the home-grown treeherder names
(b) we should store the friendly platform name in the taskcluster in-tree config instead

See also bug 1174186, bug 1060769 and bug 1196933.

Comment 3

[Ed Morley [:emorley]]

(In reply to Ed Morley [:emorley] from comment #2)
> As part of this, it will be worth considering whether:
> (a) we see ourselves standardising on platform names used elsewhere (eg
> mozinfo or taskcluster) instead of the home-grown treeherder names
> (b) we should store the friendly platform name in the taskcluster in-tree
> config instead
> 
> See also bug 1174186, bug 1060769 and bug 1196933.

Closing this in favour of (b) - for which I've filed bug 1458560.