Skip to content
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

Set ci-runner concretizer and view configuration in higher scope #56

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

CodeGat
Copy link
Member

@CodeGat CodeGat commented Mar 12, 2025

References ACCESS-NRI/build-ci#195

Background

It seems that build-ci spack manifests should almost always use concretizer.unify: when_possible as the concretization will fail when installing multiple of the same model component packages. Instead of having to create a schema on build-ci spack manifests, we should set this configuration higher up than the environment level (namely, here!).

This only solves half the problem, though. If a view can't be created due to having the same prefix (say, installing two mom5s with different compilers) the installation will fail. Hence we either need to have no view (which needs to be tested), or we need to set a custom spack.views.default.projections section in the spack manifest. Since it's not a first order configuration file (like config.yaml or concretizer.yaml, we need to inject this file into the environment. This is why it's called injected.view.yaml.

The PR

  • Update ci-runner concretizer to contain unify: when_possible
  • Create injected.view.yaml snippet for ci-runner spack manifests

@CodeGat CodeGat added the enhancement New feature or request label Mar 12, 2025
@CodeGat CodeGat self-assigned this Mar 12, 2025
@CodeGat
Copy link
Member Author

CodeGat commented Mar 12, 2025

Going to test out using the above spack configuration with the compose file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Parked 🅿️
Development

Successfully merging this pull request may close these issues.

1 participant