Skip to content

ci: add s390x big-endian workflow for hash function tests#1132

Closed
ijatinydv wants to merge 2 commits intofortran-lang:masterfrom
ijatinydv:ci-big-endian
Closed

ci: add s390x big-endian workflow for hash function tests#1132
ijatinydv wants to merge 2 commits intofortran-lang:masterfrom
ijatinydv:ci-big-endian

Conversation

@ijatinydv
Copy link
Contributor

Hi @jvdp1,

Following up on our discussion in #1130 , this PR adds a dedicated CI job to catch any future Big-Endian regressions.

It uses uraimo/run-on-arch-action to emulate an s390x (IBM Z) environment using QEMU. Since QEMU emulation is notoriously slow, I've heavily optimized the workflow so it doesn't bottleneck the rest of the project's CI or burn through Actions minutes:

  • Targeted builds: Instead of building all of stdlib, it uses cmake --build build --target test_hash_functions. This only compiles the hash modules and their direct dependencies, completely skipping heavy modules like linalg or stats.
  • Smart triggers: On PRs, this job will only run if src/hash/, test/hash_functions/, or the workflow file itself is modified. Otherwise, it just runs on merges to master.
  • Caching & Safety: Uses the GitHub token to cache the apt-get dependency layer, and adds --no-tests=error to ctest to ensure the CI forcefully fails if the test target gets renamed in the future.

I've tested this on my fork and it successfully catches the endianness issues we just fixed. Let me know if you'd like to tweak the scope or triggers at all!

@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.05%. Comparing base (ee33055) to head (8d27d63).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1132      +/-   ##
==========================================
+ Coverage   68.02%   68.05%   +0.03%     
==========================================
  Files         402      402              
  Lines       12839    12839              
  Branches     1383     1383              
==========================================
+ Hits         8734     8738       +4     
+ Misses       4105     4101       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ijatinydv ijatinydv closed this Mar 2, 2026
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.

1 participant