Skip to content

Conversation

@Larsjep
Copy link

@Larsjep Larsjep commented Apr 30, 2017

To ensure that the cache from docker command client
can be reused, we must clear group/user name as the docker
client does.

Together with golang/go#20150 this fixes #998

Signed-off-by: Lars Jeppesen [email protected]

To ensure that the cache from docker command client
can be reused, we must clear group/user name as the docker
client does.

Together with golang/go#20150 this fixes docker#998

Signed-off-by: Lars Jeppesen <[email protected]>
@Larsjep
Copy link
Author

Larsjep commented May 2, 2017

Anyone knows what this "janky" check is and why nothing happens with it ?

@tomyan
Copy link

tomyan commented May 23, 2017

Hi @Larsjep

Thanks for working on a fix for this. Any idea when it's likely to be merged?

Thanks

Tom

@Larsjep
Copy link
Author

Larsjep commented May 23, 2017

Hi @tomyan,

I'm sorry but I'm also waiting for a response from the maintainers.
They might be busy with other stuff, because I can see several other pull requests waiting.

Br
Lars

@shin-
Copy link
Contributor

shin- commented Jul 3, 2017

Thank you @Larsjep !

Sorry about the late response - my understanding is that this patch will not do anything unless it is used with a Docker Engine built with Go 1.9+, of which there has been none so far. Once one such Engine has been released, we can test those changes to ensure they have the expected result, and if confirmed, merge the PR.

@edmorley
Copy link

edmorley commented Sep 1, 2017

I may be misreading, but it looks like moby/moby#33935 has re-added the very bits that were intentionally removed in golang/go#20150, in an attempt to fix the test failures (due to cache hashes now being different) seen in moby/moby#33892?

@oskarpearson
Copy link

Hi @shin- and @Larsjep

Docker CE 17.12.0 has just been released, and includes "Bump Go to 1.9.2" for both moby and the cli. See the Runtime section of https://github.com/docker/docker-ce/releases/tag/v17.12.0-ce

Is there a way I can test this patch out?

Thanks for your help!

@Larsjep
Copy link
Author

Larsjep commented Dec 28, 2017

Unfortunately, the docker team decided to reintroduce the incorrect bits to avoid breaking the cache.
They said that they already planned to break the cache when introducing buildkit and then correct the bits at that point. But I haven't been able to get an answer on when buildkit would be implemented and that is 4 months ago :(.
See moby/moby#33935

@r631269
Copy link

r631269 commented Jun 7, 2018

looking forward to this. is it stuck indefinitely, or is there still hope?

@kilianc
Copy link

kilianc commented Jul 26, 2018

Any updates on this?

@AlJohri
Copy link

AlJohri commented Jan 7, 2020

Bump, also running into this issue

@Larsjep
Copy link
Author

Larsjep commented Jan 8, 2020

Bump, also running into this issue

Have you tried using buildkit?. That should have been include since Docker 18.06. The docker team promised that they would fix the bits when switching to buildkit. If that is the case this PR should fix the problem.
I'm no longer working on the project where I discovered this issue, so I will not have time to test it myself.

@AlJohri
Copy link

AlJohri commented Jan 8, 2020

No, I haven't tried using buildkit yet. Just the default docker and docker-compose that comes with Docker for Mac (19.03.5).

@Larsjep
Copy link
Author

Larsjep commented Jan 8, 2020

No, I haven't tried using buildkit yet. Just the default docker and docker-compose that comes with Docker for Mac (19.03.5).

It should be included with that build, but you will need to enable it by setting DOCKER_BUILDKIT=1.
See here: https://docs.docker.com/develop/develop-images/build_enhancements/

But I'm quite sure that you will also need the patch from this PR.

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.

Client.build and docker build don't share a build cache

8 participants