-
Notifications
You must be signed in to change notification settings - Fork 977
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
Bump gvisor to v0.0.0-20240313225113-67a078058255, fix service #1093
Conversation
Thanks for the contribution! Unfortunately we can't verify the commit author(s): Jack Doan <j***@r***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated. |
Thanks for the contribution! Before we can merge this, we need @JackDoan to sign the Salesforce Inc. Contributor License Agreement. |
Thanks! I might need to combine these changes with #981 to get the tests to pass for merging. Looking into it now. (the tests seem to fail with |
merging your branch into #981 (so your branch + go1.22) still seems to be getting some weird failures:
|
oh I see, Github Actions is still selecting the wrong go version. I thought we had it configured to use the version in go.mod. The other issue is that we need to do |
I have the tests passing in #981 by merging in your changes and making a few more fixes. I'll make sure to credit you in the merge commit there. Thanks! |
Closing as superseded by #981. Thank you! |
gvisor and the service example didn't build for me on go 1.22, this fixes that.