Skip to content

test: Add txindex migration test - #23046

Merged
maflcko merged 1 commit into
bitcoin:masterfrom
maflcko:2109-txindexMigrate
Nov 15, 2021
Merged

test: Add txindex migration test#23046
maflcko merged 1 commit into
bitcoin:masterfrom
maflcko:2109-txindexMigrate

Conversation

@maflcko

@maflcko maflcko commented Sep 20, 2021

Copy link
Copy Markdown
Member

Test for #22626

@theStack

Copy link
Copy Markdown
Contributor

Concept ACK

1 similar comment
@dunxen

dunxen commented Sep 25, 2021

Copy link
Copy Markdown
Contributor

Concept ACK

@practicalswift

Copy link
Copy Markdown
Contributor

Concept ACK (obviously :))

@maflcko
maflcko force-pushed the 2109-txindexMigrate branch from fafecc9 to fadc4c7 Compare November 9, 2021 11:06
@maflcko

maflcko commented Nov 9, 2021

Copy link
Copy Markdown
Member Author

Removed now-redundant sync_all.

@theStack theStack left a comment

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.

Tested ACK fadc4c7 🌁
(code-reviewed test, ran it under Debian bookworm/sid, with downloaded binary previous releases v0.16.3 and v0.20.1)

@maflcko
maflcko merged commit 41a1b5f into bitcoin:master Nov 15, 2021
@maflcko
maflcko deleted the 2109-txindexMigrate branch November 15, 2021 09:05
@hebasto

hebasto commented Nov 15, 2021

Copy link
Copy Markdown
Member

See #23514.

@bitcoin bitcoin locked and limited conversation to collaborators Nov 15, 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.

6 participants