cri: add deprecation warning for auths

samuelkarp · samuelkarp · commit 689a1036dda3 · 2023-11-02T18:20:33.000-07:00
Signed-off-by: Samuel Karp <samuelkarp@google.com> (cherry picked from commit 35924bc) Signed-off-by: Samuel Karp <samuelkarp@google.com>
diff --git a/pkg/cri/config/config.go b/pkg/cri/config/config.go
@@ -505,7 +505,8 @@ func ValidatePluginConfig(ctx context.Context, c *PluginConfig) ([]deprecation.W
 			config.Auth = &auth
 			c.Registry.Configs[endpoint] = config
 		}
-		log.G(ctx).Warning("`auths` is deprecated, please use `configs` instead")
+		warnings = append(warnings, deprecation.CRIRegistryAuths)
+		log.G(ctx).Warning("`auths` is deprecated, please use `ImagePullSecrets` instead")
 	}
 
 	// Validation for stream_idle_timeout
diff --git a/pkg/cri/config/config_test.go b/pkg/cri/config/config_test.go
@@ -317,6 +317,7 @@ func TestValidateConfig(t *testing.T) {
 					},
 				},
 			},
+			warnings: []deprecation.Warning{deprecation.CRIRegistryAuths},
 		},
 		"invalid stream_idle_timeout": {
 			config: &PluginConfig{
diff --git a/pkg/deprecation/deprecation.go b/pkg/deprecation/deprecation.go
@@ -27,6 +27,8 @@ const (
 	GoPluginLibrary Warning = Prefix + "go-plugin-library"
 	// CRIRegistryMirrors is a warning for the use of the `mirrors` property
 	CRIRegistryMirrors Warning = Prefix + "cri-registry-mirrors"
+	// CRIRegistryAuths is a warning for the use of the `auths` property
+	CRIRegistryAuths Warning = Prefix + "cri-registry-auths"
 )
 
 var messages = map[Warning]string{
@@ -35,6 +37,8 @@ var messages = map[Warning]string{
 	GoPluginLibrary: "Dynamically-linked Go plugins as containerd runtimes will be deprecated in containerd v2.0 and removed in containerd v2.1.",
 	CRIRegistryMirrors: "The `mirrors` property of `[plugins.\"io.containerd.grpc.v1.cri\".registry]` is deprecated since containerd v1.5 and will be removed in containerd v2.0." +
 		"Use `config_path` instead.",
+	CRIRegistryAuths: "The `auths` property of `[plugins.\"io.containerd.grpc.v1.cri\".registry]` is deprecated since containerd v1.3 and will be removed in containerd v2.0." +
+		"Use `ImagePullSecrets` instead.",
 }
 
 // Valid checks whether a given Warning is valid

Original file line number	Diff line number	Diff line change
`@@ -505,7 +505,8 @@ func ValidatePluginConfig(ctx context.Context, c *PluginConfig) ([]deprecation.W`
`505`	`505`	`config.Auth = &auth`
`506`	`506`	`c.Registry.Configs[endpoint] = config`
`507`	`507`	`}`
`508`		- log.G(ctx).Warning("`auths` is deprecated, please use `configs` instead")
	`508`	`+ warnings = append(warnings, deprecation.CRIRegistryAuths)`
	`509`	+ log.G(ctx).Warning("`auths` is deprecated, please use `ImagePullSecrets` instead")
`509`	`510`	`}`
`510`	`511`
`511`	`512`	`// Validation for stream_idle_timeout`