fix: remove unnecessary COPY command from Dockerfile #3771
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
.docker/Dockerfile-build
has aCOPY
command that does not match any files.There are no files that match
internal/httpclient/go.*
.In some systems (Podman, Skaffold) that causes problems. Each of these commands produces a layer. In the case of Skaffold, it is unable to match this layer from the cache and then exits with an error.
The workaround with Skaffold is to disable caching. That creates a successful Docker image, but slows down the build process significantly.
Removing this line allowed Skaffold to build an image successfully with caching. It also produces a fully functional image.
Related issue(s)
#3446
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got the approval (please contact
[email protected]) from the maintainers to push
the changes.
works.
Further Comments