Skip to content

Windows Gitlab CI: Explicitly override image entrypoint#3887

Merged
scheibelp merged 1 commit intospack:developfrom
johnwparent:ci-win-gen-override-entrypoint
Mar 19, 2026
Merged

Windows Gitlab CI: Explicitly override image entrypoint#3887
scheibelp merged 1 commit intospack:developfrom
johnwparent:ci-win-gen-override-entrypoint

Conversation

@johnwparent
Copy link
Copy Markdown
Contributor

@johnwparent johnwparent commented Mar 18, 2026

By default, all docker image entrypoints are overridden for CI runs on Gitlab as configured by the default:image:override setting in the base gitlab-ci yaml. When overriding part of an image definition in a job definition, gitlab will throw away the non overriden component as well. Windows specifies a specific docker image, and in doing so, removes the entrypoint override. This is fine for our cloud hosted runners as kube overrides the entrypoint itself. Docker+machine runners however will respect the non overridden entrypoint.

Explicitly override the entrypoint in our Windows CI config to bring Windows stacks in line with the rest of Spack's stacks.

Adds an explicit entrypoint override to the CI config image override for Windows generate and build jobs.

Fix spun out from #3843

By default, all docker image entrypoints are overridden for CI runs on Gitlab as configured by the `default:image:override` setting in the base gitlab-ci yaml.
When overriding part of an image definition in a job definition, gitlab will throw away the non overriden component as well. Windows specifies a specific docker image, and in doing so, removes the entrypoint override.
This is fine for our cloud hosted runners as kube overrides the entrypoint itself.
Docker+machine runners however will respect the non overridden entrypoint.

Explicitly override the entrypoint in our Windows CI config to bring Windows stacks in line with the rest of Spack's stacks.

Adds an explicit entrypoint override to the CI config image override for Windows generate and build jobs.

Signed-off-by: John Parent <john.parent@kitware.com>
@spackbot-triage spackbot-triage bot added the ci Issues related to Continuous Integration label Mar 18, 2026
@johnwparent
Copy link
Copy Markdown
Contributor Author

@zackgalbreath @eugeneswalker I spun out the fix from the UO test branch and added the fix to the build jobs as well.

Copy link
Copy Markdown
Contributor

@eugeneswalker eugeneswalker left a comment

Choose a reason for hiding this comment

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

Thank you!

@scheibelp scheibelp enabled auto-merge (squash) March 18, 2026 15:17
@scheibelp scheibelp merged commit eb4ccc4 into spack:develop Mar 19, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Issues related to Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants