Skip to content

Update/fix release tooling#85

Merged
nikepan merged 6 commits intonikepan:masterfrom
rjocoleman:fix/release-tooling
Jun 27, 2025
Merged

Update/fix release tooling#85
nikepan merged 6 commits intonikepan:masterfrom
rjocoleman:fix/release-tooling

Conversation

@rjocoleman
Copy link
Contributor

This PR updates the release pipeline and tidies up a few log messages.

  • Migrates to GoReleaser v2 with multi-arch support (amd64 + arm64)
  • Updates CI to current actions/* versions and auto-detects Go version from go.mod
  • Adds minimal GoReleaser Dockerfile
  • Enable GoReleaser to push to DockerHub (repo secret and variable required)
  • Cleans up startup logs and Echo framework banner output

When tagged with vx.x.x it should cut build via GoReleaser, update Github released and push to DockerHub.

It should all be backwards compatible but you may want to cherry-pick parts of this to fit your workflow

Two repo level items are required:

vars.DOCKERHUB_USERNAME
secrets.DOCKERHUB_TOKEN

Changes

  • GitHub Actions

    • Updated test workflow action versions
    • Use Golang version from go.mod in the actions/setup-go step
    • Added Docker builder to GoReleaser
    • Add Docker login step to allow GoReleaser to publish docker images to DockerHub (repo secret and variable are required to be set)
  • .goreleaser.yml

    • Added arm64 builds
    • Added multi-arch build output
    • Added OCI image labels and bundled config.sample.json everywhere
    • Cleaned up some issues from goreleaser check
    • Fixed goreleaser docker via Dockerfile.goreleaser - Alpine image for runtime as GoReleaser Docker support uses the binaries it builds
  • Docker

    • standalone Dockerfile now honors TARGETOS/TARGETARCH for multi-arch
  • Other

    • Version banner now shows version, commit, and build date passed in from GoReleaser
    • Echo server banner/port hidden as they're covered in other output
    • Servers message re-worded for consistency
    • Added build artifact GoReleaser dir /dist to .gitignore

@nikepan nikepan merged commit 2e229ea into nikepan:master Jun 27, 2025
0 of 2 checks passed
@rjocoleman
Copy link
Contributor Author

rjocoleman commented Jun 27, 2025

Sorry @nikepan I hadn't run this in actions, a new PR #86 or the below should fix: https://github.com/nikepan/clickhouse-bulk/actions/runs/15921392156/job/44909031826 (ref: https://github.com/docker/setup-buildx-action?tab=readme-ov-file#usage)

      - name: Set up QEMU
        uses: docker/setup-qemu-action@v3
      - name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3

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