Skip to content
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

Detect ko images for debugging, by image author #6569

Merged
merged 1 commit into from
Sep 8, 2021

Conversation

halvards
Copy link
Contributor

@halvards halvards commented Sep 6, 2021

Description

Container images built by ko have the author property set to the value github.com/google/ko.

Skaffold debug can use this property to detect a Go runtime artifact.

This feature can be used for images built using Skaffold's ko builder (when ready), and for images build using ko via a custom build (usable now).

Tracking: #6041
Related: #6563
Context: #6496 (comment)

**Description**

Container images built by ko have the
[author](https://github.com/opencontainers/image-spec/blob/main/config.md#properties)
property set to the value
[`github.com/google/ko`](https://github.com/google/ko/blob/v0.8.3/pkg/build/gobuild.go#L610).

Skaffold debug can use this property to detect a Go runtime artifact.

This feature can be used for images built using Skaffold's ko builder (when ready),
and for images build using ko via a custom build (usable now).

**Tracking**: GoogleContainerTools#6041
**Related**: GoogleContainerTools#6563
**Context**: GoogleContainerTools#6496 (comment)
@codecov
Copy link

codecov bot commented Sep 6, 2021

Codecov Report

Merging #6569 (ef44ba5) into main (290280e) will increase coverage by 0.03%.
The diff coverage is 61.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6569      +/-   ##
==========================================
+ Coverage   70.48%   70.51%   +0.03%     
==========================================
  Files         515      515              
  Lines       23150    23154       +4     
==========================================
+ Hits        16317    16328      +11     
+ Misses       5776     5770       -6     
+ Partials     1057     1056       -1     
Impacted Files Coverage Δ
cmd/skaffold/app/cmd/cmd.go 70.68% <0.00%> (-0.38%) ⬇️
cmd/skaffold/app/cmd/flags.go 90.81% <ø> (ø)
cmd/skaffold/skaffold.go 0.00% <ø> (ø)
pkg/skaffold/build/buildpacks/logger.go 0.00% <ø> (ø)
pkg/skaffold/config/options.go 100.00% <ø> (ø)
pkg/skaffold/debug/apply_transforms.go 22.64% <0.00%> (-0.44%) ⬇️
pkg/skaffold/debug/transform.go 95.91% <ø> (ø)
pkg/skaffold/schema/latest/v1/config.go 62.16% <ø> (ø)
pkg/skaffold/server/server.go 46.87% <0.00%> (-0.75%) ⬇️
.../skaffold/kubernetes/status/resource/deployment.go 85.09% <50.00%> (-0.71%) ⬇️
... and 12 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 70e9db9...ef44ba5. Read the comment docs.

@halvards halvards added the kokoro:run runs the kokoro jobs on a PR label Sep 6, 2021
@kokoro-team kokoro-team removed the kokoro:run runs the kokoro jobs on a PR label Sep 6, 2021
@tejal29 tejal29 merged commit eb73ab6 into GoogleContainerTools:main Sep 8, 2021
halvards added a commit to halvards/skaffold that referenced this pull request Sep 13, 2021
Disable Go compiler optimizations when building an artifact with the ko
builder, and the Skaffold run mode is debug.

**Tracking**: GoogleContainerTools#6041
**Related**: GoogleContainerTools#6563, GoogleContainerTools#6569
MarlonGamez pushed a commit that referenced this pull request Sep 20, 2021
Disable Go compiler optimizations when building an artifact with the ko
builder, and the Skaffold run mode is debug.

**Tracking**: #6041
**Related**: #6563, #6569
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.

4 participants