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

Delete .ko.yaml #2997

Closed
wants to merge 1 commit into from
Closed

Conversation

AlanGreene
Copy link
Member

Changes

Replacement for #2996 which is stuck with
image

The default base image for ko since v0.12.0 is cgr.dev/chainguard/static This is a very lightweight base image with just the bare minimum required to run static binaries. It also runs as nonroot by default so matches the previous image.

Comparison of base image sizes (output from docker images):

cgr.dev/chainguard/static        latest      90320c6851cc  6 weeks ago    3.06 MB
gcr.io/distroless/base-debian11  nonroot     053d24ed112e  4 seconds ago  19.3 MB

Comparison of built Dashboard image (using installer script with default settings):

root@tekton-dashboard-control-plane:/# crictl images | grep dashboard
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86                  latest
921213af83dc0       23.2MB
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86
94370ac46698f930b2b8366a1f6d2a9ca3f24080c20a440106bc1599d8decc76   921213af83dc0       23.2MB
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86
ecc31e47e5b7e50c1b9c92805d3c4b69d6425f9757677fdaa800b008d198487c   e5e5e2f4101d1       31.3MB

This gives an over 25% reduction in the final image size.

/kind misc

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Docs included if any changes are user facing
  • Tests included if any functionality added or changed
  • Follows the commit message standard
  • Meets the Tekton contributor standards (including
    functionality, content, code)
  • Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
  • Release notes block below has been updated with any user facing changes (new features, significant UI changes, API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
  • Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from alangreene after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. kind/misc Categorizes issue or PR as a miscellaneuous one. labels Jul 20, 2023
The default base image for `ko` since v0.12.0 is `cgr.dev/chainguard/static`
This is a very lightweight base image with just the bare minimum required to run static binaries.
It also runs as nonroot by default so matches the previous image.

Comparison of base image sizes (output from `docker images`):

```
cgr.dev/chainguard/static        latest      90320c6851cc  6 weeks ago    3.06 MB
gcr.io/distroless/base-debian11  nonroot     053d24ed112e  4 seconds ago  19.3 MB
```

Comparison of built Dashboard image (using installer script with default settings):

```
root@tekton-dashboard-control-plane:/# crictl images | grep dashboard
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86                  latest
921213af83dc0       23.2MB
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86
94370ac46698f930b2b8366a1f6d2a9ca3f24080c20a440106bc1599d8decc76   921213af83dc0       23.2MB
kind.local/dashboard-9623576a202fe86c8b7d1bc489905f86
ecc31e47e5b7e50c1b9c92805d3c4b69d6425f9757677fdaa800b008d198487c   e5e5e2f4101d1       31.3MB
```

This gives an over 25% reduction in the final image size.
@AlanGreene AlanGreene closed this Jul 20, 2023
@AlanGreene AlanGreene deleted the ko_baseimage_update branch July 20, 2023 15:40
@AlanGreene
Copy link
Member Author

Tide was restarted and seems to be in better shape now, the original PR is unblocked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/misc Categorizes issue or PR as a miscellaneuous one. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants