Test issue #458 (#494)

Co-authored-by: Shashank Sinha <[email protected]>

Author: percona-csalguero

Makefile

@@ -77,7 +77,7 @@ release:                      ## Build the binaries using goreleaser
 	docker run --rm --privileged \
 		-v ${PWD}:/go/src/github.com/user/repo \
 		-w /go/src/github.com/user/repo \
-		goreleaser/goreleaser release --snapshot --skip-publish --rm-dist
+		goreleaser/goreleaser release --snapshot --skip-publish --rm-dist 
 
 FILES = $(shell find . -type f -name '*.go' -not -path "./vendor/*")
 

exporter/diagnostic_data_collector.go

@@ -96,7 +96,9 @@ func (d *diagnosticDataCollector) collect(ch chan<- prometheus.Metric) {
 
 		nodeType, err := getNodeType(d.ctx, client)
 		if err != nil {
-			logger.Errorf("Cannot get node type to check if this is a mongos: %s", err)
+			logger.WithFields(logrus.Fields{
+				"component": "diagnosticDataCollector",
+			}).Errorf("Cannot get node type to check if this is a mongos: %s", err)
 		} else if nodeType == typeMongos {
 			metrics = append(metrics, mongosMetrics(d.ctx, client, logger)...)
 		}

exporter/exporter.go

@@ -137,7 +137,7 @@ func (e *Exporter) makeRegistry(ctx context.Context, client *mongo.Client, topol
 
 	nodeType, err := getNodeType(ctx, client)
 	if err != nil {
-		e.logger.Errorf("Cannot get node type to check if this is a mongos: %s", err)
+		e.logger.Errorf("Registry - Cannot get node type to check if this is a mongos : %s", err)
 	}
 
 	// Enable collectors like collstats and indexstats depending on the number of collections

exporter/topology_info.go

@@ -184,11 +184,6 @@ func getClusterRole(ctx context.Context, client *mongo.Client) (string, error) {
 		return "mongos", nil
 	}
 
-	// standalone
-	if walkTo(cmdOpts, []string{"parsed", "replication", "replSet"}) == nil {
-		return "", nil
-	}
-
 	clusterRole := ""
 	if cr := walkTo(cmdOpts, []string{"parsed", "sharding", "clusterRole"}); cr != nil {
 		clusterRole, _ = cr.(string)