[release-1.29] fix: match tags issue in account search

pkg/provider/azure_storageaccount.go

@@ -840,17 +840,16 @@ func isTagsEqual(account storage.Account, accountOptions *AccountOptions) bool {
 		return true
 	}
 
-	for k, v := range account.Tags {
-		var value string
-		// nil and empty value should be regarded as equal
-		if v != nil {
-			value = *v
-		}
-		if accountOptions.Tags[k] != value {
+	if len(account.Tags) < len(accountOptions.Tags) {
+		return false
+	}
+
+	// ensure all tags in accountOptions are in account
+	for k, v := range accountOptions.Tags {
+		if pointer.StringDeref(account.Tags[k], "") != v {
 			return false
 		}
 	}
-
 	return true
 }
 

pkg/provider/azure_storageaccount_test.go

@@ -917,6 +917,34 @@ func TestIsTagsEqual(t *testing.T) {
 			},
 			expectedResult: false,
 		},
+		{
+			desc: "non-identitical tags with different keys",
+			account: storage.Account{
+				Tags: map[string]*string{
+					"key1": pointer.String("value2"),
+				},
+			},
+			accountOptions: &AccountOptions{
+				MatchTags: true,
+				Tags: map[string]string{
+					"key2": "value",
+				},
+			},
+			expectedResult: false,
+		},
+		{
+			desc: "account tags is empty",
+			account: storage.Account{
+				Tags: map[string]*string{},
+			},
+			accountOptions: &AccountOptions{
+				MatchTags: true,
+				Tags: map[string]string{
+					"key": "value",
+				},
+			},
+			expectedResult: false,
+		},
 	}
 
 	for _, test := range tests {