feat: Added health endpoint for the UI by blaketastic2 · Pull Request #5665 · feast-dev/feast
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
What this PR does / why we need it:
Adds a /health endpoint to the ui server for Load Balancer checks. Also uses status for HTTP codes.
Which issue(s) this PR fixes:
None
Misc
None
Signed-off-by: Blake <blaketastic2@gmail.com>
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a /health endpoint to the UI server for load balancer health checks and improves HTTP status code handling by using FastAPI's status constants instead of raw numeric codes.
- Added a new
/healthendpoint that returns HTTP status based on registry availability - Replaced raw HTTP status codes with FastAPI status constants for better readability
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Probably good to have a simple test here
added health endpoint for the UI
feat: Added health endpoint for the UI
Signed-off-by: Blake <blaketastic2@gmail.com>
Signed-off-by: Blake <blaketastic2@gmail.com>
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
