Skip to content

[Serve] Revert "Revert "[Serve] Fix ServeHandle serialization"" and disable failing Windows test#13771

Merged
simon-mo merged 13 commits intoray-project:masterfrom
architkulkarni:fix-serialize-handle-disable-win
Feb 4, 2021
Merged

[Serve] Revert "Revert "[Serve] Fix ServeHandle serialization"" and disable failing Windows test#13771
simon-mo merged 13 commits intoray-project:masterfrom
architkulkarni:fix-serialize-handle-disable-win

Conversation

@architkulkarni
Copy link
Copy Markdown
Contributor

Reverts #13753 and skips the failing Windows test (test_handle.py)

Why are these changes needed?

Related issue number

Checks

  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

@architkulkarni
Copy link
Copy Markdown
Contributor Author

test_stats_collector.py::test_node_info, test_stats_collector.py::test_logs, and test_dashboard::test_aiohttp_cache are failing, only on Mac. These don't appear in the Flaky test tracker. @simon-mo do you think they could be related to this PR?

@architkulkarni
Copy link
Copy Markdown
Contributor Author

Merged master to re-trigger CI and now there's a new kind of failure (npm errored?): https://travis-ci.com/github/ray-project/ray/jobs/480094615 @simon-mo have you seen anything like this before? Let me know if you have any ideas for what I should try next.

@simon-mo
Copy link
Copy Markdown
Contributor

simon-mo commented Feb 4, 2021

just re-ran the build, looks transient

@simon-mo simon-mo merged commit e89bbcb into ray-project:master Feb 4, 2021
@architkulkarni architkulkarni deleted the fix-serialize-handle-disable-win branch February 4, 2021 22:51
fishbone pushed a commit to fishbone/ray that referenced this pull request Feb 16, 2021
fishbone added a commit to fishbone/ray that referenced this pull request Feb 16, 2021
…"" and disable failing Windows test (ray-project#13771)"

This reverts commit 14c5222.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants