Skip to content

CI: GnuTests: Split native and selinux tests to 2 different jobs#8388

Merged
sylvestre merged 8 commits intouutils:mainfrom
drinkcat:gnutests-ci-split
Jul 27, 2025
Merged

CI: GnuTests: Split native and selinux tests to 2 different jobs#8388
sylvestre merged 8 commits intouutils:mainfrom
drinkcat:gnutests-ci-split

Conversation

@drinkcat
Copy link
Collaborator

GnuTests is the slowest job in CI right now, we can split it in 2 mostly equal parts to make it much faster (~30' vs 1h).

While at it, I tried to clean the job a bit, and simplify it.

--- Log is a bit of a mess, we could just squash, hard to make it into clean individual commits

.github/workflows/GnuTests.yml: Fix aggregate job

.github/workflows/GnuTests.yml: More misc cleanup

In particular, SELinux flow is simplified, and fixed.

Also fix artifact download name.

.github/workflows/GnuTests.yml: Make selinux and native flow identical

Also add some headers to make things easier to follow.

.github/workflows/GnuTests.yml: Improve aggregate

Variables don't always help make things clearer.

.github/workflows/GnuTests.yml: Fix log uploading

This... didn't work, even in the previous version.

(found by Gemini...)

.github/workflows/GnuTests.yml: Create a new summary job

That aggregates between native and selinux results.

.github/workflows/GnuTests.yml: Move all Selinux steps to a separate job

.github/workflows/GnuTests.yml: Move variables to env

Will make it easier to split the work into 2 jobs.

drinkcat added 8 commits July 25, 2025 00:39
Will make it easier to split the work into 2 jobs.
That aggregates between native and selinux results.
This... didn't work, even in the previous version.

(found by Gemini...)
Variables don't always help make things clearer.
Also add some headers to make things easier to follow.
In particular, SELinux flow is simplified, and fixed.

Also fix artifact download name.
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)

@drinkcat drinkcat marked this pull request as ready for review July 25, 2025 00:19
@sylvestre sylvestre merged commit b4c6ad1 into uutils:main Jul 27, 2025
78 checks passed
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.

2 participants