Skip to content

Allow tests to run in parallel - #2296

Merged
matusmarhefka merged 15 commits into
OpenSCAP:mainfrom
Mab879:test_fixes
Aug 3, 2026
Merged

Allow tests to run in parallel#2296
matusmarhefka merged 15 commits into
OpenSCAP:mainfrom
Mab879:test_fixes

Conversation

@Mab879

@Mab879 Mab879 commented Dec 19, 2025

Copy link
Copy Markdown
Member

Significantly speed up the tests if ran with -j. See each commit for details. We have room for additional speed up if we can remove RESOURCE_LOCK groups.

Review hints

ctest -j$(nproc) a few times.

@Mab879
Mab879 marked this pull request as draft December 19, 2025 20:36
@Mab879
Mab879 marked this pull request as ready for review January 6, 2026 00:38
@sonarqubecloud

Copy link
Copy Markdown

@Mab879
Mab879 force-pushed the test_fixes branch 2 times, most recently from 610c79e to ec96b1a Compare April 23, 2026 17:47
@sonarqubecloud

Copy link
Copy Markdown

@Mab879

Mab879 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

@jan-cerny can I get a review on this?

@Mab879

Mab879 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Rebased on latest main.

@sonarqubecloud

Copy link
Copy Markdown

@matusmarhefka matusmarhefka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice improvement, before:

Total Test time (real) = 425.16 sec

After:

Total Test time (real) = 172.90 sec

Once we merge this, consider updating https://src.fedoraproject.org/tests/openscap/blob/main/f/Sanity/smoke-test/runtest.sh#_124 as well to benefit from tests parallelization too.

Comment thread tests/API/XCCDF/unittests/CMakeLists.txt Outdated
Comment thread tests/probes/rpm/rpminfo/CMakeLists.txt Outdated
Comment thread tests/probes/rpm/rpmverify/CMakeLists.txt Outdated
Comment thread tests/probes/rpm/rpmverifyfile/CMakeLists.txt Outdated
Comment thread tests/probes/rpm/rpmverifypackage/CMakeLists.txt Outdated
Comment thread tests/probes/yamlfilecontent/test_probes_yamlfilecontent_key.sh
Mab879 added 2 commits July 31, 2026 12:49
If you had "OpenSCAP" in your path to the OpenSCAP repo
it would cause issues
@sonarqubecloud

Copy link
Copy Markdown

@matusmarhefka
matusmarhefka merged commit 9f2c4a6 into OpenSCAP:main Aug 3, 2026
19 checks passed
@Mab879
Mab879 deleted the test_fixes branch August 3, 2026 12:35
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