Merge pull request #10656 from djdongjin/remove-cri-SandboxInfo-RuntimeHandler

Remove deprecated cri SandboxInfo RuntimeHandler
This commit is contained in:
Akihiro Suda 2024-10-17 05:23:12 +00:00 committed by GitHub
commit 6c386c343d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 13 additions and 19 deletions

View File

@ -63,7 +63,6 @@ func toCRISandboxInfo(ctx context.Context, sb *types.PodSandbox) (map[string]str
si := &critypes.SandboxInfo{
Pid: sb.Status.Get().Pid,
Config: sb.Metadata.Config,
RuntimeHandler: sb.Metadata.RuntimeHandler,
CNIResult: sb.Metadata.CNIResult,
Metadata: &sb.Metadata,
}

View File

@ -148,7 +148,6 @@ func toDeletedCRISandboxInfo(sandbox sandboxstore.Sandbox) (map[string]string, e
si := &types.SandboxInfo{
Pid: sandbox.Status.Get().Pid,
Config: sandbox.Config,
RuntimeHandler: sandbox.RuntimeHandler,
CNIResult: sandbox.CNIResult,
}

View File

@ -33,10 +33,6 @@ type SandboxInfo struct {
Image string `json:"image"`
SnapshotKey string `json:"snapshotKey"`
Snapshotter string `json:"snapshotter"`
// Note: a new field `RuntimeHandler` has been added into the CRI PodSandboxStatus struct, and
// should be set. This `RuntimeHandler` field will be deprecated after containerd 1.3 (tracked
// in https://github.com/containerd/cri/issues/1064).
RuntimeHandler string `json:"runtimeHandler"` // see the Note above
RuntimeType string `json:"runtimeType"`
RuntimeOptions interface{} `json:"runtimeOptions"`
Config *runtime.PodSandboxConfig `json:"config"`