Skip to content

feat(ko): Add ko builder to local artifact builder #6785

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

Merged

Conversation

halvards
Copy link
Contributor

@halvards halvards commented Nov 1, 2021

Description
Call the ko builder from the local artifact builder and dependency graph resolver.

User facing changes
None. Config file unmarshalling still skips ko config.

Tracking: #6041

@halvards halvards requested a review from a team as a code owner November 1, 2021 02:07
@halvards halvards requested a review from briandealwis November 1, 2021 02:07
@google-cla google-cla bot added the cla: yes label Nov 1, 2021
@codecov
Copy link

codecov bot commented Nov 1, 2021

Codecov Report

Merging #6785 (798793a) into main (290280e) will decrease coverage by 0.85%.
The diff coverage is 69.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6785      +/-   ##
==========================================
- Coverage   70.48%   69.62%   -0.86%     
==========================================
  Files         515      536      +21     
  Lines       23150    24428    +1278     
==========================================
+ Hits        16317    17009     +692     
- Misses       5776     6294     +518     
- Partials     1057     1125      +68     
Impacted Files Coverage Δ
cmd/skaffold/app/cmd/deploy.go 52.00% <ø> (-1.85%) ⬇️
cmd/skaffold/app/cmd/flags.go 89.00% <0.00%> (-1.82%) ⬇️
cmd/skaffold/app/cmd/render.go 36.66% <0.00%> (-4.72%) ⬇️
cmd/skaffold/skaffold.go 0.00% <ø> (ø)
cmd/skaffold/app/cmd/lint.go 52.94% <52.94%> (ø)
cmd/skaffold/app/cmd/cmd.go 70.49% <75.00%> (-0.57%) ⬇️
cmd/skaffold/app/cmd/debug.go 100.00% <100.00%> (ø)
cmd/skaffold/app/cmd/runner.go 64.17% <100.00%> (ø)
cmd/skaffold/app/flags/image.go 76.27% <100.00%> (+3.72%) ⬆️
cmd/skaffold/app/skaffold.go 85.71% <100.00%> (+1.09%) ⬆️
... and 124 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33dbba4...798793a. Read the comment docs.

@halvards halvards added the kokoro:run runs the kokoro jobs on a PR label Nov 1, 2021
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Nov 1, 2021
@halvards halvards changed the title feat: Add ko builder to local artifact builder feat(ko): Add ko builder to local artifact builder Nov 1, 2021
"github.com/GoogleContainerTools/skaffold/testutil"
)

func Test_newPerArtifactBuilder(t *testing.T) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've generally been just capitalizing the N as in TestNewPerArtifactBuilder.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@@ -55,3 +58,41 @@ func TestSourceDependenciesCache(t *testing.T) {
}
})
}

func Test_sourceDependenciesForArtifact(t *testing.T) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And we'd use TestSourceDependenciesForArtifact

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Call the ko builder from the local artifact builder and dependency
graph resolver.

Tracking: GoogleContainerTools#6041
@halvards halvards force-pushed the ko-builder-plumbing branch from 9408ee6 to 798793a Compare November 1, 2021 21:45
@briandealwis briandealwis enabled auto-merge (squash) November 1, 2021 21:47
@halvards halvards added the kokoro:run runs the kokoro jobs on a PR label Nov 1, 2021
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Nov 1, 2021
@briandealwis briandealwis merged commit 08852b0 into GoogleContainerTools:main Nov 1, 2021
@halvards halvards deleted the ko-builder-plumbing branch November 2, 2021 00:04
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