Skip to content

Simplify kaniko after we removed the GCS build context#3455

Merged
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:simplify-kaniko
Jan 15, 2020
Merged

Simplify kaniko after we removed the GCS build context#3455
dgageot merged 1 commit intoGoogleContainerTools:masterfrom
dgageot:simplify-kaniko

Conversation

@dgageot
Copy link
Copy Markdown
Contributor

@dgageot dgageot commented Dec 26, 2019

This removes the GCS build context for Kaniko and instead always defaults to local directory. This seems to create much less friction and makes it possible to simplify the codebase a lot.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 26, 2019

Codecov Report

Merging #3455 into master will decrease coverage by 0.03%.
The diff coverage is 74.47%.

Impacted Files Coverage Δ
pkg/skaffold/schema/latest/config.go 100% <ø> (ø) ⬆️
pkg/skaffold/build/cluster/cluster.go 27.27% <0%> (ø) ⬆️
pkg/skaffold/build/cache/retrieve.go 68.13% <0%> (ø) ⬆️
pkg/skaffold/build/cluster/kaniko.go 0% <0%> (-54.44%) ⬇️
pkg/skaffold/schema/defaults/defaults.go 94.15% <100%> (ø) ⬆️
pkg/skaffold/build/cluster/pod.go 86.58% <86.58%> (ø)

@dgageot dgageot force-pushed the simplify-kaniko branch 3 times, most recently from 31adccb to 91115b9 Compare December 27, 2019 08:29
@dgageot dgageot force-pushed the simplify-kaniko branch 3 times, most recently from ba5cbaf to d976dba Compare January 10, 2020 05:46
@dgageot dgageot changed the title Simplify kaniko after we remove the GCS build context Simplify kaniko after we removed the GCS build context Jan 14, 2020
Signed-off-by: David Gageot <david@gageot.net>
@tejal29 tejal29 self-assigned this Jan 14, 2020
@dgageot dgageot merged commit 58a72fc into GoogleContainerTools:master Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants