Skip to content

Fix handling of multi stage builds#2340

Merged
dgageot merged 5 commits intoGoogleContainerTools:masterfrom
dgageot:fix-2315
Jun 26, 2019
Merged

Fix handling of multi stage builds#2340
dgageot merged 5 commits intoGoogleContainerTools:masterfrom
dgageot:fix-2315

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Jun 26, 2019

Fixes #2315

dgageot added 5 commits June 26, 2019 11:24
Signed-off-by: David Gageot <david@gageot.net>
Fixes GoogleContainerTools#2315

Signed-off-by: David Gageot <david@gageot.net>
Signed-off-by: David Gageot <david@gageot.net>
It will fail anyways when getting the WORKDIR.
This simplifies the tests because now, we can just
let them fail if they try to fetch an unknown image.

Signed-off-by: David Gageot <david@gageot.net>
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 26, 2019

Codecov Report

Merging #2340 into master will decrease coverage by 0.16%.
The diff coverage is 84.61%.

Impacted Files Coverage Δ
pkg/skaffold/docker/parse.go 88.33% <84.61%> (+2.78%) ⬆️
pkg/skaffold/docker/image_util.go 0% <0%> (-33.34%) ⬇️
pkg/skaffold/docker/remote.go 53.12% <0%> (-9.38%) ⬇️
pkg/skaffold/jib/jib.go 76.34% <0%> (-2.16%) ⬇️
pkg/skaffold/docker/client.go 69.6% <0%> (-1.97%) ⬇️
pkg/skaffold/docker/image.go 75.78% <0%> (-1.06%) ⬇️

@dgageot dgageot merged commit 44111c3 into GoogleContainerTools:master Jun 26, 2019
valery-vitko added a commit to Steinpilz/docker that referenced this pull request Jun 27, 2019
Downgrade to a previous version until fix for multi stage Docker builds
released (GoogleContainerTools/skaffold#2340).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove

3 participants