-
Notifications
You must be signed in to change notification settings - Fork 8
refactor endpoint health checker docs for alb usage #245
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
WalkthroughDocumentation updated to scope Endpoint Health Checker behavior to ALB + MetalLB environments, add a Rapid Failover concept with quantified timing (40s → 10s), and replace activation/uninstallation examples with environment-scoped YAML (namespaces, names, labels, replicas, container names/images, and ports/probes standardized to 8080). Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant ALB as ALB
participant MetalLB as MetalLB
participant EHC as Endpoint Health Checker
participant Pod as Pod (alb-pod / alb-pod-legacy)
participant Kubelet as Kubelet
rect rgb(235,245,255)
ALB->>EHC: Route traffic / probe endpoints
MetalLB->>EHC: Route traffic / probe endpoints
EHC->>Pod: Active health checks (TCP/HTTP on 8080)
Pod-->>EHC: Health OK / FAIL
end
rect rgb(245,255,235)
alt Failure detected
EHC->>ALB: Update endpoint/remove failed node
EHC->>MetalLB: Update endpoint/remove failed node
ALB-->>Traffic: Rapid failover (~10s)
MetalLB-->>Traffic: Rapid failover (~10s)
else No failure
ALB-->>Traffic: Normal routing
MetalLB-->>Traffic: Normal routing
end
end
rect rgb(255,245,235)
Note over Kubelet,EHC: Legacy kubelet-only detection (previous) ≈ 40s
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🔇 Additional comments (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
docs/en/configure/networking/functions/endpoint_health_checker.mdx
Outdated
Show resolved
Hide resolved
Signed-off-by: clyi <[email protected]>
816a189 to
5b14172
Compare
Summary by CodeRabbit