Merge pull request #51 from tigrisdata/hk/locality2

fix: by default use "unknown" zone if locality is not set

Author: singler

metrics/group_processes.go

@@ -44,16 +44,17 @@ func (p *ProcessesMetricGroup) getTags(processName string, process *models.Proce
 		// For no Kubenetes deployments, metrics will be tagged with the machine hostname and the address of the process
 		tags["machine_id"] = process.MachineId
 		tags["address"] = process.Address
+		tags["zone"] = "unknown"
 		if process.Locality != nil {
 			tags["zone"] = process.Locality.ZoneId
 		}
 	} else {
+		tags["fdb_pod_name"] = processName
+		tags["zone"] = "unknown"
 		// Try to tag the name with the human readable name (as published by the operator) instead of the key in the json
 		if process.Locality != nil {
 			tags["fdb_pod_name"] = process.Locality.InstanceId
 			tags["zone"] = process.Locality.ZoneId
-		} else {
-			tags["fdb_pod_name"] = processName
 		}
 	}
 	tags["class_type"] = process.ClassType