Use Github Checks for taskcluster reporting

[bhearsum]

No description provided.

[escapewindow]

Timeout bustage appears to be #509 ; merging master in or rebasing the PR should resolve.

[bhearsum]

https://phabricator.services.mozilla.com/D123440 for the necessary ci-config change for this

[escapewindow]

Looks good, and I like that the checks tab is populated now.
The ci-config patch appears to be merged.

Do you know why we're getting this?

Setting change needed? Are we missing a decision task name here?

[bhearsum]

Looks good, and I like that the checks tab is populated now.
The ci-config patch appears to be merged.

Do you know why we're getting this?

Setting change needed? Are we missing a decision task name here?

That's due to branch protection settings:

If we want to depend on a single required check, we'll need to add a diamond closing one (eg: pr-complete). Alternatively, we can add all of the tasks as required checks (or not worry about it at all).

[escapewindow]

It looks like in #512 we had that. I'm guessing moving to checks-v1 removes that?

[escapewindow]

I think the real answer here is "we should port the scriptworker repo to taskgraph". =\

[bhearsum]

It looks like in #512 we had that. I'm guessing moving to checks-v1 removes that?

Yeah, checks replaces the overall "this task group passed or failed" with per-task status'. We can punt on this for now if you want, it's not a big deal.

[escapewindow]

Taskgraph is #534.