pi-hole / FTL

The Pi-hole FTL engine
https://pi-hole.net
Other
1.34k stars 187 forks source link

Slightly simplify the CI tests #1924

Closed DL6ER closed 3 months ago

DL6ER commented 3 months ago

What does this implement/fix?

No functional change

This PR slightly simplifies the CI tests causing the RISCV64 builds to fail quite frequently. The reason for this is that a few character get lost on their way through the pipe between the testing fork and the controlling process. So far, I do not know why this is happening and first attempts to fix this have failed. When I can find more time to get a proper fix, we will restore this. Until then, this PR ensures we do not have to restart the CI tests a dozen times until they finally succeed by pure luck (because said bytes do not get lost).

This doesn't happen on any other architecture so it may even be a problem with our qemu emulation of the processor screwing something up.


Related issue or feature (if applicable): N/A

Pull request in docs with documentation (if applicable): N/A


By submitting this pull request, I confirm the following:

  1. I have read and understood the contributors guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented my proposed changes within the code.
  3. I am willing to help maintain this change if there are issues with it later.
  4. It is compatible with the EUPL 1.2 license
  5. I have squashed any insignificant commits. (git rebase)

Checklist: