-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Runtime rs centralise workspace config #11217
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
Runtime rs centralise workspace config #11217
Conversation
42032ea
to
5005cbc
Compare
693dcdb
to
0e9f995
Compare
f49bc86
to
131ddbc
Compare
As discussed in kata-containers#9538, with anyhow >=1.0.77 we have test failures due to backtrace behaviour changing, so set RUST_LIB_BACKTRACE=0, so that we only have backtrace on panics Signed-off-by: stevenhorsman <steven@uk.ibm.com>
Update the runtime-rs workspace packages to use workspace package versions where applicable to centralise the config and reduce maintenance when updating these Signed-off-by: stevenhorsman <steven@uk.ibm.com>
131ddbc
to
a126884
Compare
I'm happy to rebase this on #11131 if it's merged first, but I think this can be reviewed now as all the tests seem to be passing. The only package I couldn't unify was nix - we have three different versions as when I tried to just pick a single one we got a bunch of incompatibilities, so I've left them separated for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @stevenhorsman!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, we are one-step closer to a consistent workspace 😄, thanks @stevenhorsman
Update the runtime-rs workspace packages to
use workspace package versions where applicable
to centralise the config and reduce maintenance
when updating these