docs: add doc comments to exported symbols for CI coverage

Author: cbumb

fault-quarantine/pkg/config/config.go

@@ -67,6 +67,8 @@ type RuleSet struct {
 	Cordon   Cordon `toml:"cordon"`
 }
 
+// SkipNodeLabel defines a label key/value pair that causes fault-quarantine
+// to silently drop events for any node carrying the matching label.
 type SkipNodeLabel struct {
 	Key   string `toml:"key"`
 	Value string `toml:"value"`

fault-quarantine/pkg/reconciler/reconciler_e2e_test.go

@@ -6719,6 +6719,8 @@ func TestE2E_ManualUntaintAnnotationCleanup(t *testing.T) {
 	}, eventuallyTimeout, eventuallyPollInterval, "Manual untaint annotation should be removed, FQ annotations added with taint applied")
 }
 
+// TestE2E_SkipNodeLabels_UnhealthyEvent_SkipsQuarantine verifies that an unhealthy
+// event is silently dropped for a node carrying a configured skip label.
 func TestE2E_SkipNodeLabels_UnhealthyEvent_SkipsQuarantine(t *testing.T) {
 	ctx, cancel := context.WithTimeout(e2eTestContext, 20*time.Second)
 	defer cancel()
@@ -6785,6 +6787,8 @@ func TestE2E_SkipNodeLabels_UnhealthyEvent_SkipsQuarantine(t *testing.T) {
 	}, neverTimeout, neverPollInterval, "Node with skip label should not be quarantined")
 }
 
+// TestE2E_SkipNodeLabels_HealthyEvent_DoesNotUncordon verifies that a healthy event
+// does not uncordon a quarantined node when the node carries a configured skip label.
 func TestE2E_SkipNodeLabels_HealthyEvent_DoesNotUncordon(t *testing.T) {
 	ctx, cancel := context.WithTimeout(e2eTestContext, 30*time.Second)
 	defer cancel()
@@ -6884,6 +6888,8 @@ func TestE2E_SkipNodeLabels_HealthyEvent_DoesNotUncordon(t *testing.T) {
 	}, neverTimeout, neverPollInterval, "Node should remain cordoned when skip label is present")
 }
 
+// TestE2E_SkipNodeLabels_EmptyConfig_QuarantinesNode verifies that events are
+// processed normally when the skipNodeLabels configuration is empty.
 func TestE2E_SkipNodeLabels_EmptyConfig_QuarantinesNode(t *testing.T) {
 	ctx, cancel := context.WithTimeout(e2eTestContext, 20*time.Second)
 	defer cancel()
@@ -6942,6 +6948,8 @@ func TestE2E_SkipNodeLabels_EmptyConfig_QuarantinesNode(t *testing.T) {
 	}, eventuallyTimeout, eventuallyPollInterval, "Node should be cordoned when skipNodeLabels is empty")
 }
 
+// TestE2E_SkipNodeLabels_NoMatchingValue_QuarantinesNode verifies that events are
+// processed normally when the node's label value does not match the skip config.
 func TestE2E_SkipNodeLabels_NoMatchingValue_QuarantinesNode(t *testing.T) {
 	ctx, cancel := context.WithTimeout(e2eTestContext, 20*time.Second)
 	defer cancel()
@@ -7003,6 +7011,8 @@ func TestE2E_SkipNodeLabels_NoMatchingValue_QuarantinesNode(t *testing.T) {
 	}, eventuallyTimeout, eventuallyPollInterval, "Node should be cordoned when label value doesn't match skip config")
 }
 
+// TestE2E_SkipNodeLabels_CacheMissFailsOpen verifies that isNodeSkipped returns
+// false (fail-open) when the node is not present in the informer cache.
 func TestE2E_SkipNodeLabels_CacheMissFailsOpen(t *testing.T) {
 	ctx, cancel := context.WithTimeout(e2eTestContext, 20*time.Second)
 	defer cancel()