Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

chore: raise an issue when nightly build fails #11

Merged
merged 5 commits into from
Mar 21, 2023

Conversation

TomAFrench
Copy link
Member

@TomAFrench TomAFrench commented Mar 10, 2023

We've had failing nightly builds for a couple of weeks now but I didn't notice it until today due to us not having monitoring set up.

As a simple version of this, the build workflows now open an issue when they fail to ping us. I've added a couple of people to be assigned based on who tends to tinker with CI the most, if you'd rather be removed from this then let me know.

@TomAFrench TomAFrench force-pushed the raise-issue-on-failing-nightly branch 2 times, most recently from 0093c38 to 5cea614 Compare March 10, 2023 17:51
@TomAFrench TomAFrench force-pushed the raise-issue-on-failing-nightly branch from 5cea614 to 01a8eaa Compare March 10, 2023 18:06
@TomAFrench TomAFrench force-pushed the raise-issue-on-failing-nightly branch from 01a8eaa to 476abd7 Compare March 10, 2023 18:07
@TomAFrench TomAFrench requested a review from phated March 10, 2023 18:22
@TomAFrench
Copy link
Member Author

Looks like this is triggering even when the tests pass now for some reason. Will investigate later.

@TomAFrench
Copy link
Member Author

Seems like moving the condition inside the braces worked.

@TomAFrench TomAFrench mentioned this pull request Mar 13, 2023
Copy link
Contributor

@phated phated left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay. I wanted to look over the action. In general, I think we should fork an audit each action we pull into the organization, especially ones that are provided our secrets.

Alternatively, we can use the GitHub CLI which is install in each runner. Something like: https://cli.github.com/manual/gh_issue_create

@TomAFrench TomAFrench merged commit 2535662 into master Mar 21, 2023
@TomAFrench TomAFrench deleted the raise-issue-on-failing-nightly branch March 21, 2023 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants