Skip to content

Add matrix tests for dtable-server implementations - #82

Draft
simonhammes wants to merge 4 commits into
v7.0from
dtable-server-matrix-tests
Draft

Add matrix tests for dtable-server implementations#82
simonhammes wants to merge 4 commits into
v7.0from
dtable-server-matrix-tests

Conversation

@simonhammes

@simonhammes simonhammes commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

End goal: Have matrix tests for both implementations with 0 regressions :)

  • Fix version-compare workflow (CI)

test:
runs-on: ubuntu-latest
name: "test (dtable-server-${{ matrix.implementation }})"
runs-on: ubuntu-26.04

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I updated the runner images from ubuntu-latest (which pointed to 24.04 at the time of writing) to ubuntu-26.04 to work around the following race condition that happened in CI:

Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/runner/work/openapi/openapi/version-compare/seatable-license.txt" to rootfs at "/shared/seatable/seatable-license.txt": create mountpoint for /shared/seatable/seatable-license.txt mount: make mountpoint "/shared/seatable/seatable-license.txt": openat seatable-license.txt: file exists

This might have been caused by a bug in runc (opencontainers/runc#4543) that was patched in 2024 (opencontainers/runc#4549)

Comment thread version-compare/config/seatable-nginx.conf Outdated
@simonhammes
simonhammes force-pushed the dtable-server-matrix-tests branch from 0af7105 to 260710e Compare September 2, 2026 09:47
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.

1 participant