Fix Ubuntu CI builds 2024/12/22: turn off cylc variant in skylab-dev template, shorten env path for oneapi-ifx #1433
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes the CI build errors on the Ubuntu CI runners by:
cylc
variant in theskylab-dev
template. We wanted to keep one CI build forcylc
, even though the Skylab environment doesn't require it at the moment. But with the increasing complexity of Python dependencies, this is getting too hard. Note that Add cylc to neptune-python-env, enable in neptune-dev, disable in skylab-dev #1410 will allow us to buildcylc
again (because we can finally pinpy-cython
to a newer version). I haven't decided yet how to exactly do that in the context of CI and for releases of spack-stack, but likely it will be a separate environment (config template) altogether (deferred to Add cylc to neptune-python-env, enable in neptune-dev, disable in skylab-dev #1410).oneapi-ifx
, we useoneifx
as the name of the build environment (it makes zero difference, other than shortening the path by four characters).Testing
Applications affected
None
Systems affected
Ubuntu CI builds
Dependencies
n/a
Issue(s) addressed
None created - the CI build failures were first noted in #1432 (comment), therefore this PR should be merged before #1432 so that all CI tests pass.
Checklist
All dependency PRs/issues have been resolved and this PR can be merged.