-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Gitlab CI: Windows Configs #43967
Gitlab CI: Windows Configs #43967
Conversation
share/spack/gitlab/cloud_pipelines/scripts/common/expand_vars.py
Outdated
Show resolved
Hide resolved
share/spack/gitlab/cloud_pipelines/stacks/build_systems/spack.yaml
Outdated
Show resolved
Hide resolved
`ruamel`'s `Emitter.write_line_break()` method takes an extra argument that we forgot to implement in our custom emitter.
0176ee7
to
43544f5
Compare
33d0303
to
d05ec1b
Compare
Excited to see this merged! I know we're still a long way off from getting all dependencies working, but I would be keen on porting our ML CI stacks to build on Windows someday. Let me know if there's anything I can do to help with the Python on Windows effort. |
Porting all the stacks is the endgame of Spack on Windows, so I'm happy to support any effort doing so. It's not on my radar for the next month or so, but if you have specific packages/stacks you'd like to see, I'd be more than happy to iterate over slack, feel free to DM me. |
Support for Gitlab CI on Windows is here!
This PR adds the config changes required to configure and execute Gitlab pipelines running Windows builds on Windows runners using the existing Gitlab CI infrastructure (and newly added Windows side infra).
Adds support for generating child pipelines dispatched to Windows runners
Refactors the relevant pre-scripts, scripts, and post scripts to be compatible with Windows
Adds Windows config section describing Windows jobs
Adds VTK as Windows build stack (to be expanded later).
DO NOT MERGE THIS without validating pre-reqs have been met with a member of the Gitlab CI team and/or @johnwparent, @scottwittenburg, @kwryankrattiger, or @mvandenburgh.
PR Prereqs:
Manual Checks: