optimize func ConvertToRegistryAuths

Signed-off-by: liheng.zms <[email protected]>

pkg/daemon/criruntime/imageruntime/helpers_test.go

@@ -86,7 +86,7 @@ func TestMatchRegistryAuths(t *testing.T) {
 				}
 				return []v1.Secret{demo}
 			},
-			Expect: 2,
+			Expect: 1,
 		},
 		{
 			name:  "test5",
@@ -102,6 +102,20 @@ func TestMatchRegistryAuths(t *testing.T) {
 			},
 			Expect: 0,
 		},
+		{
+			name:  "test6",
+			Image: "registry.private.com/app/echoserver:v1",
+			GetSecrets: func() []v1.Secret {
+				demo := v1.Secret{
+					Data: map[string][]byte{
+						"data": []byte("not docker config json"),
+					},
+					Type: v1.SecretTypeOpaque,
+				}
+				return []v1.Secret{demo}
+			},
+			Expect: 0,
+		},
 	}
 	for _, cs := range cases {
 		t.Run(cs.name, func(t *testing.T) {

pkg/util/secret/parse.go

@@ -6,7 +6,6 @@ import (
 
 	daemonutil "github.com/openkruise/kruise/pkg/daemon/util"
 	corev1 "k8s.io/api/core/v1"
-	"k8s.io/kubernetes/pkg/credentialprovider"
 	credentialprovidersecrets "k8s.io/kubernetes/pkg/credentialprovider/secrets"
 )
 
@@ -21,20 +20,22 @@ func AuthInfos(ctx context.Context, imageName, tag string, pullSecrets []corev1.
 	return authInfos
 }
 
-var (
-	keyring = credentialprovider.NewDockerKeyring()
-)
-
 func ConvertToRegistryAuths(pullSecrets []corev1.Secret, repo string) (infos []daemonutil.AuthInfo, err error) {
-	keyring, err := credentialprovidersecrets.MakeDockerKeyring(pullSecrets, keyring)
+	keyring, err := credentialprovidersecrets.MakeDockerKeyring(pullSecrets, nil)
 	if err != nil {
 		return nil, err
 	}
+	if keyring == nil {
+		return nil, nil
+	}
 	creds, withCredentials := keyring.Lookup(repo)
 	if !withCredentials {
 		return nil, nil
 	}
 	for _, c := range creds {
+		if c.Username == "" {
+			continue
+		}
 		infos = append(infos, daemonutil.AuthInfo{
 			Username: c.Username,
 			Password: c.Password,