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

Generalize UCX errors on connect() and correct pytest fixtures #6434

Merged
merged 3 commits into from
May 26, 2022

Commits on May 24, 2022

  1. Generalize UCX errors on connect()

    Various different errors may happen when trying to connect to a remote
    endpoint. By generalizing the exception we catch, we leave the
    responsibility of raising the appropriate error depending on known
    communication issues to Distributed.
    pentschev committed May 24, 2022
    Configuration menu
    Copy the full SHA
    2ae11ed View commit details
    Browse the repository at this point in the history

Commits on May 25, 2022

  1. Move UCX pytest fixture to utils_test, fix usage

    The fixture was not being properly used, it must be added to each
    function that requires it as an argument.
    
    Fixture was moved to `utils_test.ucx_loop` and the argument added to all
    pytest functions that do UCX testing.
    pentschev committed May 25, 2022
    Configuration menu
    Copy the full SHA
    b71e0d9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e95fb9f View commit details
    Browse the repository at this point in the history