Add warning for CRIU config usage

ruiwen-zhao · ruiwen-zhao · commit 1fdefdd2242f · 2023-12-07T00:08:14.000Z
Signed-off-by: ruiwen-zhao &lt;ruiwen@google.com&gt;
diff --git a/pkg/cri/config/config.go b/pkg/cri/config/config.go
@@ -491,6 +491,11 @@ func ValidatePluginConfig(ctx context.Context, c *PluginConfig) ([]deprecation.W
 			r.SandboxMode = string(ModePodSandbox)
 			c.ContainerdConfig.Runtimes[k] = r
 		}
+
+		if p, ok := r.Options["CriuPath"].(string); ok && p != "" {
+			log.G(ctx).Warning("`CriuPath` is deprecated, please use a criu binary in $PATH instead.")
+			warnings = append(warnings, deprecation.CRICRIUPath)
+		}
 	}
 
 	useConfigPath := c.Registry.ConfigPath != ""
diff --git a/pkg/cri/config/config_test.go b/pkg/cri/config/config_test.go
@@ -477,6 +477,35 @@ func TestValidateConfig(t *testing.T) {
 			},
 			expectedErr: "invalid `drain_exec_sync_io_timeout`",
 		},
+		"deprecated CRIU path": {
+			config: &PluginConfig{
+				ContainerdConfig: ContainerdConfig{
+					DefaultRuntimeName: RuntimeDefault,
+					Runtimes: map[string]Runtime{
+						RuntimeDefault: {
+							SandboxMode: string(ModePodSandbox),
+							Options: map[string]interface{}{
+								"CriuPath": "/path/to/criu-binary",
+							},
+						},
+					},
+				},
+			},
+			expected: &PluginConfig{
+				ContainerdConfig: ContainerdConfig{
+					DefaultRuntimeName: RuntimeDefault,
+					Runtimes: map[string]Runtime{
+						RuntimeDefault: {
+							SandboxMode: string(ModePodSandbox),
+							Options: map[string]interface{}{
+								"CriuPath": "/path/to/criu-binary",
+							},
+						},
+					},
+				},
+			},
+			warnings: []deprecation.Warning{deprecation.CRICRIUPath},
+		},
 	} {
 		t.Run(desc, func(t *testing.T) {
 			w, err := ValidatePluginConfig(context.Background(), test.config)
diff --git a/pkg/deprecation/deprecation.go b/pkg/deprecation/deprecation.go
@@ -49,6 +49,8 @@ const (
 	RuntimeV1 Warning = Prefix + "runtime-v1"
 	// RuntimeRuncV1 is a warning for the io.containerd.runc.v1 runtime
 	RuntimeRuncV1 Warning = Prefix + "runtime-runc-v1"
+	// CRICRIUPath is a warning for the use of the `CriuPath` property
+	CRICRIUPath Warning = Prefix + "cri-criu-path"
 )
 
 var messages = map[Warning]string{
@@ -75,6 +77,8 @@ var messages = map[Warning]string{
 	AUFSSnapshotter: "The aufs snapshotter is deprecated since containerd v1.5 and removed in containerd v2.0. Use the overlay snapshotter instead.",
 	RuntimeV1:       "The `io.containerd.runtime.v1.linux` runtime is deprecated since containerd v1.4 and removed in containerd v2.0. Use the `io.containerd.runc.v2` runtime instead.",
 	RuntimeRuncV1:   "The `io.containerd.runc.v1` runtime is deprecated since containerd v1.4 and removed in containerd v2.0. Use the `io.containerd.runc.v2` runtime instead.",
+	CRICRIUPath: "The `CriuPath` property of `[plugins.\"io.containerd.grpc.v1.cri\".containerd.runtimes.*.options]` is deprecated since containerd v1.7 and will be removed in containerd v2.0. " +
+		"Use a criu binary in $PATH instead.",
 }
 
 // Valid checks whether a given Warning is valid

Original file line number	Diff line number	Diff line change
`@@ -491,6 +491,11 @@ func ValidatePluginConfig(ctx context.Context, c *PluginConfig) ([]deprecation.W`
`491`	`491`	`r.SandboxMode = string(ModePodSandbox)`
`492`	`492`	`c.ContainerdConfig.Runtimes[k] = r`
`493`	`493`	`}`
	`494`	`+`
	`495`	`+ if p, ok := r.Options["CriuPath"].(string); ok && p != "" {`
	`496`	+ log.G(ctx).Warning("`CriuPath` is deprecated, please use a criu binary in $PATH instead.")
	`497`	`+ warnings = append(warnings, deprecation.CRICRIUPath)`
	`498`	`+ }`
`494`	`499`	`}`
`495`	`500`
`496`	`501`	`useConfigPath := c.Registry.ConfigPath != ""`