Skip to content
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] split window serve jobs into two #46595

Merged
merged 1 commit into from
Jul 12, 2024
Merged

[ci] split window serve jobs into two #46595

merged 1 commit into from
Jul 12, 2024

Conversation

can-anyscale
Copy link
Collaborator

@can-anyscale can-anyscale commented Jul 12, 2024

Split the window serve jobs into two - apparently the build_upload_info on windows will fail if it is called twice (https://buildkite.com/ray-project/postmerge/builds/5411#0190a59d-96ad-49a2-91d5-7572c86e4e3c/7936-7992), and this only happens on the master branch

Test:

  • CI

@can-anyscale can-anyscale force-pushed the can-winserve branch 2 times, most recently from f0e62d3 to 98cd0d5 Compare July 12, 2024 16:30
@aslonnie
Copy link
Collaborator

maybe test it on premerge or even postmerge before merging?

@can-anyscale
Copy link
Collaborator Author

the buggy code path is called only on the master branch ;), but premerge will test the rest of the integration yes

@can-anyscale can-anyscale enabled auto-merge (squash) July 12, 2024 16:39
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Jul 12, 2024
@can-anyscale can-anyscale removed the go add ONLY when ready to merge, run all tests label Jul 12, 2024
@can-anyscale can-anyscale force-pushed the can-winserve branch 2 times, most recently from eb376ac to c835812 Compare July 12, 2024 18:36
@can-anyscale can-anyscale enabled auto-merge (squash) July 12, 2024 18:45
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Jul 12, 2024
@can-anyscale can-anyscale enabled auto-merge (squash) July 12, 2024 19:55
@can-anyscale can-anyscale merged commit 42a4cb4 into master Jul 12, 2024
6 of 8 checks passed
@can-anyscale can-anyscale deleted the can-winserve branch July 12, 2024 20:06
Catch-Bull pushed a commit to Catch-Bull/ray that referenced this pull request Jul 15, 2024
Split the window serve jobs into two - apparently the build_upload_info
on windows will fail if it is called twice
(https://buildkite.com/ray-project/postmerge/builds/5411#0190a59d-96ad-49a2-91d5-7572c86e4e3c/7936-7992),
and this only happens on the master branch

Test:
- CI

Signed-off-by: can <[email protected]>
Signed-off-by: hejialing.hjl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants