-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Clear uname/gname when creating tar archive #1582
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
base: main
Are you sure you want to change the base?
Conversation
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]>
|
Anyone knows what this "janky" check is and why nothing happens with it ? |
|
Hi @Larsjep Thanks for working on a fix for this. Any idea when it's likely to be merged? Thanks Tom |
|
Hi @tomyan, I'm sorry but I'm also waiting for a response from the maintainers. Br |
|
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. |
|
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? |
|
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! |
|
Unfortunately, the docker team decided to reintroduce the incorrect bits to avoid breaking the cache. |
|
looking forward to this. is it stuck indefinitely, or is there still hope? |
|
Any updates on this? |
|
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. |
|
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. But I'm quite sure that you will also need the patch from this PR. |
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]