Skip to content

fix vfkit socket disposal#486

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
lstocchi:disposeVfKitSocket
Mar 12, 2025
Merged

fix vfkit socket disposal#486
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
lstocchi:disposeVfKitSocket

Conversation

@lstocchi
Copy link
Copy Markdown
Collaborator

when disposing the vfkit socket, gvproxy fails at removing it bc it tries to remove the unixgram uri (e.g. unixgram:///var/folders/n4/n5hyrstd2739lcy9903jn8f40000gn/T/podman/podman-gvproxy.sock). This patch fixes it by extracting the path from the uri.

@lstocchi lstocchi requested a review from cfergeau March 11, 2025 17:10
when disposing the vfkit socket, gvproxy fails at removing it bc it tries to remove the unixgram uri (e.g. unixgram:///var/folders/n4/n5hyrstd2739lcy9903jn8f40000gn/T/podman/podman-gvproxy.sock).
This patch fixes it by extracting the path from the uri.

Signed-off-by: Luca Stocchi <lstocchi@redhat.com>
@lstocchi lstocchi force-pushed the disposeVfKitSocket branch from cfd2d76 to 4bbb832 Compare March 11, 2025 17:25
@cfergeau
Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Mar 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfergeau, lstocchi

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

The pull request process is described here

Details 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-bot openshift-merge-bot bot merged commit ea44bed into containers:main Mar 12, 2025
22 checks passed
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.

2 participants