Skip to content

Commit 0d01ce6

Browse files
authored
[ci] add an always tag for cond testing (#51662)
so that "lint" tag can be used for just lints, and will not be abused to be used as "always" Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
1 parent 9f441d9 commit 0d01ce6

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

ci/pipeline/determine_tests_to_run.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111

1212
_ALL_TAGS = set(
1313
"""
14+
always
1415
lint python cpp core_cpp java workflow accelerated_dag dashboard
1516
data serve ml tune train llm rllib rllib_gpu rllib_directly
1617
linux_wheels macos_wheels docker doc python_dependencies tools
@@ -189,6 +190,7 @@ def match_tags(self, changed_file: str) -> Tuple[Set[str], bool]:
189190

190191
tags: Set[str] = set()
191192

193+
tags.add("always")
192194
tags.add("lint")
193195

194196
def _emit(line: str):

ci/pipeline/test_conditional_testing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ def __init__(self, file: str, tags: Set[str]):
153153
)
154154
tags = output.split()
155155

156-
want = test_case.tags
156+
want = set(list(test_case.tags) + ["always"])
157157
assert want == set(tags), f"file {test_case.file}, want {want}, got {tags}"
158158

159159

0 commit comments

Comments
 (0)