Skip to content

Commit 2cad050

Browse files
Rename autoscaling metric test variants for what they now do
With controller aggregation the only mode, "metr_agg_at_controller" no longer distinguishes anything. Name each variant by the collection site it actually configures: COLLECT_AUTOSCALING_METRICS_ON_HANDLE=0 collects on replicas, and the flag defaults to 1, so an unset variant collects on handles. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: John Taylor <john.taylor@anyscale.com>
1 parent 6923f04 commit 2cad050

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

python/ray/serve/tests/BUILD.bazel

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@ py_test_module_list(
3434
py_test_module_list_with_env_variants(
3535
size = "medium",
3636
env_variants = {
37-
"metr_agg_at_controller": {
37+
"metr_collect_on_replica": {
3838
"env": {
3939
"RAY_SERVE_COLLECT_AUTOSCALING_METRICS_ON_HANDLE": "0",
4040
"RAY_SERVE_AUTOSCALING_METRIC_RECORD_INTERVAL_FACTOR": "0.001",
4141
"RAY_SERVE_RECORD_AUTOSCALING_STATS_TIMEOUT_S": "3",
4242
},
43-
"name_suffix": "_metr_agg_at_controller",
43+
"name_suffix": "_metr_collect_on_replica",
4444
},
4545
},
4646
files = [
@@ -556,12 +556,12 @@ AUTOSCALING_METRIC_ENV_VARIANTS = {
556556
},
557557
"name_suffix": "_metr_disab",
558558
},
559-
"metr_agg_at_controller": {
559+
"metr_collect_on_handle": {
560560
"env": {
561561
# Make sure queued metrics are cleared out quickly.
562562
"RAY_SERVE_HANDLE_AUTOSCALING_METRIC_PUSH_INTERVAL_S": "0.1",
563563
},
564-
"name_suffix": "_metr_agg_at_controller",
564+
"name_suffix": "_metr_collect_on_handle",
565565
},
566566
}
567567

python/ray/serve/tests/unit/BUILD.bazel

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,12 +80,12 @@ py_test_module_list_with_env_variants(
8080
},
8181
"name_suffix": "_metr_disab",
8282
},
83-
"metr_agg_at_controller": {
83+
"metr_collect_on_handle": {
8484
"env": {
8585
"RAY_SERVE_COLLECT_AUTOSCALING_METRICS_ON_HANDLE": "1",
8686
"RAY_SERVE_FAIL_ON_RANK_ERROR": "1",
8787
},
88-
"name_suffix": "_metr_agg_at_controller",
88+
"name_suffix": "_metr_collect_on_handle",
8989
},
9090
},
9191
files = [

0 commit comments

Comments
 (0)