Skip to content

ci: add daily fuzz job#331

Merged
apoelstra merged 1 commit into
BlockstreamResearch:masterfrom
gerau:daily-fuzz
May 25, 2026
Merged

ci: add daily fuzz job#331
apoelstra merged 1 commit into
BlockstreamResearch:masterfrom
gerau:daily-fuzz

Conversation

@gerau
Copy link
Copy Markdown
Contributor

@gerau gerau commented May 25, 2026

Addresses #320 (comment).

Mostly copied from rust-bitcoin and from this PR: ElementsProject/rust-elements#267.

@gerau gerau requested a review from delta1 as a code owner May 25, 2026 16:19
~/.cargo/bin
fuzz/target
target
key: cache-${{ matrix.fuzz_target }}-${{ hashFiles('**/Cargo.toml','**/Cargo.lock') }}
Copy link
Copy Markdown
Contributor Author

@gerau gerau May 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also changed this line since it seems to be a bug -- we only have matrix.fuzz_target, but in the rust-bitcoin it references matrix.target

original: https://github.com/rust-bitcoin/rust-bitcoin/blob/9e21bc479bb0041af64a9d50b76672f82597981c/.github/workflows/cron-daily-fuzz.yml#L136

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat! I'll raise an issue on rust-bitcoin. It may be that we're failing to cache build artifacts in the fuzz job because of this.

Copy link
Copy Markdown
Contributor

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d23cc59; successfully ran local tests

@apoelstra apoelstra merged commit d1d31ed into BlockstreamResearch:master May 25, 2026
11 checks passed
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.

2 participants