CI status in PR differs from status in ci.nodejs.org

[vsemozhetbyt]

#17345

PR status: All checks have passed. 13 successful checks

ci.nodejs.org status: one failure (centos6-64), one machine (freebsd10-64) still running (https://ci.nodejs.org/job/node-test-commit/14457/)

P.S. Now the status has suddenly turned into "1 pending and 12 successful checks".
P.P.S Now it is "All checks have passed. 13 successful checks" again, but we still have that one failure.

[Fishrock123]

@vsemozhetbyt did you try hard reloading the page?

Also I don't see CI status on any of that PR's commits?

[vsemozhetbyt]

@Fishrock123 Yes, I tried several times. Now, after landing and closing, the status is gone in the PR, but it was all green in the end (despite the last https://ci.nodejs.org/job/node-test-pull-request/11798/). Should I close this issue until something similar happens?

[refack]

We still have a race problem with the status updates:

1. One of the linux variants fail
2. GitHub reports fail
3. Another linux variant succeeds
4. GitHub reports success

/CC @maclover7 @gibfahn IMHO simplest fix is to report on completion of node-test-commit

[apapirovski]

@maclover7 @refack @gibfahn any news on this? Does it work as expected now or is there still a race condition?

[refack]

node-test-commit does report it's status now so this can be closed