-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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][postmerge/2] schedule gap builds #44576
Conversation
6412bd0
to
b8dbc1a
Compare
5c18a9b
to
054f6f4
Compare
85ffd49
to
f9c868a
Compare
dbda268
to
3745eae
Compare
f"git rev-list --reverse ^{passing_revision} {failing_revision}~", | ||
shell=True, |
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.
don't use shell
? but send in the args list?
@mock.patch( | ||
"ci.ray_ci.pipeline.gap_filling_scheduler.GapFillingScheduler.get_gap_commits" | ||
) | ||
def test_run(mock_get_gap_commits, mock_trigger_build): |
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.
add test case where no builds are triggered?
@aslonnie's comments |
Signed-off-by: can <[email protected]>
Signed-off-by: can <[email protected]>
Signed-off-by: can <[email protected]>
.split("\n") | ||
) | ||
|
||
def _trigger_build(self, commit: str) -> int: |
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.
what is the return value of this? should this return an integer as a place holder so that it respects the typing annotation?
we should probably turn on typing lints for CI python scripts..
Signed-off-by: can <[email protected]>
* [ci][postmerge/2] schedule gap filling builds Signed-off-by: can <[email protected]> * [ci][postmerge/2] schedule gap filling builds Signed-off-by: can <[email protected]> * [ci][postmerge/3] actually trigger build in commit gaps Signed-off-by: can <[email protected]> --------- Signed-off-by: can <[email protected]>
- Add logic to compute the list of gap commits and schedule one build for each gap - Implement the function to trigger a blocked build on postmerge - Also cache the _get_builds function which is used repeatedly and quite expensive Signed-off-by: can <[email protected]>
- Add logic to compute the list of gap commits and schedule one build for each gap - Implement the function to trigger a blocked build on postmerge - Also cache the _get_builds function which is used repeatedly and quite expensive Signed-off-by: can <[email protected]>
Test: