Skip to content

Conversation

@ChrisDryden
Copy link
Collaborator

This isn't the most elegant of fixes, but the reason that we see varying integration test numbers is that if we don't re-use the gnu build from the cache then before we run the tests we will regenerate the test files again which includes all of the files that we patched out and all of the factor tests. This removes all of the patches to the Makefile and also causes the test suite to take double the amount of time to run.

The three changes are: changing the factor total test count to 40 from 37 since this was update upstream, removing all of these tests from the tests/local.mk so that even if in the future we regress and rebuild the tests again it will not have those factor tests, then lastly touching Makefile.in Makefile after all of the modifications to the build files so that when we run make-check it wont rebuild the test files again.

I made a branch to stress test this to validate that it fixes the bug and it appears that it does, but at the same time the conditions don't match the main CI exactly so I'm not 100% confident that this fixes it completely.

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/chcon/chcon. tests/chcon/chcon is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/cp/no-ctx. tests/cp/no-ctx is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/shuf/shuf-reservoir (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/sort/sort-stale-thread-mem (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cp/cp-a-selinux is no longer failing!
Note: The gnu test tests/pr/bounded-memory is now being skipped but was previously passing.

@oech3
Copy link
Contributor

oech3 commented Feb 13, 2026

Just stop caching GNU coreutils build?

@ChrisDryden
Copy link
Collaborator Author

Might be a good option, tests/chcon/chcon. tests/chcon/chcon these failures are unexpected, when looking at the test failures they appear to be legitimate error message differences. Not sure whats happening here

@oech3
Copy link
Contributor

oech3 commented Feb 13, 2026

Actuall, cache is working only on forks and 2nd push. So caches are not useful.

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