Repository navigation
build: remove GitHub Actions for tarballs? #34123
Description
Activity
To be clear: I'd rather keep the action and get building fixed/reliable. But if we're not going to do that, I'm not excited about having basically permanently failing actions.
If they're generally passing on Linux perhaps we could start by removing the macOS/Windows tests rather than remove the whole thing?
* The output is annoying-at-best to work with efficiently. * It seems that no one is looking in to why they're failing. * It seems that code is landing anyway, despite the failures.I feel these three points are general to our GitHub Actions workflows in general and not specific to the tarball workflow.
* They fail on at least one of Windows or macOS on almost every run on PRs against master.I've sampled a few of the Windows failures and they're all failing
parallel/test-vm-timeout-escape-promise-2. I've raised #34163.I'm still not sure just fixing the test failures will do any good, there are many runs of this action that are hanging/failing with no output (example: https://github.com/nodejs/node/runs/784411094?check_suite_focus=true).
This seems as good a place to note this -- until we come up with a better way of representing test results in the actions interface you may find it easier to search through the raw log:
Reacted by Rich Trott and mary marchiniI'd like to prod this again... I really think we should consider removing this action and perhaps re-implementing something in our primary CI
Reacted by Michaël ZassoI'm still +1 on removing it from Actions
Going to go ahead and close this as we remove windows+ osx from the action, which were AFAIK the platforms that were failing
- added a commit that references this issue
on Jul 27, 2020
I could be wrong but it seems to me:
The intention was good, but the end result is not that people are making sure the tarballs build. It's that we're ignoring test failures.
Correction welcome! If these are resulting in activity I'm not seeing, then great! But if not...
Let's disable/remove them?