Skip to content

fuzz: Disable shuffle when merge=1#21023

Merged
maflcko merged 1 commit intobitcoin:masterfrom
maflcko:2101-fuzzMerge
Feb 11, 2021
Merged

fuzz: Disable shuffle when merge=1#21023
maflcko merged 1 commit intobitcoin:masterfrom
maflcko:2101-fuzzMerge

Conversation

@maflcko
Copy link
Copy Markdown
Member

@maflcko maflcko commented Jan 28, 2021

This should hopefully help make the deletion of fuzz inputs more deterministic.

My tests (N=1) revealed that without this patch 7000 files differ (bitcoin-core/qa-assets#44 (comment)). With this patch, "only" 2000 files differ.

@practicalswift
Copy link
Copy Markdown
Contributor

cr ACK fabeb5b: -shuffle=0 and -prefer_small=1 make sense

@maflcko maflcko merged commit dd0521b into bitcoin:master Feb 11, 2021
@maflcko maflcko deleted the 2101-fuzzMerge branch February 11, 2021 09:36
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 11, 2021
fabeb5b fuzz: Disable shuffle when merge=1 (MarcoFalke)

Pull request description:

  This should hopefully help make the deletion of fuzz inputs more deterministic.

  My tests (N=1) revealed that without this patch 7000 files differ (bitcoin-core/qa-assets#44 (comment)). With this patch, "only" 2000 files differ.

ACKs for top commit:
  practicalswift:
    cr ACK fabeb5b: `-shuffle=0` and `-prefer_small=1` make sense

Tree-SHA512: 21a701f52450d402a91dd6e0b33d564c63a9c3b919738eb9a80c24d48fc5b964088e325470738f39af0d595612c844acc7bf0941590cc2dc8c6f6ee4cb69c861
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants