Ryan VanderMeulen [:RyanVM] Reporter
	Description • 10 years ago

Looks like that got missed when porting the grouping patch over from TBPL

	Ed Morley [:emorley]
	Comment 1 • 10 years ago

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #0)
> Looks like that got missed when porting the grouping patch over from TBPL

This change was made:
https://github.com/mozilla/treeherder-service/commit/d725cae68a1b06d3ac3733f8cc261ac75d256418

And strangely the tests added both seem correct & are passing:
https://github.com/mozilla/treeherder-service/blob/master/tests/etl/test_buildbot.py#L1060

But yet the data from a recent job shows a job_group_symbol of "?" and job_group_name of "unknown", when they should be "W" and "W3C Web Platform Tests" like one of the numbered web platform jobs (eg https://treeherder.mozilla.org/api/project/mozilla-central/jobs/492978/).

Very strange.

	Ed Morley [:emorley]
	Comment 2 • 10 years ago

Mauro, we've deployed several times since the commit linked in comment 1 landed - and whilst we know workers can get stuck - I believe you asked fubar to manually kill them recently, so that shouldn't be the cause. Can you think of any other reason why the job ingestion might not be using the latest code here?

	Ed Morley [:emorley]
	Comment 3 • 10 years ago

Based on the conversation in IRC, Mauro has an idea what is going on here.

	Treeherder GitHub Bugbot
	Comment 4 • 10 years ago

Commits pushed to master at https://github.com/mozilla/treeherder-service

https://github.com/mozilla/treeherder-service/commit/70c85484cd317dd6cbb8ef4f1bf7cdfdf9d11654
Bug 1081600 - add a test to confirm the behaviour of job type/job group associations

https://github.com/mozilla/treeherder-service/commit/d4bbb0475d8df54c6ac07fa2f70a7d7ff794e659
Merge pull request #253 from mozilla/bug-1081600-investgate-job-group-update

Bug 1081600 - add a test to confirm the behaviour of job type/job group ...