Closed saschagrunert closed 1 month ago
Thank you for fixing this! Much appreciated!
/approve /lgtm
@kwilczynski: changing LGTM is restricted to collaborators
@kubernetes-sigs/cri-tools-maintainers PTAL
Still looks good!
/approve /lgtm
@kwilczynski: changing LGTM is restricted to collaborators
@kubernetes-sigs/cri-tools-maintainers PTAL
/approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kwilczynski, saschagrunert
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What type of PR is this?
/kind bug
What this PR does / why we need it:
containerd on Windows may not escape the return message which may result in invalid JSON in crictl info.
Message from containerd: cni config load failed: no network config found in C:\Program Files \containerd\cni\conf: cni plugin
not initialized: failed to load cni config
Cherry-picked: 88df400df7b1b65dbd9d6fbab447a6b6da5177ad
Which issue(s) this PR fixes:
Fixes https://github.com/kubernetes-sigs/cri-tools/issues/1426
Special notes for your reviewer:
Taking over the reverts of https://github.com/kubernetes-sigs/cri-tools/pull/1429 and https://github.com/kubernetes-sigs/cri-tools/pull/1446 to finally fix that issue.
Does this PR introduce a user-facing change?