-
Notifications
You must be signed in to change notification settings - Fork 166
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
Label update and consolidation of CITGM jobs #3986
Comments
@StefanStojanovic I am not sure about Windows. Since in the regular CI, the build from source is never done on arm64 hardware, I set "fake" labels on the |
Nobuild pipeline: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-pipeline/265/console ✅ |
https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/3525/nodes=win-vs2022-arm64/console is doing an x64 build. Those Windows labels are difficult to get right... |
Hey, @targos, yes, you are correct, we are cross-compiling Node.js for ARM64, so the Additionally, regardless of the label (
So, we'd need to change the job configuration to take the label value in consideration on Windows. |
Is this going to change soon? I have no idea how to setup a cross-compile-then-run-on-arm-hardware config here |
Not very likely. I'm not 100% sure as I haven't tried it in some time, but I think ARM64 native compilation was possible (at least at some point). If that is still the case, we can use ARM64 machines to compile and run CITGM. I'm OOF until Jan 13 and I'll investigate it after I'm back. |
FYI @nodejs/citgm, @nodejs/releasers, I made the following changes to the Jenkins CITGM jobs, hopefully to make them more reliable:
fedora-last-latest
, and replacedrhel8
withrhel9
.arm64
binaries for labels that end with-arm64
. This should make the arm64 Windows tests faster (and cover arm64 instructions too) and fix the errors that we've been having on macOSalpine-latest-x64
, which I only added tocitgm-smoker
because we don't have official binaries to download for the other jobs.Updated jobs (you can see the new set of labels on each of those pages):
The text was updated successfully, but these errors were encountered: