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

Mark entire CudaTest class as requiring docker #302

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented Dec 3, 2024

The call to get_docker_client in setUpClass will fail if docker is not available on the host.

Resolves:

test/test_nvidia.py:256: in setUpClass
    client = get_docker_client()
../../install_isolated/rocker/lib/python3.12/site-packages/rocker/core.py:220: in get_docker_client
    raise DependencyMissing('Docker Client failed to connect to docker daemon.'
E   rocker.core.DependencyMissing: Docker Client failed to connect to docker daemon. Please verify that docker is installed and running. As well as that you have permission to access the docker daemon. This is usually by being a member of the docker group. The underlying error was:
E   """
E   Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
E   """

The call to get_docker_client in setUpClass will fail if docker is not
available on the host.
@cottsay cottsay added the bug Something isn't working label Dec 3, 2024
@cottsay cottsay self-assigned this Dec 3, 2024
@cottsay cottsay requested a review from tfoote as a code owner December 3, 2024 18:32
Copy link
Collaborator

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, yeah the invocation of get_docker_client in the setUpClass does require that now.

@tfoote tfoote merged commit b150019 into main Dec 6, 2024
8 checks passed
@tfoote tfoote deleted the cottsay/test-markings branch December 6, 2024 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants