Skip to content

Conversation

@mrc0mmand
Copy link
Member

@mrc0mmand mrc0mmand commented May 8, 2020

Apart from running CIFuzz for each relevant PR, let's run it
unconditionally for each push to master to detect possible issues
(caused by ignored PRs, etc.).

Followup to 94f660a.

/cc @evverx

@mrc0mmand mrc0mmand added ci fuzzing Implementation of fuzzers and fixes for stuff found through fuzzing labels May 8, 2020
Apart from running CIFuzz for each relevant PR, let's run it
unconditionally for each push to master to detect possible issues
(caused by ignored PRs, etc.).

Followup to 94f660a.
@mrc0mmand mrc0mmand force-pushed the cifuzz-run-on-master branch from 7283dd2 to cf70a1b Compare May 8, 2020 18:23
@evverx
Copy link
Contributor

evverx commented May 8, 2020

I'll go ahead and merge it. Though given google/oss-fuzz#3731 (comment) I'm not sure how exactly CIFuzz reacts to push events. Let's find out :-)

@evverx evverx merged commit a114ab6 into systemd:master May 8, 2020
@mrc0mmand
Copy link
Member Author

Looks like the GH gods have been merciful and the fuzzers built successfully, at least in this case. Hopefully it's going to stay that way :-)

@mrc0mmand mrc0mmand deleted the cifuzz-run-on-master branch May 8, 2020 18:39
@evverx
Copy link
Contributor

evverx commented May 8, 2020

To judge from
Screenshot 2020-05-08 at 21 33 57

workflow runs aren't cancelled when PRs are updated. I might have to read https://github.community/t5/GitHub-Actions/Is-it-possible-to-cancel-previous-jobs-automatically-when-the/td-p/37139 :-)

@mrc0mmand
Copy link
Member Author

Ouch, that's unfortunate. I naively hoped this is (finally) handled automagically.

evverx added a commit to evverx/systemd that referenced this pull request May 19, 2020
evverx added a commit to evverx/systemd that referenced this pull request May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci fuzzing Implementation of fuzzers and fixes for stuff found through fuzzing

Development

Successfully merging this pull request may close these issues.

2 participants