Skip to content

Commit 3dbfb52

Browse files
Fix: Add EKS evidence checks to check_evidence_availability
1 parent 7b157ae commit 3dbfb52

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

app/nodes/root_cause_diagnosis/evidence_checker.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,9 @@ def check_evidence_availability(
6565
or evidence.get("aws_cloudwatch_metrics")
6666
or evidence.get("aws_rds_events")
6767
or evidence.get("aws_performance_insights")
68+
or evidence.get("eks_pods") is not None
69+
or evidence.get("eks_events") is not None
70+
or evidence.get("eks_node_health") is not None
6871
)
6972

7073
# Check for evidence in alert annotations or raw text

tests/nodes/root_cause_diagnosis/test_evidence_checker.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ def test_is_clearly_healthy_pure_eks():
1616
"eks_deployments": [{"name": "payments-api", "ready": 3, "desired": 3}],
1717
"eks_node_health": [{"name": "node-1", "ready": "True"}],
1818
}
19-
19+
2020
assert is_clearly_healthy(raw_alert, eks_only) is True
2121

2222
def test_is_clearly_healthy_mixed():
@@ -31,7 +31,7 @@ def test_is_clearly_healthy_mixed():
3131
"eks_pods": [{"name": "payments-api-x", "phase": "Running"}],
3232
"datadog_logs": []
3333
}
34-
34+
3535
assert is_clearly_healthy(raw_alert, mixed) is True
3636

3737
def test_is_clearly_healthy_not_healthy_state():
@@ -45,5 +45,5 @@ def test_is_clearly_healthy_not_healthy_state():
4545
eks_only = {
4646
"eks_pods": [{"name": "payments-api-x", "phase": "Running"}],
4747
}
48-
48+
4949
assert is_clearly_healthy(raw_alert, eks_only) is False

0 commit comments

Comments
 (0)