Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration Tests CI #40

Closed
wants to merge 41 commits into from
Closed

Conversation

ThoFrank
Copy link
Contributor

@ThoFrank ThoFrank commented Dec 20, 2024

GH runners can't create a vcan interface. This makes it imposible to directly run the integration tests.

As a workaround this PR uses the NixOS testing library to spawn a VM (nested in the GitHub provided vm) in which we can enable socketcan and thus run the tests.

A MacOS based runner is required because the referenceApp freezes during on a vm nested on a linux host. @frankdarcyacn is investigating why, but this is really hard to debug

TODO: Clean up the git history

@ThoFrank ThoFrank force-pushed the nixos_test branch 2 times, most recently from e0ddec8 to 552dd18 Compare December 20, 2024 13:54
@ThoFrank ThoFrank force-pushed the nixos_test branch 3 times, most recently from e3b94c6 to f2cff47 Compare January 7, 2025 17:02
Change-Id: I800ed8b9c4da2455212160fd62fdcc1e409b3a06
Change-Id: I03ea5777ba61b9b5fbe7525b16f35fe6e838f558
Change-Id: I720ba70583abf20058a32a395864197983abe234
Change-Id: I14f093145c6084f94d8170904db5585cfc6af375
Change-Id: I26a2e85d5b4a090354fd9ac58e210f0d26e4b80e
Change-Id: I6de21704b70c6dee8b05104ed3bef779d9367f6e
Change-Id: Ic5c484e92b03d69e35504af88f528d7c977c2506
Change-Id: Iff471e6ae9d9fb186b9f80f9b95979616a95a562
Change-Id: Ic7309260dd56b02108d27d150415af89012461ea
Change-Id: Ie2d2087a07cbb6f601c6ea53de6df253850d176b
Change-Id: Id4ec187543e73997eba7cd374d7db2c73a206c6f
Change-Id: Ic50d21bf9cc43445d648f0ade1474ae6c1bdea3f
Change-Id: I38fc1c6f42301b97249d0346e76ed730720077d5
Change-Id: I925ceccc004a6047b073d6e45c80af12b0a14d0c
This reverts commit a076b1d.

Change-Id: I83b568dd72ca4801af105c9a58f37110c59bc411
Change-Id: I35a14819512eb1614a6c993f8f8113aea7dd4e98
Change-Id: I585b96bca403c6af8c2f986e344219c472880d7c
Change-Id: I254cfae1da638de3fb2a67eb5ca15d83cff22d24
Change-Id: I813b1639402764c5bd33737e31ad047c2f5eebc6
Flake lock file updates:

• Updated input 'nixpkgs':
    'path:/nix/store/w66xaybnxjf5zhxapxs3pd3zp7gxk9dw-source?lastModified=0&narHash=sha256-eq51hyiaIwtWo19fPEeE0Zr2s83DYMKJoukNLgGGpek%3D' (1970-01-01)
  → 'github:NixOS/nixpkgs/8c9fd3e564728e90829ee7dbac6edc972971cd0f?narHash=sha256-CHd7yhaDigUuJyDeX0SADbTM9FXfiWaeNyY34FL1wQU%3D' (2025-01-07)

Change-Id: I6a1cc43b084e82d38bd9df0c382bbc54edd892c8
Change-Id: I10b8a60efddfcca87885298b200e6823a81f8ef7
Change-Id: I7cda06f822511dc32ec9ab0dd0fb42423a913e7c
@ThoFrank
Copy link
Contributor Author

This PR was created under the false assumption, that vcan isn't supported on the GitHub runners. #53 shows that it is possible. Thus this PR is no longer needed.

@ThoFrank ThoFrank closed this Jan 23, 2025
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