Skip to content

Reworked Gitlab-CI#112

Open
pearzt wants to merge 2 commits intotudasc:develfrom
pearzt:ci/spack-based
Open

Reworked Gitlab-CI#112
pearzt wants to merge 2 commits intotudasc:develfrom
pearzt:ci/spack-based

Conversation

@pearzt
Copy link
Member

@pearzt pearzt commented Jan 29, 2026

With this PR, I propose a new, revised Gitlab-CI. These are the key changes:

  • Environment is managed using Spack
  • Designed to run on Lichtenberg's RHEL9 nodes
  • Extended matrix of tested GCC and LLVM versions
  • Improved CI runtimes, as dependencies (cxxopts, etc.) are managed by Spack and do not need to be rebuilt. Currently, a CI run takes around 6 minutes
  • Removed some CI jobs (stripped builds, mostly)
  • PGIS is no longer tested in the CI, as it is deprecated anyway
Screenshot_20260129_145716

@pearzt pearzt self-assigned this Jan 29, 2026
@TimHeldmann
Copy link
Member

I going through this PR together with @pearzt, I think it looks good to me.

There are two things that remain after this PR is merged:
One thing that still irks me, but should not be part of this PR, is how many different testing scripts we now have.
In the future I hope we could get some of these scripts either unified, or ported to ctest (or whatever we come up with).

Secondly, it seems there have been problems with compiling if SPD_LOG tries to compile with exceptions.
I have no idea why this would occur. I think this needs some further investigation.

Copy link
Member

@jplehr jplehr left a comment

Choose a reason for hiding this comment

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

I personally disagree with removing PGIS from the build. I understand that it is deprecated. The reason I disagree is because it is the one (only one?) tool that acts as a full end-to-end test for the library and package components.

I simply do not trust the rest of our testing enough.

If this is what we want to do, I won't get in the way, but I want to raise my concern.

- 'llvm@<LLVMVERSION> +clang +flang +utils %gcc@<GCCVERSION>'

# Build tools
- cmake
Copy link
Member

Choose a reason for hiding this comment

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

Can / should we no-matter-what put names in ' here?
My inner monk does not like that it is done sometimes and sometimes not.



# Stages
stages:
Copy link
Member

Choose a reason for hiding this comment

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

Do we actually still need the stages or can we remove them as we now simply use needs to define the DAG?

Helps with clear visualization?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants