docs: document nat serve /health endpoint in REST API guide - #2173
docs: document nat serve /health endpoint in REST API guide#2173alwaysprince05 wants to merge 1 commit into
Conversation
- Add /health to Default Endpoint Paths table - Add Health Check Endpoint section with request/response examples - Document use cases for liveness/readiness probes - Cross-reference MCP/FastMCP health endpoints Closes NVIDIA#2073 Signed-off-by: alwaysprince05 <prince.p24@medhaviskillsuniversity.edu.in>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. WalkthroughThe REST API guide now lists and documents the FastAPI ChangesREST health endpoint documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change adds guidance for the existing health endpoint and introduces no actionable merge-blocking risk; it is merge-ready after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Comment |
|
@alwaysprince05 this is a duplicate of #2094 This is the second PR you've opened which is duplicative work. Please stop. |
Closes #2073
Description
Closes
By Submitting this PR I confirm:
Summary by CodeRabbit
/healthendpoint to the REST API reference.