Skip to content

Conversation

yubiuser
Copy link
Member

@yubiuser yubiuser commented Jul 2, 2025

  1. Builds arm and riscv images on arm as it is faster
  2. Completely restructure the test suite
    a) so far, it was a convoluted mess from bash scripts building a python test image which used bash scripts to build the docker-image within itself and run the test in this docker-in-docker container
    b) now we use tox to build the docker-container and run the tests locally/CI
    c) we have support for all platforms docker supports
    d) much less code and less convoluted
    e) the pytest's conftest.py was cleaned up a lot
    f) one major cause for the random failures we've seen previously on the emulated platforms was due to using pytest-xdist and running the tests in parallel - this is removed now
    g) another issue I observed was a timing issues, esp. on RISCV. Adding some sleep commands before running the test commands fixed those.
    h) I added a test that compares the value of the CIPLATFORM var to the arch reported by FTL

Supersedes #1843

@PromoFaux

This comment was marked as resolved.

@yubiuser

This comment was marked as resolved.

@PromoFaux

This comment was marked as resolved.

@yubiuser

This comment was marked as resolved.

yubiuser added 2 commits July 2, 2025 22:22
Signed-off-by: yubiuser <github@yubiuser.dev>
Signed-off-by: yubiuser <github@yubiuser.dev>
@yubiuser
Copy link
Member Author

yubiuser commented Jul 2, 2025

Just a rebase, no functional change.

@PromoFaux
Copy link
Member

I need to change the required tests

@PromoFaux PromoFaux merged commit 29bde17 into development Jul 2, 2025
10 checks passed
@PromoFaux PromoFaux deleted the test_suite branch July 2, 2025 20:40
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