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

CIFuzz spawns ClusterFuzzLite CI job with neutral status #12507

Open
echeran opened this issue Sep 16, 2024 · 3 comments
Open

CIFuzz spawns ClusterFuzzLite CI job with neutral status #12507

echeran opened this issue Sep 16, 2024 · 3 comments

Comments

@echeran
Copy link

echeran commented Sep 16, 2024

We are currently using CIFuzz in our Github CI. It turns out that at the end of each CIFuzz job that is kicked off by the Github Action, a followup job called ClusterFuzzLite/CIFuzz is also run. It was interfering with one of our one our CI workflows, which uses an action that is designed to intelligently enforce that CI checks are passing. See unicode-org/icu#3160 for our experience with the interference.

The reason for the interference is that, empirically, the ClusterFuzzLite/CIFuzz job starts off initially in a "neutral" state, but then appears to run to completion. The neutral state is interpreted as unsuccessful by the wait-for-status-checks job (example).

So the fact that the job ClusterFuzzLite/CIFuzz starts off reporting its status in a "neutral" state seems like a bug. Is there a way that it can start off with some type of "in progress" status of some kind instead? And only change the status to something else once the job conclusively ends successfully one way or another?

@echeran
Copy link
Author

echeran commented Sep 16, 2024

cc @FrankYFTang

@jonathanmetzman
Copy link
Contributor

Sorry for this terribly late reply. Do you have any pointers to docs on neutral vs inprogress status on github actions?

@echeran
Copy link
Author

echeran commented Nov 15, 2024

I don't have any docs about how Github Actions interprets things. All I have is empirical conclusions based on observations. One of the links that I included above is PR unicode-org/icu#3160 above. In that PR, you can also look at the discussion and inline "code" comments to get a better sense of what happened and how the workaround needed to be designed to avoid those specific issues.

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