Skip to content

Commit

Permalink
[CI][Core] Set some GCE smoke tests to run on manual frequency (ray-p…
Browse files Browse the repository at this point in the history
…roject#34516)

I noticed some GCE smoke versions are run on nightly. Let's move them to run on manual instead, since we don't want to spend the cost on run them on an automatic cadence yet.

---------

Signed-off-by: Cuong Nguyen <[email protected]>
Signed-off-by: elliottower <[email protected]>
  • Loading branch information
can-anyscale authored and elliottower committed Apr 22, 2023
1 parent 6a614fa commit e6e49fa
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion release/ray_release/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ def parse_test_definition(test_definitions: List[TestDefinition]) -> List[Test]:
)
test = copy.deepcopy(test_definition)
test["name"] = f'{test["name"]}.{variation.pop("__suffix__")}'
test.update(variation)
test = deep_update(test, variation)
tests.append(test)
return tests

Expand Down
2 changes: 1 addition & 1 deletion release/ray_release/tests/test_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ def test_parse_test_definition():
- __suffix__: aws
- __suffix__: gce
cluster:
cluster_env: env_gce.yaml
cluster_compute: compute_gce.yaml
"""
)
Expand All @@ -79,6 +78,7 @@ def test_parse_test_definition():
assert not validate_test(gce_test, schema)
assert aws_test["name"] == "sample_test.aws"
assert gce_test["cluster"]["cluster_compute"] == "compute_gce.yaml"
assert gce_test["cluster"]["cluster_env"] == "env.yaml"
invalid_test_definition = test_definitions[0]
# Intentionally make the test definition invalid by create an empty 'variations'
# field. Check that the parser throws exception at runtime
Expand Down
7 changes: 6 additions & 1 deletion release/release_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -1670,7 +1670,6 @@
env: gce
frequency: manual
cluster:
cluster_env: app_config.yaml
cluster_compute: tpl_gce_4x8.yaml

alert: tune_tests
Expand Down Expand Up @@ -4201,6 +4200,8 @@
cluster:
cluster_env: stress_tests/state_api_app_config.yaml
cluster_compute: stress_tests/stress_tests_compute_large_gce.yaml
smoke_test:
frequency: manual


- name: shuffle_20gb_with_state_api
Expand Down Expand Up @@ -4264,6 +4265,8 @@
cluster:
cluster_env: stress_tests/stress_tests_app_config.yaml
cluster_compute: stress_tests/stress_tests_compute_gce.yaml
smoke_test:
frequency: manual

- name: stress_test_dead_actors
group: core-daily-test
Expand Down Expand Up @@ -4303,6 +4306,8 @@
cluster:
cluster_env: stress_tests/stress_tests_app_config.yaml
cluster_compute: stress_tests/stress_tests_compute_gce.yaml
smoke_test:
frequency: manual

# The full test is not stable, so run the smoke test only.
# See https://github.com/ray-project/ray/issues/23244.
Expand Down

0 comments on commit e6e49fa

Please sign in to comment.