Skip to content

Fix Run the tests workflow failure by removing brittle host setup steps - #2

Merged
MarcelRB merged 1 commit into
mainfrom
copilot/fix-run-the-tests-job
Oct 9, 2026
Merged

MarcelRB merged 1 commit into
mainfrom
copilot/fix-run-the-tests-job

Conversation

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

The Run the tests job was failing before test execution due to swap-file creation on the GitHub runner (fallocate ... /swapfile → Text file busy). This PR removes fragile pre-test runner mutations so the job reaches Docker-based test execution reliably.

  • Root-cause removal in run-tests job

    • Removed Maximize build disk space step (jlumbroso/free-disk-space@main).
    • Removed Setup Swap step that created/enabled /swapfile.
  • Preserved test execution path

    • Kept checkout, GHCR auth, image pull, and unittest execution unchanged.
    • Retained container memory constraints used by the test run.
  • Workflow clarity

    • Normalized the remaining test-step comment to match current behavior.
- name: Run tests in Docker
  run: |
    docker pull ${{ needs.get-docker-meta-data.outputs.tags }}
    docker run --rm \
      --memory="6g" \
      --memory-swap="12g" \
      -v ${{ github.workspace }}:/mnt \
      ${{ needs.get-docker-meta-data.outputs.tags }} \
      python3 -m unittest discover /mnt/tests "*_test.py"

Co-authored-by: MarcelRB <56345142+MarcelRB@users.noreply.github.com>
@MarcelRB
MarcelRB marked this pull request as ready for review October 9, 2026 12:49
@MarcelRB
MarcelRB merged commit 4ce16dd into main Oct 9, 2026
3 of 5 checks passed
@MarcelRB
MarcelRB deleted the copilot/fix-run-the-tests-job branch October 9, 2026 13:37
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