cleanup

Author: oliver006

exporter/exporter.go

@@ -727,11 +727,11 @@ func (e *Exporter) scrapeRedisHost(ch chan<- prometheus.Metric) error {
 		e.extractLatencyMetrics(ch, infoAll, c)
 	}
 
-	// skip these metrics for master?
+	// skip these metrics for master if SkipCheckKeysForRoleMaster is set
 	// (can help with reducing work load on the master node)
 	log.Infof("checkKeys metric collection for role: %s  flag: %#v", role, e.options.SkipCheckKeysForRoleMaster)
 	debug.PrintStack()
-	if role != InstanceRoleSlave && !e.options.SkipCheckKeysForRoleMaster {
+	if role == InstanceRoleSlave || !e.options.SkipCheckKeysForRoleMaster {
 		if err := e.extractCheckKeyMetrics(ch, c); err != nil {
 			log.Errorf("extractCheckKeyMetrics() err: %s", err)
 		}