Skip to content

build: replace ko with make images using podman or docker - #508

Open
rakyll wants to merge 1 commit into
mainfrom
ko
Open

rakyll wants to merge 1 commit into
mainfrom
ko

Conversation

@rakyll

@rakyll rakyll commented Oct 22, 2026

Copy link
Copy Markdown
Member

Remove ko from the build and deploy flow. The new make images target cross-compiles ax-server and ax-task-runner for linux/$IMAGE_ARCH, builds their container images with podman (falling back to docker), and pushes them to $AX_IMAGE_REPO. AX_IMAGE_REPO is required, and IMAGE_TAG overrides the default latest tag.

make deploy-server now fills in the ax-server image in deploy/ax-server.yaml. It also sets AX_DEFAULT_TASK_IMAGE, so the server runs the task-runner image you pushed for tasks that do not set spec.image. Since the default tag is latest, imagePullPolicy is now Always.

Other changes:

  • Delete .ko.yaml.
  • Move Dockerfile.server from test/e2e/ to the repo root so the e2e setup and make images share it.
  • Remove redundant Makefile targets (all, build-binaries, build-task-runner, push-task-runner, push).
  • Update README, CONTRIBUTING, and docs to use make images.

Remove ko from the build and deploy flow. The new `make images` target
cross-compiles ax-server and ax-task-runner for linux/$IMAGE_ARCH,
builds their container images with podman (falling back to docker),
and pushes them to $AX_IMAGE_REPO. AX_IMAGE_REPO is required, and
IMAGE_TAG overrides the default `latest` tag.

`make deploy-server` now fills in the ax-server image in
deploy/ax-server.yaml. It also sets AX_DEFAULT_TASK_IMAGE, so the
server runs the task-runner image you pushed for tasks that do not
set spec.image. Since the default tag is `latest`, imagePullPolicy is
now Always.

Other changes:
- Delete .ko.yaml.
- Move Dockerfile.server from test/e2e/ to the repo root so the e2e
  setup and `make images` share it.
- Remove redundant Makefile targets (all, build-binaries,
  build-task-runner, push-task-runner, push).
- Update README, CONTRIBUTING, and docs to use `make images`.
@rakyll
rakyll requested a review from mayawang October 22, 2026 18:36

@mayawang mayawang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @rakyll, dropping ko is a big step for onboarding. One issue: deploy-server sets AX_DEFAULT_TASK_IMAGE to the :latest tag, and Substrate rejects images that aren't pinned by digest, so any task without spec.image fails at template creation. Could images resolve the pushed digest and deploy-server use image@sha256:... instead? The old push-task-runner printed it. Also heads-up: this conflicts with #505 in README.md, and #500's install guide is built around ko, so it'll need updating after this lands. I'd suggest landing #505 first.

@rakyll

rakyll commented Oct 22, 2026

Copy link
Copy Markdown
Member Author

This can wait, let's focus on #500 and #505.

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