-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ci][microcheck/13] reuse the logic to determine microcheck tests #45504
Conversation
d535dc2
to
bf3907a
Compare
91a4bb8
to
bca5449
Compare
a030647
to
8705ac3
Compare
@@ -198,33 +198,52 @@ def gen_from_s3(cls, prefix: str): | |||
] | |||
|
|||
@classmethod | |||
def gen_high_impact_tests(cls, prefix: str) -> Dict[str, List]: | |||
def gen_microcheck_step_ids(cls, prefix: str, bazel_workspace_dir: str) -> Set[str]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe add docstring about what this method does?
high_impact_tests = _get_high_impact_test_targets( | ||
team, operating_system, container | ||
) | ||
prefix = f"{operating_system}:" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe update the comment before this line about the union of new tests?
added comments; test all microcheck tests: https://buildkite.com/ray-project/microcheck/builds/592 |
00d2f89
to
914bad4
Compare
Test class Signed-off-by: can <[email protected]>
Test class Signed-off-by: can <[email protected]>
Signed-off-by: can <[email protected]>
Signed-off-by: can <[email protected]>
68757b3
to
c6c69c4
Compare
Signed-off-by: can <[email protected]>
4682bb7
to
52d799f
Compare
falls there; until we can figure out something better Signed-off-by: can <[email protected]>
…y-project#45504) Currently the logic to determine the list of microcheck tests and their rayci step ids are diverging. This PR move more function into Test class, so we converge more these two logic. This make sure that we will trigger the right buildkite steps to cover most if not all microcheck tests. Test: - CI --------- Signed-off-by: can <[email protected]> Signed-off-by: Ryan O'Leary <[email protected]>
…y-project#45504) Currently the logic to determine the list of microcheck tests and their rayci step ids are diverging. This PR move more function into Test class, so we converge more these two logic. This make sure that we will trigger the right buildkite steps to cover most if not all microcheck tests. Test: - CI --------- Signed-off-by: can <[email protected]> Signed-off-by: Ryan O'Leary <[email protected]>
…y-project#45504) Currently the logic to determine the list of microcheck tests and their rayci step ids are diverging. This PR move more function into Test class, so we converge more these two logic. This make sure that we will trigger the right buildkite steps to cover most if not all microcheck tests. Test: - CI --------- Signed-off-by: can <[email protected]>
Currently the logic to determine the list of microcheck tests and their rayci step ids are diverging. This PR move more function into Test class, so we converge more these two logic.
This make sure that we will trigger the right buildkite steps to cover most if not all microcheck tests.
Test: