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

Test win-sshproxy with qemu VM #449

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lstocchi
Copy link
Collaborator

@lstocchi lstocchi commented Jan 8, 2025

it adds the logic to test win-sshproxy by using a qemu VM with fcos and see if the ssh tunneling between host/guest on the podman socket works as expected.

The qemu VM can also be reused for further extensive testing.

It is based on #427

Tests that are currently on the main branch only runs against a qemu VM. We have other use cases that needs to be tested like running against a vfkit VM.
This commit reorganizes the tests code a bit by moving the files that can be shared to support different implementation in their own folder.
The reasoning behind this is that every hypervisor should have its own beforeSuite func to download/run a specific VM image. By moving the utils files we can reuse the same code.

For the same reason the code targeting qemu is moved to the test-qemu folder. By doing so, we can run the tests within the test-qemu folder on the ubuntu workflow and, in future, when the nested virt will be enabled on github runners, the vfkit tests on macOS.

Signed-off-by: Luca Stocchi <[email protected]>
Adds a basic implementation for testing against a vfkit VM. Tests are based on the existing qemu version. It just changes the way the VM gets created/started.

Signed-off-by: Luca Stocchi <[email protected]>
it adds the logic to test win-sshproxy by using a qemu VM with fcos and see if
the ssh tunneling between host/guest on the podman socket works as
expected.

The qemu VM can also be reused for further extensive testing

Signed-off-by: lstocchi <[email protected]>
Copy link
Contributor

openshift-ci bot commented Jan 8, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lstocchi
Once this PR has been reviewed and has the lgtm label, please assign cfergeau for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot
Copy link
Collaborator

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

}
defer file.Close()

// using bufio.NewReader speeds up the decompressions -> https://github.com/ulikunitz/xz/issues/23
Copy link
Collaborator

Choose a reason for hiding this comment

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

even with the bufio reader, it is still a lot slower than the external tool? crc uses https://github.com/xi2/xz?tab=readme-ov-file for xz decompression, but I think it is also slower than native xz. And it hasn't seen any updates in years.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants