-
Notifications
You must be signed in to change notification settings - Fork 680
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
CI: Test Landlock on various kernels with UML #1601
Comments
Why do we need to test against multiple versions of the kernel? Isn't landlock itself versioned? Aka we only need to test against version X and Y? |
Good point. After all, we don't test every kernel feature we use against multiple kernel versions. But since landlock is a new kernel feature, and some bug or regression can potentially affect consensus, it might be good to test different kernels. And the tooling is already provided for us. All we'd have to do is extend the CI job we already have. Even if we didn't use landlock, it seems good to test consensus-critical code (PVF execution) on multiple kernel versions, at least the most common ones according to telemetry. |
Testing with different versions of the kernel makes sense when relying on Landlock's features that may not be provided by the running kernel. It looks like you're only relying on The Rust library is tested with several kernel versions to make sure this best-effort mode works as expected and gives stability guarantees. You can either directly use https://github.com/landlock-lsm/landlock-test-tools or extract what makes sense for you. See landlock-lsm/rust-landlock#41 |
We already have a
test-linux-oldkernel
CI job for the purpose of testing the case where Landlock is not available in the kernel.This can be extended to more kernel versions using UML: https://github.com/landlock-lsm/landlock-test-tools.
Here is an example using the above repository: https://github.com/landlock-lsm/rust-landlock/pull/41/files
@paritytech/ci
The text was updated successfully, but these errors were encountered: