Repository navigation
ci(cache): declare ancestor promotion for dylint and dylint-out (#335) - #422
Merged
Merged
Conversation
zackees
force-pushed
the
cache-promote-ancestor
branch
from
October 5, 2026 02:26
9b6677b to
67ae428
Compare
zackees/ci.yml#335 standardizes promotion on setup-soldr's ancestor encoding (shortest parent-edge distance from a live git DAG walk) rather than a first-parent ordinal, which inverts on wide merges: a 50-commit branch tip reports ordinal 53 while the merge commit containing it reports 4, so a cache saved at the merge looks OLDER than one saved 50 commits before it. #335 keeps the `m<n>` label for offline audit only. dylint (600MB) and dylint-out (700MB) are the two families at or above CACHE-031's 256MB floor, and they are what held this repository's budget -- 87% of its entries were superseded before the janitor work. The other five families sit at or below 210MB and stay unopted: a small family's entry count is noise, not a budget threat. Both families now also carry evict = "lru" (#421), so CACHE-029 is clean. A declaration only: setup-soldr#566 is merged but unreleased (no tag contains it; v0.9.85 predates it; the sanctioned v0 float trails main), so CACHE-030 and CACHE-032 report `needs_review` and the opt-in is inert until a release carries it. Validated against the merged rules: schema clean, CACHE-029 clean, CACHE-031 clean (and it fires for exactly these two families when the opt-in is removed), CACHE-030/032 `needs_review` as expected. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Local-Gate: v1 tree=51c622f181d9a3eb1735a2b528a06a8ea73c3c9e secs=794 lanes=linux-minimal:run,dylint-minimal:run Ci-Attestation: {"at":1791168159,"gate":"general/all/ci-guards","host":"linux-x86_64","key":"38f79479d0d8128012ebe7e9432b03395fa814b16e0610d4812c902473338141","lane":"linux-minimal","parents":["d5e66b030b89cd788d2fca26321a10521a644778"],"secs":302,"stamp":"e4be5d15cb58b396f9cd8877b7cf8c9a","tree":"51c622f181d9a3eb1735a2b528a06a8ea73c3c9e","v":1,"via":"run"} Ci-Attestation: {"at":1791168159,"gate":"rust/x86_64-unknown-linux-gnu/default-facade-clippy","host":"linux-x86_64","key":"38f79479d0d8128012ebe7e9432b03395fa814b16e0610d4812c902473338141","lane":"linux-minimal","parents":["d5e66b030b89cd788d2fca26321a10521a644778"],"secs":302,"stamp":"6de01abf2a62725daef5c66b6361c929","tree":"51c622f181d9a3eb1735a2b528a06a8ea73c3c9e","v":1,"via":"run"} Ci-Attestation: {"at":1791168159,"gate":"rust/x86_64-unknown-linux-gnu/default-facade-test","host":"linux-x86_64","key":"38f79479d0d8128012ebe7e9432b03395fa814b16e0610d4812c902473338141","lane":"linux-minimal","parents":["d5e66b030b89cd788d2fca26321a10521a644778"],"secs":302,"stamp":"ec2c89460937515e87bdc712e134ab74","tree":"51c622f181d9a3eb1735a2b528a06a8ea73c3c9e","v":1,"via":"run"} Ci-Attestation: {"at":1791168159,"gate":"rust/x86_64-unknown-linux-gnu/desktop-status-test","host":"linux-x86_64","key":"38f79479d0d8128012ebe7e9432b03395fa814b16e0610d4812c902473338141","lane":"linux-minimal","parents":["d5e66b030b89cd788d2fca26321a10521a644778"],"secs":302,"stamp":"9bc2fb172dbdc571d2e5c5da787a3407","tree":"51c622f181d9a3eb1735a2b528a06a8ea73c3c9e","v":1,"via":"run"} Ci-Attestation: {"at":1791168159,"gate":"rust/x86_64-unknown-linux-gnu/dylint-all-features-all-targets","host":"linux-x86_64","key":"47939ce85da57bea5dd277f4db1980721a2ff73648eb03e90a9460b64967789a","lane":"dylint-minimal","parents":["d5e66b030b89cd788d2fca26321a10521a644778"],"secs":492,"stamp":"ffc3ed5502e7a89db8e3ff03d78ec5e1","tree":"51c622f181d9a3eb1735a2b528a06a8ea73c3c9e","v":1,"via":"run"}
zackees
force-pushed
the
cache-promote-ancestor
branch
from
October 5, 2026 02:43
67ae428 to
0dc5149
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Second and final pilot application of zackees/ci.yml#335 (tooling half merged as zackees/ci.yml#337), after clud#1802.
What this declares
[cache.promote] mode = "ancestor"(previously absent) pluspromote = "ancestor"on the two families at or above CACHE-031's 256 MB floor:dylint-outdylinttoolchaindepscompilesoldr-miniuvThese two are exactly what held this repository's budget — 87% of its entries were superseded before the janitor work. Opting in declares their keys carry a lineage label, which is what makes a nearest-ancestor search decidable; a key ending in a bare content hash destroys the ancestry at save time.
Validation
A pre-existing finding this surfaced
CACHE-029also fires fordylintanddylint-out— neither declaresevict = "lru", so neither bounds its entry count. This reproduces on unmodifiedmainand is not caused by this change (verified by stashing the diff and re-running). Left alone deliberately to keep this diff to the #335 question, but it's real and worth its own PR.Still a declaration
setup-soldr#566 is merged but unreleased; the opt-in is inert until a release carries it. Request is on zackees/setup-soldr#552.
🤖 Generated with Claude Code