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

tests/vfio: Extend test to cover cold-plug #7971

Closed
wants to merge 3 commits into from

Conversation

jepio
Copy link
Member

@jepio jepio commented Sep 15, 2023

This is a port of:

It allows testing cold-plug-vfio and makes the test more parametrized which makes it easier to test GPUs.

@katacontainersbot katacontainersbot added the size/large Task of significant size label Sep 15, 2023
zvonkok and others added 3 commits September 15, 2023 14:50
Fix default port assignment with an explicit value in the TOML
It was implicit before and now the runtime will bail out if we did not set it explicitly.
Update the configuration and set it. This will make the intent more clear.

Fixes: kata-containers#5726
Signed-off-by: Zvonko Kaiser <[email protected]>
Signed-off-by: Jeremi Piotrowski <[email protected]>
These are not used by the test.

Signed-off-by: Jeremi Piotrowski <[email protected]>
This is not implemented (yet) but should be in the config file for overriding
in the test case.

Signed-off-by: Jeremi Piotrowski <[email protected]>
@zvonkok
Copy link
Contributor

zvonkok commented Sep 28, 2023

Can you add

Co-authored-by: Zvonko Kaiser [email protected] 

to your commit message, please? Thanks.

@zvonkok
Copy link
Contributor

zvonkok commented Sep 28, 2023

This is cool; CLH getting cold-plug support. I will add a runner soon to test this with GPUs.

@zvonkok
Copy link
Contributor

zvonkok commented Sep 28, 2023

@jepio should we close the other one if favour of this one?

Copy link

This PR has been opened without with no activity for 180 days. Comment on the issue otherwise it will be closed in 7 days

@github-actions github-actions bot added the stale Issue or PR was not updated in a timely fashion label Mar 27, 2024
@github-actions github-actions bot closed this Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/large Task of significant size stale Issue or PR was not updated in a timely fashion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants