-
Notifications
You must be signed in to change notification settings - Fork 3
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
Automatically create an Issue on failed scheduled builds #21
Automatically create an Issue on failed scheduled builds #21
Conversation
What I'm wondering about: If a downstream test is flaky, we will get a new issue every day?
Maybe we can additionally look for bots (or program one), which can be activated from every PR ( |
No, it shouldn't. With the
Not that I know of. This would be nice but I'm afraid one would have to inspect the build which is why I added the job link to the issue.
As stated above that should work already, however it will check for example for "CI build job rolling/main failed!" and update that.
That sounds nice. One possibility might be https://github.com/GregaVrbancic/gh-workflow-bot but that looks rather unmaintained. I think in the scope of this PR that might be over the top, we could keep that in mind for later, would you agree? |
I agree to all of your points. ICI has exit codes which can be parsed: ros-industrial/industrial_ci#755 at least the test step can be parsed as
see their CI otherwise we would have to write an own js wrapper around their script and parse the text output.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's test it as it is and improve it later if it is worth it.
Co-authored-by: Felix Exner (fexner) <[email protected]>
This should add an issue like this when the ICI fails in a scheduled build.
See workflow run here, where I artificially introduced a build error.
The issues data could be modified (title, labels, description...) but I would need input for that.