Skip to content

Scrap redundant docker builds on releases#8999

Open
antondlr wants to merge 5 commits intorelease-v8.1from
optimise-release-workflow
Open

Scrap redundant docker builds on releases#8999
antondlr wants to merge 5 commits intorelease-v8.1from
optimise-release-workflow

Conversation

@antondlr
Copy link
Member

@antondlr antondlr commented Mar 18, 2026

Issue Addressed

Our release workflow is pretty inefficient and slow. This PR aims to consolidate and cut down on duplicate tasks.

Proposed Changes

  1. We now run the whole build process both on pushing to the stable branch and pushing a version tag.
    A quick win is to not fire off separate builds.

~~2) The Docker release workflow could re-use the binaries being built instead of doing its own cross-compilation. ~~
we won't take this on right now

@antondlr antondlr changed the base branch from stable to unstable March 18, 2026 07:48
@michaelsproul
Copy link
Member

We might need to target this at release-v8.1 branch. I don't think the next release will be cut from unstable and afaik the workflow from the tagged branch/stable is what will be run? Or is it the workflow from the default branch?

@antondlr
Copy link
Member Author

it will run the workflow from the tagged branch.
I'll re-target it against release-8.1 then.
do we have a tentative timeline for 8.1 ?

@antondlr antondlr force-pushed the optimise-release-workflow branch from f6e8c30 to 72480ae Compare March 19, 2026 12:13
@antondlr antondlr changed the base branch from unstable to release-v8.1 March 19, 2026 12:13
@antondlr antondlr marked this pull request as ready for review March 19, 2026 12:36
@antondlr antondlr changed the title Optimise release workflow Scrap redundant docker builds on releases Mar 19, 2026
@antondlr antondlr requested a review from jimmygchen March 20, 2026 10:27
@michaelsproul
Copy link
Member

Cargo audit fail will be fixed by:

@michaelsproul michaelsproul added infra-ci ready-for-review The code is ready for review labels Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra-ci ready-for-review The code is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants