[ksmcore] Remap unknown to warning for 'node.ready' service check

pkg/collector/corechecks/cluster/ksm/kubernetes_state_transformers.go

@@ -126,6 +126,9 @@ func statusForCondition(status string, positiveEvent bool) metrics.ServiceCheckS
 		}
 		return metrics.ServiceCheckOK
 	case "unknown":
+		if positiveEvent {
+			return metrics.ServiceCheckWarning
+		}
 		return metrics.ServiceCheckUnknown
 	default:
 		log.Tracef("Unknown 'status' label: '%s'", status)

pkg/collector/corechecks/cluster/ksm/kubernetes_state_transformers_test.go

@@ -1189,7 +1189,7 @@ func Test_nodeConditionTransformer(t *testing.T) {
 			expectedServiceCheck: &serviceCheck{
 				name:    "kubernetes_state.node.ready",
 				tags:    []string{"node:foo", "condition:Ready", "status:unknown"},
-				status:  metrics.ServiceCheckUnknown,
+				status:  metrics.ServiceCheckWarning,
 				message: "foo is currently reporting Ready = unknown",
 			},
 			expectedMetric: &metricsExpected{

releasenotes/notes/ksmcore-6fd90b79747b6321.yaml

@@ -0,0 +1,12 @@
+# Each section from every release note are combined when the
+# CHANGELOG.rst is rendered. So the text needs to be worded so that
+# it does not depend on any information only available in another
+# section. This may mean repeating some details, but each section
+# must be readable independently of the other.
+#
+# Each section note must be formatted as reStructuredText.
+---
+enhancements:
+  - |
+    KSMCore `node.ready` service check now reports `warning`
+    instead of `unknown` when a node enters an unknown state.