Conversation
Signed-off-by: harshit <harshit@anyscale.com>
There was a problem hiding this comment.
Code Review
This pull request adds documentation for the new External Scaling Webhook feature. The new documentation page provides a good overview, API reference, and a Python example. I've suggested a couple of improvements to the example code to make it more robust by adding a request timeout and using a more general success check. The other changes to index and config files look good.
Signed-off-by: harshit <harshit@anyscale.com>
abrarsheikh
left a comment
There was a problem hiding this comment.
my high level feedback
- this content should be part of https://docs.ray.io/en/master/serve/advanced-guides/advanced-autoscaling.html, specifically after
Custom autoscaling policies. This will make it easier to tie a story aroundexternal scalerand why its needed. - The document is too verbose and can be condensed.
Co-authored-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
abrarsheikh
left a comment
There was a problem hiding this comment.
can you confirm that the example work if I try to run it as is following the documentation?
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
|
@cursoragent can you please fix the unresolved review comments on the latest changes? |
|
Perfect! I've successfully addressed all the unresolved review comments on your PR. Here's a summary of the changes I made: Changes Made:1. Fixed bug in
|
Co-authored-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
Signed-off-by: harshit <harshit@anyscale.com>
| - **Query current replica count**: You can get the current number of replicas for any deployment by querying the GET `/applications` API: | ||
|
|
||
| ```bash | ||
| curl -X GET http://localhost:8265/api/serve/applications/ \ |
There was a problem hiding this comment.
Is this trailing backslash intentional?
|
This pull request has been automatically marked as stale because it has not had You can always ask for help on our discussion forum or Ray's public slack channel. If you'd like to keep this open, just leave any comment, and the stale label will be removed. |
- adding docs for POST API, here are more details: https://docs.google.com/document/d/1KtMUDz1O3koihG6eh-QcUqudZjNAX3NsqqOMYh3BoWA/edit?tab=t.0#heading=h.2vf4s2d7ca46 - also, making changes for the external scaler enabled in the existing serve application docs to be merged after ray-project#57554 --------- Signed-off-by: harshit <harshit@anyscale.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Signed-off-by: YK <1811651+ykdojo@users.noreply.github.com>
- adding docs for POST API, here are more details: https://docs.google.com/document/d/1KtMUDz1O3koihG6eh-QcUqudZjNAX3NsqqOMYh3BoWA/edit?tab=t.0#heading=h.2vf4s2d7ca46 - also, making changes for the external scaler enabled in the existing serve application docs to be merged after ray-project#57554 --------- Signed-off-by: harshit <harshit@anyscale.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com>
- adding docs for POST API, here are more details: https://docs.google.com/document/d/1KtMUDz1O3koihG6eh-QcUqudZjNAX3NsqqOMYh3BoWA/edit?tab=t.0#heading=h.2vf4s2d7ca46 - also, making changes for the external scaler enabled in the existing serve application docs to be merged after ray-project#57554 --------- Signed-off-by: harshit <harshit@anyscale.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Signed-off-by: peterxcli <peterxcli@gmail.com>
adding docs for POST API, here are more details: https://docs.google.com/document/d/1KtMUDz1O3koihG6eh-QcUqudZjNAX3NsqqOMYh3BoWA/edit?tab=t.0#heading=h.2vf4s2d7ca46
also, making changes for the external scaler enabled in the existing serve application docs
to be merged after #57554