Skip to content

Conversation

stevenhorsman
Copy link
Member

@stevenhorsman stevenhorsman commented Apr 30, 2025

Update the runtime-rs workspace packages to
use workspace package versions where applicable
to centralise the config and reduce maintenance
when updating these

@katacontainersbot katacontainersbot added the size/huge Largest and most complex task (probably needs breaking into small pieces) label Apr 30, 2025
@stevenhorsman stevenhorsman force-pushed the runtime-rs-centralise-workspace-config branch from 42032ea to 5005cbc Compare April 30, 2025 10:35
@katacontainersbot katacontainersbot added size/large Task of significant size and removed size/huge Largest and most complex task (probably needs breaking into small pieces) labels Apr 30, 2025
@stevenhorsman stevenhorsman force-pushed the runtime-rs-centralise-workspace-config branch 2 times, most recently from 693dcdb to 0e9f995 Compare April 30, 2025 11:26
@katacontainersbot katacontainersbot added size/huge Largest and most complex task (probably needs breaking into small pieces) and removed size/large Task of significant size labels Apr 30, 2025
@stevenhorsman stevenhorsman force-pushed the runtime-rs-centralise-workspace-config branch 11 times, most recently from f49bc86 to 131ddbc Compare April 30, 2025 16:55
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>
@stevenhorsman stevenhorsman force-pushed the runtime-rs-centralise-workspace-config branch from 131ddbc to a126884 Compare April 30, 2025 18:40
@stevenhorsman stevenhorsman marked this pull request as ready for review May 1, 2025 07:58
@stevenhorsman
Copy link
Member Author

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.

@stevenhorsman stevenhorsman requested a review from RuoqingHe May 1, 2025 08:00
Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks @stevenhorsman!

@stevenhorsman stevenhorsman requested a review from pmores May 1, 2025 08:27
Copy link
Member

@RuoqingHe RuoqingHe left a 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

@fidencio fidencio merged commit bc66d75 into kata-containers:main May 1, 2025
325 of 347 checks passed
@stevenhorsman stevenhorsman deleted the runtime-rs-centralise-workspace-config branch May 1, 2025 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/huge Largest and most complex task (probably needs breaking into small pieces)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants