Replace FindExecutablePeer with FindHelperBinary when looking for gvproxy/win-sshproxy on WSL #27612
Conversation
Signed-off-by: Mario Loriedo <[email protected]>
The WSL machine start was using the function FindExecutablePeer that ignores user configuration (helper_binaries_dir). FindHelperBinary instead is used when starting the machine for the rest of the providers and honors user configuration. This commit requires containers/container-libs@4877783 Signed-off-by: Mario Loriedo <[email protected]>
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. 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. |
openshift-ci
bot
added
do-not-merge/work-in-progress
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: l0rd 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 |
openshift-ci
bot
added
the
approved
l0rd
changed the title
| return nil | ||
| } | ||
|
|
||
| exe, err := machine.FindExecutablePeer(gvProxy) |
There was a problem hiding this comment.
there is another user with win-sshproxy that hsould be repalced I think? Then we can delete this function as it would be no longer needed.
| tags.cncf.io/container-device-interface/specs-go v1.0.1-0.20251120202831-139ffec09210 // indirect | ||
| ) | ||
|
|
||
| replace go.podman.io/common => github.com/containers/container-libs/common v0.66.1-0.20251126113141-baac55b96f3f |
There was a problem hiding this comment.
you should run go get go.podman.io/storage@main && go get go.podman.io/image/v5@main && go get go.podman.io/common@main && make vendor to update the module without using a replace. Then we can merge this if it is updated properly.
The replace is only needed to point to different forks.
2 participants
The WSL machine start was using the function FindExecutablePeer that
ignores user configuration (helper_binaries_dir). FindHelperBinary
instead is used when starting the machine for the rest of the providers
and honors user configuration.
Checklist
Ensure you have completed the following checklist for your pull request to be reviewed:
commits. (
git commit -s). (If needed, usegit commit -s --amend). The author email must matchthe sign-off email address. See CONTRIBUTING.md
for more information.
Fixes: #00000in commit message (if applicable)make validatepr(format/lint checks)Noneif no user-facing changes)Does this PR introduce a user-facing change?