Optimize release workflow: add caching and fix v-prefixed tag handling #2783
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit addresses two major issues in the release workflow: 1. **Slow Build Times (40+ minutes)** - Added Rust caching to build-python-release workflow using Swatinem/rust-cache - Enabled Rust caching in build-cli-release workflow (was disabled) - Improved cache keys for build-typescript-release with target-specific prefixes - Added uv caching for Python dependency management - Expected improvement: 30-50% reduction in build times after cache warmup 2. **v-Prefixed Tag Handling** - Fixed determine-version job to recognize both X.Y.Z and vX.Y.Z tag formats - Strips 'v' prefix from version string for consistency - Ensures publish jobs run correctly for both tag formats (required for Go modules) - Resolves upload step failures on v0.2xx.0 tags Changes: - .github/workflows/release.yml: Updated tag regex to match v?[0-9]+\.[0-9]+\.[0-9]+ - .github/workflows/build-python-release.reusable.yaml: Added Rust and uv caching - .github/workflows/build-typescript-release.reusable.yaml: Improved cache key specificity - .github/workflows/build-cli-release.reusable.yaml: Enabled Rust caching
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| # Setup Rust with caching for faster builds | ||
| - name: Setup Rust toolchain | ||
| uses: dtolnay/rust-toolchain@master |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
| targets: ${{ matrix._.target }} | ||
|
|
||
| - name: Setup Rust cache | ||
| uses: Swatinem/rust-cache@v2 |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR addresses the two major issues with the release workflow identified in https://github.com/BoundaryML/baml/actions/runs/19959566319:
1. Slow Build Times 🚀
The release workflow was taking 40+ minutes due to missing or misconfigured caching:
Swatinem/rust-cache@v2and enabled uv caching for Python dependenciesenable-cache: false)v6-rust-ts-${{ matrix._.target }})Expected Impact: 30-50% reduction in build times after initial cache warmup. Subsequent builds should see significant speedups in dependency compilation.
2. v-Prefixed Tag Handling 🏷️
The
determine-versionjob only recognized tags matching^[0-9]+\.[0-9]+\.[0-9]+$(e.g.,0.214.0), causing publish jobs to be skipped for v-prefixed tags (e.g.,v0.214.0) required by Go modules:X.Y.ZandvX.Y.Zformats:^v?[0-9]+\.[0-9]+\.[0-9]+$vprefix from version string for consistency across artifactsis_release_tagis set correctly for both tag formatsImpact: Publish jobs will now run correctly for both
0.2xx.0andv0.2xx.0tags, eliminating upload step failures.Changes
.github/workflows/release.yml: Updated tag regex and version extraction logic.github/workflows/build-python-release.reusable.yaml: Added Rust and uv caching.github/workflows/build-typescript-release.reusable.yaml: Improved cache key specificity.github/workflows/build-cli-release.reusable.yaml: Enabled Rust cachingTesting Plan
v0.xxx.0tag → verifyis_release_tag=trueand publish jobs run0.xxx.0tag → verifyis_release_tag=trueand publish jobs runPerformance Baseline
From run https://github.com/BoundaryML/baml/actions/runs/19959566319:
build-cli(macOS aarch64): 41.3 minutesbuild-typescript-release(Windows x64): 40.9 minutesbuild-python-release(Linux aarch64): 22.8 minutesAfter cache warmup, these should drop significantly (target: 15-25 minutes for slowest jobs).