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

Update gvproxy to v0.8.0 and disable ssh port forwarding on wsl #24394

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

vyasgun
Copy link
Member

@vyasgun vyasgun commented Oct 28, 2024

This fixes #20327.
This change will pass -1 to the -ssh-port flag of gvproxy to disable port forwarding on wsl which will prevent port conflict with crc.

Does this PR introduce a user-facing change?

No

None

Fixes: containers#24070
This change disables ssh port forwarding on wsl to prevent port conflict with crc

Signed-off-by: Gunjan Vyas <[email protected]>
@baude
Copy link
Member

baude commented Oct 28, 2024

is it possible to add a test to this PR?

@baude
Copy link
Member

baude commented Oct 28, 2024

/approve

Copy link
Contributor

openshift-ci bot commented Oct 28, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude, vyasgun

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

The pull request process is described 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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 28, 2024
@baude
Copy link
Member

baude commented Oct 28, 2024

lgtm

@cfergeau
Copy link
Contributor

is it possible to add a test to this PR?

Are there some 'high level' tests which start podman-machine/gvproxy on Windows/wsl? We could add a test there to check that on the host, nothing listens on port 2222

@l0rd
Copy link
Member

l0rd commented Oct 28, 2024

These are some machine e2e SSH tests here. The README.md file has instructions to run them (on windows too).

@lsm5
Copy link
Member

lsm5 commented Oct 29, 2024

/packit retest-failed

@baude
Copy link
Member

baude commented Oct 29, 2024

Are there some 'high level' tests which start podman-machine/gvproxy on Windows/wsl? We could add a test there to check that on the host, nothing listens on port 2222

yes ofc there are!

@vyasgun
Copy link
Member Author

vyasgun commented Oct 29, 2024

Thanks for the reviews! I have added a small modification to the windows init test to ensure port 2222 is open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. machine release-note-none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port conflict of WSL2 with CRC for SSH (usermode networking)
5 participants