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

Update in the GitHub Actions image's docker breaks docker-it-scala #153

Open
reidrac opened this issue Jun 13, 2024 · 4 comments
Open

Update in the GitHub Actions image's docker breaks docker-it-scala #153

reidrac opened this issue Jun 13, 2024 · 4 comments

Comments

@reidrac
Copy link

reidrac commented Jun 13, 2024

Looks like a recent update in the GHA image is breaking our tests with:

Cause: javax.ws.rs.client.ResponseProcessingException: com.fasterxml.jackson.databind.exc.ValueInstantiationException: Cannot construct instance of `com.spotify.docker.client.messages.ImageInfo`, problem: Null container

It started happening today. If this is an issue with the docker-client that docker-it-scala uses, how can we workaround this?

This is using ubuntu-latest image with docker-testkit-scalatest 0.12.0.

See: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20240609.1

Previous docker server was 24.0.9, now it is 26.1.3.

Thank you!

@reidrac reidrac changed the title Update in the GA image's docker breaks docker-it-scala Update in the GitHub Actions image's docker breaks docker-it-scala Jun 17, 2024
@coltfred
Copy link

I'm also running into this issue.

I think the root cause error message is:

 [info]   com.spotify.docker.client.exceptions.DockerRequestException: Request error: GET unix://localhost:80/images/postgres:14/json: 200
[info]   at com.spotify.docker.client.DefaultDockerClient.propagate(DefaultDockerClient.java:2840)
[info]   at com.spotify.docker.client.DefaultDockerClient.request(DefaultDockerClient.java:2712)

@EzequielPostan
Copy link

I am finding the same issue
Has there been any workaround found?

@coltfred
Copy link

coltfred commented Jul 8, 2024

The workaround can be seen in our PR we just linked. It's a bug with docker 26, so you can run with docker 24 and it'll work fine.

I haven't had the time to investigate the issue further, but if anyone has further info please put it here to help someone along

@vejeta
Copy link

vejeta commented Jul 17, 2024

The issue seems to stem that newer Docker Engines (from version 25 and onwards) are not providing/supporting the VirtualSize field that the underlying docker-client library from spotify is using.

Causing the error:

[info]   Cause: com.fasterxml.jackson.databind.exc.ValueInstantiationException: 
Cannot construct instance of `com.spotify.docker.client.messages.ImageInfo`, problem: Null virtualSize

I found the issue originally documented here:
openanalytics/shinyproxy#474
moby/moby#47215
moby/moby#47207

Newer versions of docker-client could be used, like:
https://github.com/dmandalidis/docker-client/

But this project should be updated to use it.

I don´t know of other workarounds, I will edit this document if I find it.

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

No branches or pull requests

4 participants