-
Notifications
You must be signed in to change notification settings - Fork 120
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
Update ci kernel #125
Comments
Hello @quininer! The project seems like a good candidate for contributing. We are starting to use this project, and I am looking for ways we can help. Would it be possible for you to provide more information about the build servers? I would like to know if it is possible to make a Replica.one image for this purpose. Thanks, Luka cc @jpetrina |
Thanks for your interest in this. Currently ci uses cross + qemu to run the specified image. It's a bit complicated because it uses ssh to communicate with the host, which introduces a lot of dependencies. The tests for io-uring itself have no external dependencies, I think image just needs a sufficiently new kernel, glibc, shell to work. qemu is a bit too complicated for me and I've been thinking about migrating to firecracker or some other lighter vm. |
Hello @quininer! I have a couple of questions regarding this issue: Is this the image with the outdated kernel you are referring to? Thanks, Alen |
Yes, it is. |
@quininer would it be okay with you if we managed the self-hosting? The only thing we would need is to setup the WebHooks to GitHub project so the self-hosted instance gets notified on pull requests & new commits. We already have a similar setup and that would be easiest for us I think. |
@lperkov This sounds like the fastest solution, but I still want a vm solution that can run anywhere. |
Is there an update on this? Am I correct in thinking the current ci is using a GitHub ci runner to run cargo cross and cargo cross is using qemu and qemu is pointed to a private linux image. When I tried this on my ubuntu machine, with docker, cross test --target aarch64-unknown-linux-gnu |
Hello, Could you confirm if they satisfy your needs? |
This looks great, but I don't have energy to do it right now. It would be great if someone would try it. |
@quininer |
Is there a reason we can't just use a more recent version of Ubuntu and remove the VM dependency? Kinda like https://github.com/tokio-rs/tokio-uring/blob/master/.github/workflows/ci.yml#L19 |
@rushilmehra We can use host kernel, vm kernel version is outdated anyway. but even with host kernel, the latest features are still not available for testing on ci. |
The kernel version currently running on ci is too old, preventing many features from being stable.
I fork cross image at the time to update the kernel, but it's too old now.
We need a better way to run tests with newer kernels.
The text was updated successfully, but these errors were encountered: