microsoft / microsoft/litebox

Leverage CI cache to bootstrap the test `test_node_with_rewriter`

Open
#325 0 comments 0 reactions 0 assignees View on GitHub
enhancement layer-META
Dominant language
Rust
Stars
2.7k
Forks
144
Avg merge
12h 21m
Merged PRs (30d)
146

Description

Regarding the rewriting taking a long time, we might be able to set up some sort of cache-checking (do not attempt to rewrite if the rewritten binary already exists and the sha256sum of the input matches the checksum we've stored beside the rewritten binary). This would work pretty well on the CI _without_ checking large binaries in, because the CI _already_ has a caching step for anything that sits inside the `target/` directory (well technically, it caches a subset of even `target/` but we can tweak that quite easily). Anyways, I think that is an orthogonal concern, and need not block merging this PR in. Thanks again!

_Originally posted by @jaybosamiya-ms in https://github.com/microsoft/litebox/pull/300#pullrequestreview-3198496536_

Contributor guide

Open the contributing guide

Research direction

Start with the test_node_with_rewriter test and inspect how rewriting outputs are placed under target/. Review the CI cache configuration and the discussion in pull request 300. Done means the test can reuse an existing rewritten binary when its stored input checksum still matches, without checking large binaries into the repository.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
build-system, ci-cd, testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.