Skip to content

Update CI to inject bitcoin binary version and expected SHA256 into called workflows and actions, move tests up to bitcoin binary 31.1 (was 26.0) - #7700

Open
dhaney-stacks wants to merge 5 commits into
stacks-network:mainfrom
dhaney-stacks:main
Open

dhaney-stacks wants to merge 5 commits into
stacks-network:mainfrom
dhaney-stacks:main

Conversation

@dhaney-stacks

@dhaney-stacks dhaney-stacks commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Update CI to inject bitcoin binary version and expected SHA256 into called workflows and actions, move tests up to bitcoin binary 31.1 (was 26.0)

Applicable issues

N/A

Additional info (benefits, drawbacks, caveats)

Moving to bitcoin binary 31.1 keeps us current with our L1 provider code and ensures things are working as expected.

Note: I've gotta leave this in place until this PR merges, at which point I can delete it via another PR. I have fixed this "chicken and the egg" problem permanently by allowing an actor to specify the bitcoin binary version (and hash) when manually triggering cache builds off main which will allow us to upgrade in the future without having to do any weird stuff like this.

Details:

https://github.com/stacks-network/stacks-core/pull/7700/changes#diff-8cf7f1d9842e5c571ad948417fa7fa400504ef4bc5c02d6ac52233e5cf814b18R35-R42

Checklist

N/A

…alled workflows and actions, move tests up to bitcoin binary 31.1 (was 26.0)
… because 31.1 has a different fee estimation than 26.0
@dhaney-stacks
dhaney-stacks force-pushed the main branch 2 times, most recently from 44b1c86 to 81f1d13 Compare October 2, 2026 19:05
@dhaney-stacks

dhaney-stacks commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator Author

Note: I've gotta leave this in place until this PR merges, at which point I can delete it via another PR. I have fixed this "chicken and the egg" problem permanently by allowing an actor to specify the bitcoin binary version (and hash) when manually triggering cache builds off main which will allow us to upgrade in the future without having to do any weird stuff like this.

Details:

https://github.com/stacks-network/stacks-core/pull/7700/changes#diff-8cf7f1d9842e5c571ad948417fa7fa400504ef4bc5c02d6ac52233e5cf814b18R35-R42

@coveralls

coveralls commented Oct 2, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 85.687% (+0.02%) from 85.668% — dhaney-stacks:main into stacks-network:main

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants