Fix: Skip the non-exist namespaces when there are multiple mongo instance metrics to scrape

Signed-off-by: LinPr <[email protected]>

Author: LinPr

exporter/common.go

@@ -164,7 +164,7 @@ func unique(slice []string) []string {
 }
 
 func checkNamespacesForViews(ctx context.Context, client *mongo.Client, collections []string) ([]string, error) {
-	onlyCollectionsNamespaces, err := listAllCollections(ctx, client, nil, nil, true)
+	onlyCollectionsNamespaces, err := listAllCollections(ctx, client, collections, nil, true)
 	if err != nil {
 		return nil, err
 	}
@@ -183,7 +183,7 @@ func checkNamespacesForViews(ctx context.Context, client *mongo.Client, collecti
 		}
 
 		if _, ok := namespaces[collection]; !ok {
-			return nil, errors.Errorf("namespace %s is a view and cannot be used for collstats/indexstats", collection)
+			continue
 		}
 
 		filteredCollections = append(filteredCollections, collection)

exporter/common_test.go

@@ -199,8 +199,9 @@ func TestCheckNamespacesForViews(t *testing.T) {
 	defer cleanupDB(ctx, client)
 
 	t.Run("Views in provided collection list (should fail)", func(t *testing.T) {
-		_, err := checkNamespacesForViews(ctx, client, []string{"testdb01.col01", "testdb01.system.views", "testdb01.view01"})
-		assert.EqualError(t, err, "namespace testdb01.view01 is a view and cannot be used for collstats/indexstats")
+		filtered, err := checkNamespacesForViews(ctx, client, []string{"testdb01.col01", "testdb01.system.views", "testdb01.view01"})
+		assert.NoError(t, err)
+		assert.Equal(t, []string{"testdb01.col01", "testdb01.system.views"}, filtered)
 	})
 
 	t.Run("No Views in provided collection list", func(t *testing.T) {