1199825 - Treeherder returns an stale builder when querying for hidden jobs

Status: RESOLVED WONTFIX

(Tree Management :: Treeherder: API, defect)

Description

[Chris Manchester (limited bugmail, email directly)]

I noticed the trigger-bot is triggering some hidden jobs when it shouldn't... I think something caused some confusion when I split out CPP and the upcoming GTest job in bug 1196464.

For https://treeherder.mozilla.org/#/jobs?repo=try&revision=4373f89a86eb&exclusion_profile=false I see the GTest job ran with builder names like "Ubuntu VM 12.04 try opt test gtest", but when I https://treeherder.mozilla.org/api/project/try/jobs/?count=2000&result_set_id=66419&return_type=list&visibility=excluded I have names like "Windows XP 32-bit try opt test cpp_gtest".

There was a short time the new regex in treeherder was matching the old builder name, so maybe treeherder only stores the first match per job symbol?

Comment 1

[Ed Morley [:emorley]]

I believe this would be fixed by the solutions in bug 1193372 - Cameron can you confirm?

Comment 2

[Cameron Dawson [:camd]]

I think this is a different issue.  bug 1193372 really has to do with a job_type always going with a specific job_group, even if the data says it should change to a different one.

tbh: I'm not sure I totally grok this bug, but it sounds like our buildername matching in ``/etl/buildbot.py`` may be wrong still?  That being said, that resultset is from August 28. 

Chmanchester: is this still happening?  this will probably require some deep digging to see what's going on in there.

Comment 3

[Chris Manchester (limited bugmail, email directly)]

It is still happening, but the mismatch is different. For instance, https://treeherder.mozilla.org/api/project/try/jobs/?count=2000&result_set_id=77817&return_type=list&visibility=excluded has "Ubuntu VM 12.04 x64 try debug test gtest", but https://treeherder.mozilla.org/#/jobs?repo=try&revision=26698a4cb478&exclusion_profile=false has "Ubuntu VM large 12.04 x64 try debug test gtest" (this is the more recent, and correct, builder).

I don't expect this job to be hidden very much longer, so it might not be a priority.

Comment 4

[Ed Morley [:emorley]]

We're going to replace exclusion profiles with people always using the in-tree (mozilla-central) tier values instead (bug 1387640).