Skip to content

chore(ci): use blacksmith runners#2429

Merged
cemalkilic merged 2 commits into
masterfrom
cemal/chore-blacksmith-runners
Apr 2, 2026
Merged

chore(ci): use blacksmith runners#2429
cemalkilic merged 2 commits into
masterfrom
cemal/chore-blacksmith-runners

Conversation

@cemalkilic

Copy link
Copy Markdown
Contributor

use blacksmith runners in CI

@cemalkilic cemalkilic requested a review from a team as a code owner March 17, 2026 17:39
Comment thread .github/workflows/conventional-commits.yml Outdated
cstockton added a commit that referenced this pull request Mar 25, 2026
The primary goals of this PR are:
* Better parallelization for github workflows and local builds
* Allow easily running the github workflows locally

Changes:
* Makefile now describes dependencies among targets to enable parallel
jobs
* Created `release` target to be called from the release workflow
    * Creates temporary build folders to isolate symlinks
* Created `release-test` target to be called form the test workflow
* Makefile now has a release target which will concurrently build all 4
archives
* Added -j flag to Makefile so it may create multiple jobs
    * This should leverage all cpus 4 cpu machines we will build on
* I've made the arch explicit (GOOS=amd64) for x86 binary for safety
* For now I've preserved the existing targets for "make build" and "make
build-strip", but ideally:
* `make build` only builds using `CGO_ENABLED=0 go build` giving native
arch for local dev.
* `make release` builds all the release binaries, without any env
defined values such as `CGO_ENABLED=0 go build` (it is currently doing
this now)
* I've collapsed the test workflows into one step so they can run
concurrently

One note is that the go tool chain does make use of multiple cpus. But
there should still be measurable benefits on 4 cpu machines with these
changes.

Related:
#2429

---------

Co-authored-by: Chris Stockton <chris.stockton@supabase.io>
@cstockton cstockton self-requested a review March 26, 2026 15:02
@cstockton cstockton force-pushed the cemal/chore-blacksmith-runners branch from b303e18 to 0a5797f Compare March 26, 2026 15:09
@cstockton

Copy link
Copy Markdown
Contributor

Good to merge these @cemalkilic ?

@cemalkilic cemalkilic merged commit 3821290 into master Apr 2, 2026
5 checks passed
@cemalkilic cemalkilic deleted the cemal/chore-blacksmith-runners branch April 2, 2026 13:58
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.

3 participants