Skip to content

Update ko version used in CI#1501

Merged
cpanato merged 2 commits intoko-build:mainfrom
maboehm:bump-go-tests
Jun 2, 2025
Merged

Update ko version used in CI#1501
cpanato merged 2 commits intoko-build:mainfrom
maboehm:bump-go-tests

Conversation

@maboehm
Copy link
Contributor

@maboehm maboehm commented Apr 10, 2025

We noticed that using the ko docker image it is currently not possible to build projects that specify the latest go version in their go.mod:

docker run -v .:/tmp/demo -w /tmp/demo ghcr.io/ko-build/ko:v0.17.1 build . --push=false 
Error: failed to publish images: qualifying local import : err: exit status 1: stderr: go: go.mod requires go >= 1.24.0 (running go 1.23.2; GOTOOLCHAIN=local)

I guess all that would be needed is to just create a new release, that rebuilds the images, since the ko image always uses the latest golang image as its base.
To prepare that it probably makes sense to run the tests against these versions, and then a new release could be published.

@maboehm
Copy link
Contributor Author

maboehm commented Apr 11, 2025

I guess #1491 will need to be merged first, for the vulnerability check to be happy.

I cannot access the 5 Issues found by "Licence Compliance" - Is there something I need to do / fix?

@cpanato
Copy link
Member

cpanato commented Apr 18, 2025

@maboehm can you please rebase

@maboehm maboehm changed the title Update go versions used in CI Update ko version used in CI Apr 22, 2025
@maboehm
Copy link
Contributor Author

maboehm commented Apr 22, 2025

I rebased. Since you changed the go version in go.mod to 1.24 the matrix tests don't make sense. So the only change left in this PR is changing the version of ko used to build ko

cpanato
cpanato previously approved these changes Apr 28, 2025
- uses: ko-build/setup-ko@3aebd0597dc1e9d1a26bcfdb7cbeb19c131d3037 # v0.7
with:
version: v0.16.0 # DO NOT REMOVE THIS OR IT WILL CREATE A CYCLE.
version: v0.17.1 # DO NOT REMOVE THIS OR IT WILL CREATE A CYCLE.
Copy link
Member

Choose a reason for hiding this comment

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

can you update to v0.18.0

@cpanato cpanato merged commit c47112d into ko-build:main Jun 2, 2025
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants