Skip to content
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: remove SharedVersions field dead code #8812

Merged

Conversation

kalil-pelissier
Copy link
Contributor

SharedVersion fiel add a versiontable property that isn't supported by upstream QEMU. This is dead code since virtcontainers isn't setting SharedVersions to true.

Fixes: #7720

@katacontainersbot katacontainersbot added the size/small Small and simple task label Jan 12, 2024
@kalil-pelissier kalil-pelissier force-pushed the feature/issue-7720/drop-dead-code branch from acdfb40 to 3b84c4a Compare January 12, 2024 12:19
@kalil-pelissier kalil-pelissier marked this pull request as ready for review January 12, 2024 12:21
@kalil-pelissier
Copy link
Contributor Author

Hi @gkurz!
I think I'm done with this issue.

@gkurz
Copy link
Member

gkurz commented Jan 15, 2024

/test

Copy link
Member

@gkurz gkurz 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 @kalil-pelissier !

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 @kalil-pelissier!

@gkurz
Copy link
Member

gkurz commented Jan 22, 2024

@kalil-pelissier Please rebase on main to fix the build-checks not starting issue.

SharedVersion fiel add a versiontable property that isn't supported by upstream QEMU.
This is dead code since virtcontainers isn't setting SharedVersions to true.

Fixes: kata-containers#7720

Signed-off-by: Kvlil <kalil.pelissier@gmail.com>
@kalil-pelissier kalil-pelissier force-pushed the feature/issue-7720/drop-dead-code branch from 3b84c4a to a4b208a Compare January 22, 2024 12:18
@gkurz
Copy link
Member

gkurz commented Jan 22, 2024

/test

@gkurz
Copy link
Member

gkurz commented Jan 23, 2024

/retest-arm-unit

@gkurz gkurz merged commit 0f67a26 into kata-containers:main Jan 23, 2024
283 of 289 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

runtime: Unused SharedVersions field in the VhostUserDevice struct
4 participants