Skip to content

CSUB-2030: Parametrize git-lfs checkout for native builds#877

Merged
atodorov merged 1 commit intousc-devfrom
testing/CSUB-2030-git-lfs-checkout-only-for-release
Apr 29, 2026
Merged

CSUB-2030: Parametrize git-lfs checkout for native builds#877
atodorov merged 1 commit intousc-devfrom
testing/CSUB-2030-git-lfs-checkout-only-for-release

Conversation

@atodorov
Copy link
Copy Markdown
Contributor

when running the majority of test jobs we don't really need chainspecs so don't checkout/build them in.

when running the majority of test jobs we don't really need chainspecs
@atodorov atodorov force-pushed the testing/CSUB-2030-git-lfs-checkout-only-for-release branch from b1c29f3 to 932e2f2 Compare April 27, 2026 14:46
@atodorov atodorov requested review from a team, BradleyOlson64, DylanVerstraete, Trantorian1, beqaabu, creditcoinprotoclaw and didac-gluwa and removed request for a team April 27, 2026 16:00
@atodorov atodorov marked this pull request as ready for review April 27, 2026 16:00
@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 27, 2026

PR Summary

Medium Risk
Changes the reusable build-native.yml workflow interface and updates multiple callers, so any missed caller or incorrect LFS setting could break CI builds or omit required large files.

Overview
Adds a git-lfs-checkout input to the reusable native build workflow (build-native.yml) and wires it into both checkout paths so LFS fetching can be enabled/disabled per job.

Updates all build-native.yml callers (CI, benchmarks, proof-gen, chainspec, release, runtime-upgrade, metadata rebuild) to explicitly set git-lfs-checkout, typically disabling it for most test builds and enabling it for chainspec/release/runtime-upgrade builds.

Reviewed by Cursor Bugbot for commit 932e2f2. Bugbot is set up for automated code reviews on this repo. Configure here.

@atodorov atodorov merged commit d72cb1f into usc-dev Apr 29, 2026
86 of 93 checks passed
@atodorov atodorov deleted the testing/CSUB-2030-git-lfs-checkout-only-for-release branch April 29, 2026 07:14
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