-
-
Notifications
You must be signed in to change notification settings - Fork 394
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
Missing v10.0.0 prebuilt packages for darwin-x64 and linux-arm #1186
Comments
GitHub failed a bunch of builds for that version due to what seems to be a pervasive network outage, and unfortunately the GitHub build pipeline doesn't seem to retry properly. Any GHA experts out there? Is there a better way to get our pipeline to retry automatically to work around GHA flakiness? As it stands, it's pretty miserable. |
Looks like the linux arm still needs to be rebuilt. I wish I had a solution for it too— From what I can tell, retrying pipelines is
|
The binaries for better-sqlite3 v10 aren't currently available on ARM due to Github Action flakiness: WiseLibs/better-sqlite3#1186
Hey @mceachen, I’ve been thinking about the GitHub flakiness for One other option is for someone to manually re-run the workflows. While it would be great if GitHub actions had automatic job retries or another way to solve the issue programmatically, I have found that I just normally re-run the workflows myself. In my project, I use rolling dependency updates, and am notified when one fails, like If you are open to granting me contributor rights to re-run workflows, I can check the I completely understand if you are uncomfortable with assigning me contributor rights. I pledge to only use it to re-run workflows. Any changes I propose will continue to use your PR process. Unfortunately, I could not find any way to limit contributors to only running GitHub Actions. Sometimes the easiest solution is a manual one. I would start by re-running this workflow https://github.com/WiseLibs/better-sqlite3/actions/runs/9048872914 |
Unfortunately, in the past when I've tried to re-run any failing workflows, they don't seem to actually work--I don't know if it's missing prior task context or something else, but in any event, it seems that the only remedy is to create a new patch release and hope for the best. As far as contributor rights, know that I've got quite restricted contributor access to this repo--you'd need to talk to @JoshuaWise who is the only admin for this project. |
Does it work if you re-run all workflows? Or would that cause npm publishing to fail when the version already exists? If manual re-running the workflows won't solve the issue, then there's no reason to have permissions to do that. If the manual route is the way we want to try, it may be possible to structure the workflows in a way that they can be re-run. Or, have a script to build and upload the prebuild from a local machine. If not, maybe using the npm retry action on flaky actions like Should I make a PR for a patch bump for the Linux arm build? |
Sure: but I don't have sufficient permissions to merge it. Also: @jlarmstrongiv @JoshuaWise if there's a way to do prebuilds, and only after that is successful, release the version to npm, that seems like it would be strictly better. Do we just flip the dependencies in the build.yml to make |
@mceachen Yup, and also add |
Ugh, so, now the windows electron v26 arm build is failing: https://github.com/WiseLibs/better-sqlite3/actions/runs/9490136109/job/26153067563#step:10:527
Do we skip Electron v26? Is there a way to run this GHA job on a box we can shell into to see what's up with line 24890? FWIW, this seems to be the most recent windows prebuild that passed: https://github.com/WiseLibs/better-sqlite3/actions/runs/8843277421/job/24283338714 Edit: neat, it’ll be fixed in the next release: #1192 (comment) |
I'm just lurking but I don't know if this releases issues on darwin-x64 is related to flakiness because there hasn't been a single darwin-x64 on 10.x or 11.x. Also hasn't worked since this PR landed. |
Thanks for the link--I expect the two PRs I merged for v11.1.1 should remedy this.
v11.1.1 is now released (retrying failed jobs worked!) |
No description provided.
The text was updated successfully, but these errors were encountered: