Invert condition check in convertMetricKindToBoolIfSupported

Co-authored-by: Mike Dame <[email protected]>

exporter/collector/metrics.go

@@ -971,18 +971,16 @@ func (m *metricMapper) numberDataPointToValue(
 // unsupported type or failure to meet constraints for conversion.
 func (me *metricMapper) convertMetricKindToBoolIfSupported(point pmetric.NumberDataPoint, metricKind metricpb.MetricDescriptor_MetricKind, metricUnit string) (*monitoringpb.TypedValue, metricpb.MetricDescriptor_ValueType) {
 	boolUnitPresent := metricUnit == specialIntToBoolUnit
-	if !boolUnitPresent || metricKind != metricpb.MetricDescriptor_GAUGE || point.ValueType() != pmetric.NumberDataPointValueTypeInt {
-		// constraints for conversion failed - will not convert to boolean
-		if boolUnitPresent {
-			// indicates the user intentionally tried to convert to BOOL and failed
-			me.obs.log.Warn("Failed to interpret metric as BOOL. Attempted conversion on BOOL metrics are only supported on integer valued gauges", zap.Any("metric_kind", metricKind), zap.Any("value_type", point.ValueType()))
-		}
-		return nil, metricpb.MetricDescriptor_VALUE_TYPE_UNSPECIFIED
-	}
-	return &monitoringpb.TypedValue{Value: &monitoringpb.TypedValue_BoolValue{
-			BoolValue: point.IntValue() != 0,
-		}},
-		metricpb.MetricDescriptor_BOOL
+    if metricUnit == specialIntToBoolUnit {
+      if metricKind == metricpb.MetricDescriptor_GAUGE && point.ValueType() == pmetric.NumberDataPointValueTypeInt {
+        return &monitoringpb.TypedValue{Value: &monitoringpb.TypedValue_BoolValue{
+		        BoolValue: point.IntValue() != 0,
+		    }},
+		    metricpb.MetricDescriptor_BOOL
+      }
+      me.obs.log.Warn("Failed to interpret metric as BOOL. Attempted conversion on BOOL metrics are only supported on integer valued gauges", zap.Any("metric_kind", metricKind), zap.Any("value_type", point.ValueType()))
+    }
+    return nil, metricpb.MetricDescriptor_VALUE_TYPE_UNSPECIFIED
 }
 
 func attributesToLabels(attrs pcommon.Map) labels {