-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add gvisor presubmit test #4984
base: master
Are you sure you want to change the base?
Conversation
6ebf2bc
to
035d902
Compare
I wonder if we'd better hard-code the gVisor release number to make test results more reproducible. |
@a-nogikh I was thinking about that too. I think it is better to start with the latest one and then we will be able to change that in case of any problem. I will introduce the same test in the gvisor presubmit suite, so I don't expect any issues here. |
I think it's better to make the test as realistic as possible, or it will miss bugs. syz-ci should provide kernel_obj dir, let's do the same here. |
035d902
to
91f1917
Compare
Now the test fails with:
|
04cc86f
to
167c274
Compare
Download the latest gvisor release and run the syz-manager smoke-test suite. Signed-off-by: Andrei Vagin <avagin@google.com>
86438fd
to
90f79f7
Compare
I fixed the test. I checked it with this patch:
|
@dvyukov Can you be a bit more detailed about this kernel_obj dir? What should it contain? |
Any dir that contains the runsc binary renamed to "vmlinux": |
No description provided.