ROX-13227: add HTTP reachability check for Central UI host#2424
ROX-13227: add HTTP reachability check for Central UI host#2424
Conversation
- Add HTTP check before marking Central as ready when using routes - Implement isCentralUIHostReachable function with 10s timeout - Check returns false if host is unreachable or returns error status - Prevents premature ready state when Central UI is not accessible 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ludydoo The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
- Create CentralUIReachabilityChecker interface for testability - Move HTTP reachability logic to HTTPCentralUIReachabilityChecker - Add MockCentralUIReachabilityChecker for testing - Inject checker dependency in reconciler for better testability - Update tests to use mock checker to ensure consistent behavior 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
@ludydoo: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
1 participant
Summary
isCentralUIHostReachablefunction with 10-second timeoutTest plan
🤖 Generated with Claude Code