Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ThreadSanitizer CI failures on a PR are not shown in the Github UI #13041

Closed
OlivierNicole opened this issue Mar 21, 2024 · 4 comments
Closed

Comments

@OlivierNicole
Copy link
Contributor

The TSan CI is no longer listed in the list of checks at the bottom of a PR, even when someone labels the PR with run-thread-sanitizer. On the one hand, PRs now look nicer when that label is absent, because when all checks pass, Github folds them into a single, satisfying green mark (I remember @dra27 was sad that the skipped TSan checks used to show in grey and prevent such folding). On the other hand, when the label run-thread-sanitizer is added, one can see “All checks passed” even when the TSan CI failed, which is clearly not ideal.

@dra27
Copy link
Member

dra27 commented Mar 21, 2024

I'm confused - at the moment the PR does show the TSAN in the list, or is that now because it's merged?

@OlivierNicole
Copy link
Contributor Author

Mmh, strange, it used not to show TSan before the merge. I’m certain of that.

@dra27
Copy link
Member

dra27 commented Mar 22, 2024

There's I think some similar strangeness which can still happen with the no-changes-entry-needed label as well where the checks status after merge is different from the status before... let's definitely keep this open until we're sure what's going on with it!

@OlivierNicole
Copy link
Contributor Author

This no longers shows up, so I’ll assuem it was a Github bug and close this. We can reopen if the issue ever reappears.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants