feat(utils): add health check utilities for cloud providers#131
feat(utils): add health check utilities for cloud providers#131shivv23 wants to merge 1 commit intoc2siorg:mainfrom
Conversation
Sandipmandal25
left a comment
There was a problem hiding this comment.
The file added in this PR is not declared as a module anywhere in the crate so it is never compiled it is dead code and has no effect on the library. Please run cargo build locally and confirm it compiles before marking this ready for review.
| use std::collections::HashMap; | ||
| use std::time::{Duration, Instant}; | ||
|
|
||
| use crate::errors::CloudError; | ||
|
|
||
| #[derive(Debug, Clone)] | ||
| pub struct HealthStatus { | ||
| pub healthy: bool, | ||
| pub latency_ms: Option<u64>, | ||
| pub message: Option<String>, | ||
| } | ||
|
|
||
| #[derive(Debug, Clone)] | ||
| pub struct HealthCheckConfig { | ||
| pub timeout: Duration, |
There was a problem hiding this comment.
The file added in this PR is not declared as a module anywhere in the crate so it is never compiled it is dead code and has no effect on the library. There are also no CI checks running on this branch so compilation errors would not be caught automatically but you should verify this before opening one more new pr reporting this again . Beyond the wiring issue there is at least one code level bug in this PR that would prevent compilation. Please run cargo build locally and confirm it compiles before marking this ready for review.
bea3fad to
72d12c5
Compare
Summary
Add health check utilities for monitoring provider availability.
Why
Production apps need to check if cloud providers are reachable.
What
HealthChecker- aggregate multiple checksHealthStatus- structured response with latencycheck_tcp_port()- TCP connectivity checkcheck_http_endpoint()- HTTP health checkUsage
Files
rustcloud/src/utils/health/mod.rs