Skip to content

fix(ci): error on disabled-cache in CI hash calculation#22904

Merged
ludamad merged 2 commits into
merge-train/barretenbergfrom
claudebox/33613d341cf3129d-2
May 4, 2026
Merged

fix(ci): error on disabled-cache in CI hash calculation#22904
ludamad merged 2 commits into
merge-train/barretenbergfrom
claudebox/33613d341cf3129d-2

Conversation

@AztecBot

@AztecBot AztecBot commented May 1, 2026

Copy link
Copy Markdown
Collaborator

rather than silently bypass disabled-cache error loudly when CI calculates a hash that would be disabled-cache. Instead ofWARNING: Noticed changes to rebuild patterns in cache_content_hash we promote to a hard error

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 1, 2026
@federicobarbacovi federicobarbacovi marked this pull request as ready for review May 1, 2026 19:15
@federicobarbacovi federicobarbacovi requested a review from ludamad May 1, 2026 19:16
@AztecBot AztecBot changed the title fix(ci): honor disabled-cache marker in run_test_cmd redis cache fix(ci): error on disabled-cache in CI hash calculation May 1, 2026
@AztecBot AztecBot force-pushed the claudebox/33613d341cf3129d-2 branch from 7cfe7ca to 4c7aa22 Compare May 1, 2026 20:17
@ludamad ludamad added the ci-full Run all master checks. label May 1, 2026
@ludamad ludamad enabled auto-merge (squash) May 4, 2026 14:52
@ludamad ludamad merged commit 6eca7fb into merge-train/barretenberg May 4, 2026
15 checks passed
@ludamad ludamad deleted the claudebox/33613d341cf3129d-2 branch May 4, 2026 14:52
danielntmd pushed a commit to danielntmd/aztec-packages that referenced this pull request May 6, 2026
BEGIN_COMMIT_OVERRIDE
fix(ci): default S3_BUILD_CACHE_AWS_PARAMS in cache_s3_transfer{,_to}
(AztecProtocol#22898)
chore: low-hanging chonk prover fixes from profiling (AztecProtocol#22855)
chore: fuse N `add_scaled` into one `parallel_for` (AztecProtocol#22893)
feat: Delayed merge implementation (AztecProtocol#22775)
chore: numeric audit response (AztecProtocol#22856)
fix: harden BN254 G2 SRS ingress (AztecProtocol#22858)
fix: remove unused hash_challenge variable in batch_merge.test.cpp
(AztecProtocol#22906)
fix(bbup): remove jq dependency (AztecProtocol#22912)
chore: fix g2 test failing on merge-train (AztecProtocol#22920)
fix(ci): error on disabled-cache in CI hash calculation (AztecProtocol#22904)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. ci-full Run all master checks. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants